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

add security context for non root user #62

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

bmaguireibm
Copy link

This PR would add a security context section to the GeoServer Pod to take into account of the fact that newer docker images of GeoServer runs with a non root user and thus encounters a permissions error when accessing mounted volumes.

@NyakudyaA
Copy link
Collaborator

@bmaguireibm Can you check the failing test if it's related to your changes

@bmaguireibm
Copy link
Author

Hi @NyakudyaA, I'm afraid I'm not very familiar with these tools very much, but from my investigation it doesn't look related to my changes, the error is Error: direnv: command not found, so it seems like the job is trying to use a binary that's not included in the image that it's running in. Maybe it used to be part of ubuntu-latest and a recent change of the image removed it? I have just tried adding a step beforehand to install direnv, not sure if this is the best approach though.

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.

2 participants