-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
libpod.conf: Podman's conmon path on openSUSE #698
Conversation
Add the path to Podman's conmon for openSUSE and SLE. Signed-off-by: Valentin Rothberg <[email protected]>
LGTM |
LGTM. Travis is a known failure while we sort out test issues, I'll rekick Fedora. |
bot, retest this please |
@vrothberg I just got something in that should fix travis tests, but I think you will need to rebase your PR> |
LGTM other than the unhappy Travis tests that @rhatdan has a fix for. |
I'm going to merge this, given it's just a trivial config change. |
📌 Commit 1dc8f41 has been approved by |
☀️ Test successful - status-papr |
We need this release out by end of day, so we don't have time to do this right. Disable the vendor task and manually add c/storage PR containers#698 to the vendored copy of c/storage to make the tests pass. Once containers#698 merges into c/storage, we need to remove this commit and backport it to the v1.20 stable branch, then cut a release there. Signed-off-by: Matthew Heon <[email protected]>
This reverts commit ae2ee65. This was a horrible hack that we did for time reasons, to get 2.0.5 out on schedule; now that it has been merged into c/storage properly we no longer need or want it. Signed-off-by: Matthew Heon <[email protected]>
Add the path to Podman's conmon for openSUSE and SLE.
Signed-off-by: Valentin Rothberg [email protected]