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.

Sign up for GitHub

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

Jump to bottom

[Feature][Doc] Explain that RBAC should be synchronized manually #640

Closed
2 tasks done
kevin85421 opened this issue Oct 17, 2022 · 0 comments · Fixed by #641
Closed
2 tasks done

[Feature][Doc] Explain that RBAC should be synchronized manually #640

kevin85421 opened this issue Oct 17, 2022 · 0 comments · Fixed by #641
Assignees
@kevin85421
Labels
enhancement New feature or request

Comments

@kevin85421
Copy link
Member

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Description

Currently, RBAC YAML files in helm-chart/kuberay-operator/templates and ray-operator/config/rbac should be synchronized manually. We have pointed this out in DEVELOPMENT.md, but it is not easy for users to find the messages. Hence, we need to make this information become more explicit in both DEVELOPMENT.md and rbac-check.py.

Thank @AlessandroPomponio for pointing this out!

Use case

No response

Related issues

#631

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!
@kevin85421 kevin85421 added the enhancement New feature or request label Oct 17, 2022
@kevin85421 kevin85421 self-assigned this Oct 17, 2022
@kevin85421 kevin85421 mentioned this issue Oct 17, 2022
Merged
4 tasks
@kevin85421 kevin85421 changed the title [Feature][Doc] RBAC should be synchronized manually [Feature][Doc] Explain that RBAC should be synchronized manually Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kevin85421 kevin85421

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Feature][Doc] Explain that RBAC should be synchronized manually kevin85421/kuberay
1 participant
@kevin85421