Skip to content

podmanv2 container exists|wait#5571

Merged
openshift-merge-robot merged 4 commits into
containers:masterfrom
baude:v2exists
Mar 20, 2020
Merged

podmanv2 container exists|wait#5571
openshift-merge-robot merged 4 commits into
containers:masterfrom
baude:v2exists

Conversation

@baude
Copy link
Copy Markdown
Member

@baude baude commented Mar 20, 2020

enable container exists and wait for podmanv2

Signed-off-by: Brent Baude bbaude@redhat.com

@baude baude changed the title podmanv2 container exists|wait [WIP]podmanv2 container exists|wait Mar 20, 2020
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 20, 2020
@openshift-ci-robot
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 20, 2020
Comment thread cmd/podmanV2/containers/inspect.go Outdated
@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Mar 20, 2020

Is there a way we could get the test suite to run against these. At some point we want to fire up the tests to make sure podman container exists is actually working.

enable container exists and wait for podmanv2

Signed-off-by: Brent Baude <bbaude@redhat.com>
in the case of exists, use a boolreport structure so that responses can be consistent pointer and error

Signed-off-by: Brent Baude <bbaude@redhat.com>
@baude
Copy link
Copy Markdown
Member Author

baude commented Mar 20, 2020

Is there a way we could get the test suite to run against these. At some point we want to fire up the tests to make sure podman container exists is actually working.

No and then yes, agree. We need enough function to be able to run the test suite first.

Signed-off-by: Brent Baude <bbaude@redhat.com>
@baude baude changed the title [WIP]podmanv2 container exists|wait podmanv2 container exists|wait Mar 20, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 20, 2020
Comment thread cmd/podmanV2/containers/exists.go Outdated
Comment thread cmd/podmanV2/containers/inspect.go Outdated
Comment thread cmd/podmanV2/containers/wait.go Outdated
Comment thread cmd/podmanV2/containers/wait.go Outdated
Comment thread pkg/domain/infra/abi/containers.go Outdated
Comment thread pkg/domain/infra/tunnel/containers.go Outdated
enable remote container wait with condition

Signed-off-by: Brent Baude <bbaude@redhat.com>
@baude
Copy link
Copy Markdown
Member Author

baude commented Mar 20, 2020

updated

@jwhonce
Copy link
Copy Markdown
Member

jwhonce commented Mar 20, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 20, 2020
@openshift-merge-robot openshift-merge-robot merged commit 7a095af into containers:master Mar 20, 2020
@baude baude deleted the v2exists branch May 18, 2022 15:22
@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 21, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. 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.

6 participants