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.

Sign up for GitHub

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

Jump to bottom

geonetwork: double fix permissions issue #84

Open

edevosc2c opened this issue Feb 13, 2024 · 0 comments

Labels

bug

Member

edevosc2c commented Feb 13, 2024 •

edited

Loading

in the geonetwork deployment, the permissions are fixed twice:

one through an initcontainer: https://github.com/georchestra/helm-georchestra/blob/main/templates/geonetwork/geonetwork-deployment.yaml#L41-L46
one inside the container: https://github.com/georchestra/geonetwork/blob/georchestra-gn4.2.x/web/src/docker/docker-entrypoint.d/00-adjust-perms

we should only keep one script that fixes the permissions.

I'm for removing it inside the geonetwork docker image because for the official docker compose image the permissions are already set correctly: https://betterstack.com/community/questions/what-is-the-best-way-to-manage-permissions-for-docker-shared-volumes/#use-named-volumes

edevosc2c added the bug label

edevosc2c changed the title ~~geonetwork: double fix permissions~~ geonetwork: double fix permissions issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment