Skip to content

Conversation

@baude
Copy link
Member

@baude baude commented Apr 19, 2019

first pass at enabling a swath of integration tests for the
remote-client.

Signed-off-by: baude [email protected]

@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 Apr 19, 2019
@openshift-ci-robot
Copy link
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 approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M labels Apr 19, 2019
@baude baude force-pushed the makeitrain branch 6 times, most recently from 5a3596a to dd94294 Compare April 22, 2019 19:21
@baude baude force-pushed the makeitrain branch 2 times, most recently from 6e268f3 to 8759f45 Compare April 24, 2019 20:13
@baude baude changed the title [wip] enable integration tests for remote-client enable integration tests for remote-client Apr 24, 2019
@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 Apr 24, 2019
@jwhonce
Copy link
Member

jwhonce commented Apr 24, 2019

LGTM

@rh-atomic-bot
Copy link
Collaborator

☔ The latest upstream changes (presumably #3014) made this pull request unmergeable. Please resolve the merge conflicts.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 26, 2019
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 28, 2019
@baude
Copy link
Member Author

baude commented Apr 28, 2019

@giuseppe any theories on the failed rootless tests?

@baude
Copy link
Member Author

baude commented Apr 29, 2019

Needs varlink/go#6

@baude baude force-pushed the makeitrain branch 3 times, most recently from 0286e93 to 9b9e228 Compare April 30, 2019 17:08
@baude baude force-pushed the makeitrain branch 11 times, most recently from cdf2003 to 86c1fcf Compare May 7, 2019 15:28
@baude
Copy link
Member Author

baude commented May 7, 2019

fixed up

first pass at enabling a swath of integration tests for the
remote-client.

Signed-off-by: baude <[email protected]>
@mheon
Copy link
Member

mheon commented May 7, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 7, 2019
@openshift-merge-robot openshift-merge-robot merged commit 74dc9a4 into containers:master May 7, 2019
@baude baude deleted the makeitrain branch December 22, 2019 18:57
@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 26, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 26, 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