Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add readOnlyRootFilesystem security context to pass trivy plugin 0.18… #33

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gradientsearch
Copy link

add readOnlyRootFilesystem security context to pass trivy plugin 0.18.4 scans

@gradientsearch gradientsearch requested a review from a team as a code owner February 14, 2024 16:34
@prezha
Copy link

prezha commented Feb 14, 2024

are we sure this will work with the r/o root fs?

usually, if there's a reason not to have r/o root fs, you'd have to define a separate mount volume to cover the path that still needs to be writable

ie, needs testing before crdb ends up in a weird situation

@prezha
Copy link

prezha commented Feb 14, 2024

looks like upstream has some of these set already in statefulset at head (here and here) - we might be a bit behind

but can't find the toolbox deployment (perhaps something we've added)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants