Added security context to console deployment#244
Conversation
dimas-b
left a comment
There was a problem hiding this comment.
LGTM 👍 Thanks for the contribution, @cccs-cat001 !
|
hi @cccs-cat001 @dimas-b, Those are already covered in https://github.com/apache/polaris-tools/pull/234/changes. Maybe we should merge that one instead as it has a lot more enhancements. Thanks, |
|
I'm fine with merging #234 , but this is a smaller change... it might be easier to merge it first to enable security contexts, then do the bigger refactoring later... but I do not really have a preference 😅 |
No concern from me from both approaches. Either merge this then I can refactor the other one or push @jbonofre to review the other one ;) |
|
@MonkeyCanCode : I merged this. Please rebase your bigger PR - thx! |
We've run into issues running the console due to our cluster admins enforcing a default security context that misaligns with the console pod. Adding the security context block(s) will allow us to set the correct user/group security context to have it work.