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 edited clamav chart with podSecurityContext: {} #44

Open
wants to merge 2 commits into
base: release/3.1.0-beta1
Choose a base branch
from

Conversation

gislab-augsburg
Copy link

In the OpenShift environment at Landeshauptstadt München (LHM) we are restricted from using predefined users or groups. The clamav subchart values for PodSecurityContext , runAsUserand runAsGroup cannot be updated via values.yaml to {} or Null because of an unresolved helm issue.

Our workaround is to add the edited clamav chart directly to the repo instead of pulling it from wiremind. The only changed value is PodSecurityContext: https://github.com/it-at-m/sddi-ckan-k8s/blob/46f8e82d4cdf712b2ab255774fb0dc5d5f74a88e/charts/sddi-ckan/charts/clamav/values.yaml#L30

@BWibo
Copy link
Member

BWibo commented Nov 7, 2024

@gislab-augsburg
The issue is the same as we resolved it in #32 for the initContainers etc., right?
Copying the entire chart is not ideal, as this brings us in the position to keep maintaining this. Maybe we could try submitting a PR to wiremind, introducing the solution we used in #32.

@BWibo BWibo self-assigned this Nov 7, 2024
@BWibo BWibo added type: fix Iterations on existing features or infrastructure. state: pending Pending requirements, dependencies, or data. and removed state: pending Pending requirements, dependencies, or data. labels Nov 7, 2024
@gislab-augsburg
Copy link
Author

@BWibo
Yes, it is similar to part of the issue we resolved in #32. The difference is that we do not have init containers which we can simply replace in the wiremind chart. And also I am not sure if this would help because the clamav user is maybe also needed during runtime.
What do you think about the following little change we could submit as PR to wiremind?
gislab-augsburg/wiremind-helm-charts@e858abc

@gislab-augsburg
Copy link
Author

gislab-augsburg commented Nov 14, 2024

I updated to the current version of the wiremind helm chart because LHM IT-Security prefers the original clamav/clamav image over ghcr.io/mailu/clamav and with the current wiremind version we could resolve another permission issue we had with the clamav/clamav image with entrypoint: /init-unprivileged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: fix Iterations on existing features or infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants