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
Do not link the /run/runc
volume unless invoker_use_runc is set
#4149
Do not link the /run/runc
volume unless invoker_use_runc is set
#4149
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.
Ran into this problem when using the latest Docker for Mac. Thanks for the fix!
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.
LGTM
- name: set invoker runc volume | ||
set_fact: | ||
volumes: "{{ volumes }},/run/runc:/run/runc" | ||
when: invoker_use_runc == true |
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.
invoker_use_runc
is not reachable here.
it probably should be when: invoker.useRunc
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.
Thanks. Updated.
Codecov Report
@@ Coverage Diff @@
## master #4149 +/- ##
==========================================
- Coverage 84.74% 80.77% -3.97%
==========================================
Files 151 152 +1
Lines 7277 7450 +173
Branches 466 487 +21
==========================================
- Hits 6167 6018 -149
- Misses 1110 1432 +322
Continue to review full report at Codecov.
|
Travis passed! 🎉 |
And it worked for me too.I would be happy to see this merged soon. |
PG3 3124 🏃 |
PG3 3124 👍 -- 4 unrelated failures, otherwise good. |
…che#4149) * Do not link the `/run/runc` volume unless invoker.useRunc is set
On Mac,
/run/runc
doesn't exist and trying to set it as a volume causes an error.My changes affect the following components
Types of changes
Checklist: