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

libpod.conf: Podman's conmon path on openSUSE #698

Closed
wants to merge 1 commit into from

Conversation

vrothberg
Copy link
Member

Add the path to Podman's conmon for openSUSE and SLE.

Signed-off-by: Valentin Rothberg vrothberg@suse.com

Add the path to Podman's conmon for openSUSE and SLE.

Signed-off-by: Valentin Rothberg <vrothberg@suse.com>
@rhatdan
Copy link
Member

rhatdan commented Apr 30, 2018

LGTM

@mheon
Copy link
Member

mheon commented Apr 30, 2018

LGTM. Travis is a known failure while we sort out test issues, I'll rekick Fedora.

@mheon
Copy link
Member

mheon commented Apr 30, 2018

bot, retest this please

@rhatdan
Copy link
Member

rhatdan commented Apr 30, 2018

@vrothberg I just got something in that should fix travis tests, but I think you will need to rebase your PR>

@TomSweeneyRedHat
Copy link
Member

LGTM other than the unhappy Travis tests that @rhatdan has a fix for.

@mheon
Copy link
Member

mheon commented Apr 30, 2018

I'm going to merge this, given it's just a trivial config change.
@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 1dc8f41 has been approved by mheon

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit 1dc8f41 with merge 2239093...

@rh-atomic-bot
Copy link
Collaborator

☀️ Test successful - status-papr
Approved by: mheon
Pushing 2239093 to master...

@vrothberg vrothberg deleted the libpodconf-opensuse branch January 7, 2019 09:45
mheon added a commit to mheon/libpod that referenced this pull request Aug 24, 2020
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 <mheon@redhat.com>
mheon added a commit to mheon/libpod that referenced this pull request Aug 25, 2020
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 <mheon@redhat.com>
@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants