WiP: update our singularity docker shim to fixup tests etc #42
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reported (well - asked if feature or a bug) Vanessa over IM. Versions of the docker containers after 3.3.0 lack suid set for the started-suid helper. That prevents running singularity within docker container under a normal user privileges.
|
@yarikoptic let me know how I can help. I can manually push the base with setuid for that particular slim tag, but it still remains a mystery why it wasn't there via the automated build. |
|
Thank you @vsoch! I think with ad-hoc chmod as in the PR that issue is a non-issue for us ;) yet to figure out what other things have changed |
This was referenced Dec 10, 2020
Since 3.6.1 there is now /etc/localtime within those docker containers. That forbids us to bind mount the correct one from outside, so I moved it aside (may be we would need to adjust further, in particular whenever there is no /etc/localtime outside -- may be we would need to provide/bind mount some fixed one then
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Unfortunately we do not have (yet) cron job running tests, so at some point things went awry and I am not sure when. This PR is WiP toward resolving them.
Part of the problem precluding update of our shim is also that singularity docker images are flawed - have no suid set, reported to @vsoch .
When done, it should close #40