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

Possible unused mount in core deployment (/etc/core/token) #21706

Open
gira0 opened this issue Mar 6, 2025 · 1 comment · May be fixed by #21709
Open

Possible unused mount in core deployment (/etc/core/token) #21706

gira0 opened this issue Mar 6, 2025 · 1 comment · May be fixed by #21709

Comments

@gira0
Copy link

gira0 commented Mar 6, 2025

Is your feature request related to a problem? Please describe.
Hi, I recently had some issues with the "/etc/core/token" mount called "psc" in the bitnami/helm chart, and I'm trying to trace it to the source. I suspect the lines below are the origin of this mount.

- name: psc
mountPath: /etc/core/token

- name: psc
mountPath: /etc/core/token

I've tested the helm chart with the mount removed, and it seems to work perfectly fine without it. I tried looking for the origin, saw it in the goharbor/harbor-helm chart and then also here in the test data. Then I tried searching the code to figure out what interacts with this mount, but I can't find anything. I would love to know if the removal of the mount is breaking something and if yes, what.

BTW, my original error that caused this search was this:

Error: container create failed: creating /etc/core/token: openat2 "etc/core/token": No such file or directory

This is what I got on my microshift instance. Without the mount in the container, everything is working.

Describe the solution you'd like
I would love to know if this mount is really needed or not, so that I or someone else can submit a patch to remove it or realize my error and go back to debugging, now with new motivation due to the consequences of this mount missing.

Thanks in advance for any input regarding this.

@Vad1mo
Copy link
Member

Vad1mo commented Mar 7, 2025

good catch, this is basically inside the helm chart since the first commit:

Image

Image

it is not used.

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

Successfully merging a pull request may close this issue.

4 participants