Skip to content

Added security context to console deployment#244

Merged
dimas-b merged 2 commits into
apache:mainfrom
cccs-cat001:main
Jun 30, 2026
Merged

Added security context to console deployment#244
dimas-b merged 2 commits into
apache:mainfrom
cccs-cat001:main

Conversation

@cccs-cat001

Copy link
Copy Markdown
Contributor

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.

@dimas-b dimas-b left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍 Thanks for the contribution, @cccs-cat001 !

@MonkeyCanCode

MonkeyCanCode commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

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,
Yong Zheng

@dimas-b

dimas-b commented Jun 29, 2026

Copy link
Copy Markdown

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 😅

@MonkeyCanCode

Copy link
Copy Markdown
Contributor

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 ;)

@dimas-b dimas-b merged commit 9cf3735 into apache:main Jun 30, 2026
0 of 2 checks passed
@dimas-b

dimas-b commented Jun 30, 2026

Copy link
Copy Markdown

@MonkeyCanCode : I merged this. Please rebase your bigger PR - thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants