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.

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

feat: support securityContext and podSecurityContext configuration #265

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

wparr-circle
Copy link
Contributor

@wparr-circle wparr-circle commented Jun 24, 2024

Fixes #264

Adds two new helm values controller.securityContext, and controller.podSecurityContext - these correspond to Container level Security Context and Pod level Security Context respectively.
This may be leveraged to setup a security context complaint with pod security standards etc.

@wparr-circle wparr-circle requested a review from a team as a code owner June 24, 2024 13:29
… helm chart

Signed-off-by: wparr-circle <william.parr@circle.com>
ushitora-anqou
ushitora-anqou previously approved these changes Jun 25, 2024
@ushitora-anqou ushitora-anqou self-requested a review June 25, 2024 00:42
@ushitora-anqou ushitora-anqou dismissed their stale review June 25, 2024 00:42

I forgot to check that the CI passed.

Copy link
Contributor

@ushitora-anqou ushitora-anqou left a comment

Choose a reason for hiding this comment

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

Could you please update the README using this guide? Thanks!

Signed-off-by: wparr-circle <william.parr@circle.com>
@wparr-circle
Copy link
Contributor Author

Could you please update the README using this guide? Thanks!

@ushitora-anqou my bad! I've pushed the updated readme in befc39d
Thanks!

@pluser pluser merged commit 913ffe9 into topolvm:main Jul 8, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Support overriding Security Context at Pod and Container Level
3 participants