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: add support for monitor chart securityContext and labels #355

Merged
merged 2 commits into from
Mar 20, 2024

Conversation

mjnagel
Copy link
Contributor

@mjnagel mjnagel commented Jan 16, 2024

Certain environments may use different images for the exporter which require a different run as user/group (i.e. a hardened image which is based on a different base image). There are also environments where certain security contexts may be enforced (i.e. no running as root, all capabilities must be dropped, etc). In these cases it is beneficial to expose the securityContext (and containerSecurityContext).

It's also generally nice to expose pod labels.

@Racer159
Copy link

Additional context, there are forks of this chart that implement these features for hardened environments but it would be nice for the community at large to benefit from some of these options:

https://repo1.dso.mil/big-bang/product/packages/neuvector

Excited to see this potentially get in!

@becitsthere becitsthere merged commit ed3bc75 into neuvector:master Mar 20, 2024
1 check passed
@mjnagel mjnagel deleted the monitor-flexibility branch March 20, 2024 00:39
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