-
-
Notifications
You must be signed in to change notification settings - Fork 231
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 Singularity support for "dockerFile" #1938
add Singularity support for "dockerFile" #1938
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @misterbrandonwalker !
Can you add a test?
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1938 +/- ##
==========================================
- Coverage 83.77% 83.58% -0.19%
==========================================
Files 46 46
Lines 8201 8217 +16
Branches 2178 2146 -32
==========================================
- Hits 6870 6868 -2
- Misses 855 863 +8
- Partials 476 486 +10 ☔ View full report in Codecov by Sentry. |
@misterbrandonwalker Does |
The test passes for me with tox I think the issue now is with how I am using current working directory rather than /tmp I see permissions error on the runner However I added to comment here if I try to use /tmp to execute singularity it will crash FATAL: Unable to create build: 'noexec' mount option set on /tmp, temporary root filesystem |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @misterbrandonwalker !
Add ability to use
dockerFile
with singularity.