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

helm: Set rthooks.podSecurityContext to empty by default #2931

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

michi-covalent
Copy link
Contributor

Set rthooks.podSecurityContext to empty by default to be consistent with the security context setting of Tetragon pods. Also note that "privileged" setting only applies to container security context 12.

Footnotes

  1. https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.31/#podsecuritycontext-v1-core

  2. https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.31/#securitycontext-v1-core

Set rthooks.podSecurityContext to empty by default to be consistent with
the security context setting of Tetragon pods. Also note that
"privileged" setting only applies to container security context [^1][^2].

[^1]: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.31/#podsecuritycontext-v1-core
[^2]: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.31/#securitycontext-v1-core

Signed-off-by: Michi Mutsuzaki <[email protected]>
@michi-covalent michi-covalent added the release-note/misc This PR makes changes that have no direct user impact. label Sep 18, 2024
Copy link

netlify bot commented Sep 18, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 1218ec8
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66ea20ea9a752100084f6f77
😎 Deploy Preview https://deploy-preview-2931--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michi-covalent michi-covalent marked this pull request as ready for review September 18, 2024 00:41
@michi-covalent michi-covalent requested a review from a team as a code owner September 18, 2024 00:41
@kkourt kkourt added the needs-backport/1.2 This PR needs backporting to 1.2 label Sep 18, 2024
@michi-covalent michi-covalent merged commit 2558359 into main Sep 18, 2024
50 checks passed
@michi-covalent michi-covalent deleted the pr/michi/helm branch September 18, 2024 14:58
@kkourt kkourt added backport-done/1.2 PR backport done. and removed needs-backport/1.2 This PR needs backporting to 1.2 labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.2 PR backport done. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants