-
Notifications
You must be signed in to change notification settings - Fork 403
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
[autoscaler] Expose autoscaler container security context. #752
[autoscaler] Expose autoscaler container security context. #752
Conversation
Signed-off-by: Dmitri Gekhtman <dmitri.m.gekhtman@gmail.com>
Signed-off-by: Dmitri Gekhtman <dmitri.m.gekhtman@gmail.com>
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.
Thank @DmitriGekhtman for this contribution! We should create a namespace with restricted
policies to test it. Follow the instructions in https://github.com/ray-project/kuberay/pull/750/files.
Signed-off-by: Dmitri Gekhtman <dmitri.m.gekhtman@gmail.com>
TODO: Try it in a namespace with restricted policies. |
Signed-off-by: Dmitri Gekhtman <dmitri.m.gekhtman@gmail.com>
@kevin85421 I just tested this with your securityContext instructions and it worked exactly as expected -- I was able to deploy a cluster and trigger autoscaling. |
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.
LGTM
…ct#752) Expose configuration for the autoscaler container's security context. Signed-off-by: Dmitri Gekhtman <dmitri.m.gekhtman@gmail.com>
Signed-off-by: Dmitri Gekhtman dmitri.m.gekhtman@gmail.com
Why are these changes needed?
Allows setting the autoscaler container's security context.
Related issue number
Checks