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

post-install-cvmfs-fix dosent-work when CVMFS is deployed as external dependency #494

Closed
NilsKrattinger opened this issue Aug 6, 2024 · 1 comment · Fixed by #498
Closed

Comments

@NilsKrattinger
Copy link
Contributor

NilsKrattinger commented Aug 6, 2024

Hi,

It seems that the post-install-cvmfs-fix is not able to terminate / run when cvmfs is deployed in another namespace.
I think this is because the namespace is retrieved from the .Release. in configmap-cvmfs-fix example.

I don't have the history of the reason for this workaround, and I don't know if it's necessary even when CVMFS is deployed cluster-wide (I think it is).

If the fix is necessary, could we set the CVMFS namespace as a value? This would probably require a dedicated SA with permissions on the CVMFS namespace.

If not, should the fix be conditional on cvmfs.deploy ?

If you could point me in the right direction here, I'd be happy to do this PR!

Pod Logs :

 kubectl logs -n galaxy-dev dev01-post-install-cvmfs-fix-job-mqr62
---
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
Waiting on nodeplugin pod to enter 'Running' status. Currently ''.
@NilsKrattinger NilsKrattinger changed the title post-install-cvmfs-fix dosent-work when CVMFS is deployed as external dependency post-install-cvmfs-fix dosent-work when CVMFS is deployed as external dependency Aug 6, 2024
@ksuderman
Copy link
Contributor

Yeah, it looks like the fix assumes CVMFS will be in the .Release.Namespace. We could do with better logic when it comes to locating the CVMFS nodeplugin. It might also be a good idea to make the fix conditional on a flag set in the values.yaml file; something like cvmfs.enable-fix=false.

The fix is conditional on CVMFS being the refdata source and we do have an open PR to remove it. We just haven't felt confident enough to apply it yet.

The fix is a result of older versions of the nodeplugin hanging after installation. Restarting the pod resolved that problem.

Any and all PR's are always welcome!

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 a pull request may close this issue.

2 participants