Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected: Additional Details Added to Aqua Troubleshooting Documentation For Auth Errors #568

Merged
merged 5 commits into from
Apr 3, 2025

Conversation

elizjo
Copy link
Member

@elizjo elizjo commented Mar 31, 2025

Added details to generally troubleshoot auth errors in AQUA. Added headers to allow us to throw errors from AQUA directly referencing solutions on the troubleshooting page.

For example:
if error is 404, list_log_groups --> we directly link the troubleshooting page and the section "list log group" within the Authorization Errors section.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Mar 31, 2025
mrDzurb
mrDzurb previously approved these changes Apr 1, 2025
- [Image Payload is Not Working](#image-payload-is-not-working)
- [Prompt Completion Payload is Not Working](#prompt-completion-payload-is-not-working)

# Authorization Issues
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be "## Authorization Issues" instead? Otherwise if seems everything is grouped currently under authorization issues.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think h1 here is ok. Troubleshooting MD should contain everything other than Authorization issue. We will have an h1 for FT and Evaluation

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put h1 auth header below section with h1 Troubleshooting header.

@@ -1,4 +1,133 @@
# Model Deployment

# Troubleshooting Model Deployment
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything outside Authorization is subsection of Troubleshooting MD

mrDzurb
mrDzurb previously approved these changes Apr 2, 2025
@mayoor mayoor merged commit c1515d5 into oracle-samples:main Apr 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants