-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add more info on the scheduler group issues #467
Conversation
✅ Deploy Preview for nebari-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
If your user is not part of one of the mentioned groups, you might see an error | ||
like this one when clicking on the job details: | ||
|
||
`Server returned status code 403 with message: 'workflows.argoproj.io is forbidden: User "system:serviceaccount:dev:argo-viewer" cannot create resource "workflows" in API group "argoproj.io" in the namespace "dev"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I use single backticks here on purpose: it allows showing the entire message inline and wraps it around to content width while still formatting it as monospace. Using three backticks would either make it too long (if one line) or would require breaking it into multiple lines (but then it would be harder for users to find this message if they search the docs by error message).
|
||
<div align="center"> | ||
<img src="/img/tutorials/jupyter-scheduler-permission-error.png" alt="Jupyter-Scheduler UI - permission error." width="100%"/> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image pretty much duplicates the text above, but I think it's nice to have both here. The textual error is good for search and the image is good for figuring out where to look or how it should look in the UI.
Thanks for double-checking it, @nkaretnikov. This looks good to me already, though can you also add an FAQ section on this error? Maybe describe it lightly in the FAQ and point to this document for a general overview. |
@viniciusdc Added FAQ, moved notes closer to the beginning of the page. Added paragraphs, so they could be referenced. PTAL. |
@kcpevey could you have a look if this is still relevant? I can do a review later and merge if all seems right (not outdated) |
We have pushed changes to jupyter scheduler to improve error messaging. The messaging is much more descriptive and useful. I appreciate the effort here, but it has been superseded by the changes in the tool itself. |
Reference Issues or PRs
Related:
Despite what these two issues say, the docs are correct. Jobs are scheduled even if only the
admin
group is used. I've just double-checked. An error is also already shown when there's a permission error. So I've just added more information on how the groups should be configured and some troubleshooting info.What does this implement/fix?
Put a
x
in the boxes that applyTesting
Documentation
Access-centered content checklist
Text styling
H1
or#
in markdown).Non-text content
Any other comments?