Skip to content

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Feb 2, 2018

…_runtime_test

picks kubernetes/kubernetes#58994
fixes #17757

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 2, 2018
@openshift-merge-robot openshift-merge-robot added the vendor-update Touching vendor dir or related files label Feb 2, 2018
@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 2, 2018
@mfojtik
Copy link
Contributor

mfojtik commented Feb 2, 2018

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 2, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, mfojtik

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

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@deads2k
Copy link
Contributor Author

deads2k commented Feb 2, 2018

/retest

@mfojtik
Copy link
Contributor

mfojtik commented Feb 2, 2018

@sjenning or @RobertKrawitz can you safety check this?

@sjenning
Copy link
Contributor

sjenning commented Feb 2, 2018

lgtm

@RobertKrawitz
Copy link
Contributor

createRemoteImageService() also needs the assert changed to require. Thanks.

@openshift-merge-robot
Copy link
Contributor

/test all [submit-queue is verifying that this PR is safe to merge]

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 18393, 18409).

@openshift-merge-robot openshift-merge-robot merged commit 4f13268 into openshift:master Feb 2, 2018
@deads2k deads2k deleted the flake-01-testversion branch July 3, 2018 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. vendor-update Touching vendor dir or related files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unit test k8s.io/kubernetes/pkg/kubelet/rkt TestVersion
6 participants