Skip to content

Add allowPrivilegeEscalation to the securityContexts#117

Closed
beahues wants to merge 2 commits intotryretool:mainfrom
beahues:main
Closed

Add allowPrivilegeEscalation to the securityContexts#117
beahues wants to merge 2 commits intotryretool:mainfrom
beahues:main

Conversation

@beahues
Copy link

@beahues beahues commented Aug 30, 2023

It's mentioned in values.yaml but not actually reflected in the code,
but enable configuring the PodSecurity policy option for
allowPrivilegeEscalation to the chart.

It's mentioned in `values.yaml` but not actually reflected in the code,
but enable configuring the PodSecurity policy option for
`allowPrivilegeEscalation` to the chart.
@beahues
Copy link
Author

beahues commented Aug 30, 2023

Addresses a tiny amount of #57

@beahues beahues marked this pull request as ready for review August 30, 2023 19:27
@cedenilla
Copy link

can we include runAsNonRoot as well? It will help us a lot and it's a minimal change.

As this is valid in either SecurityContext and PodSecurityContext, we
don't have to worry too much about the section this is in.
@lmb-djaquier
Copy link

Hi!

Could be also great to add readOnlyRootFilesystem: true

And merge that PR as well :-) thanks for the job!

@jjlgao
Copy link
Contributor

jjlgao commented Jan 22, 2025

Will supersede this PR with this change, as our helm chart has migrated quite a bit: #206

@jjlgao jjlgao closed this Jan 22, 2025
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.

4 participants