Skip to content

UPSTREAM: 54597: kubelet: check for illegal container state transition#17051

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
frobware:backpick-kubernetes-54597
Oct 28, 2017
Merged

UPSTREAM: 54597: kubelet: check for illegal container state transition#17051
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
frobware:backpick-kubernetes-54597

Conversation

@frobware
Copy link
Contributor

@frobware frobware commented Oct 26, 2017

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 26, 2017
@openshift-merge-robot openshift-merge-robot added the vendor-update Touching vendor dir or related files label Oct 26, 2017
@frobware
Copy link
Contributor Author

frobware commented Oct 26, 2017

/assign @sjenning

@derekwaynecarr
Copy link
Member

derekwaynecarr commented Oct 26, 2017

/approve
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 26, 2017
@derekwaynecarr
Copy link
Member

derekwaynecarr commented Oct 26, 2017

/kind bug

@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Oct 26, 2017
@openshift-merge-robot
Copy link
Contributor

openshift-merge-robot commented Oct 26, 2017

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: derekwaynecarr, frobware

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

Details 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

@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 26, 2017
@sjenning
Copy link
Contributor

sjenning commented Oct 26, 2017

@frobware can you pick this to stage as well?

@sjenning sjenning mentioned this pull request Oct 26, 2017
Merged
@sjenning
Copy link
Contributor

sjenning commented Oct 26, 2017

/retest

@openshift-bot
Copy link
Contributor

openshift-bot commented Oct 27, 2017

/retest

Please review the full test history for this PR and help us cut down flakes.

@frobware
Copy link
Contributor Author

frobware commented Oct 27, 2017

/test extended_networking_minimal
/test extended_conformance_gce

openshift-merge-robot added a commit that referenced this pull request Oct 27, 2017
@openshift-merge-robot
Merge pull request #17059 from sjenning/pick-54597-stage
6e3c76a 
Automatic merge from submit-queue.

[stage] UPSTREAM: 54597: kubelet: check for illegal container state transition

#17051

@frobware @joelsmith @derekwaynecarr
@sjenning
Copy link
Contributor

sjenning commented Oct 27, 2017

CI is busted because rc0 was tagged but the images aren't in docker hub yet. Image build and push is in progress.

@sjenning
Copy link
Contributor

sjenning commented Oct 27, 2017

Images should be pushed now

/retest

@sjenning
Copy link
Contributor

sjenning commented Oct 28, 2017

/retest

@openshift-bot
Copy link
Contributor

openshift-bot commented Oct 28, 2017

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

openshift-bot commented Oct 28, 2017

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot
Copy link
Contributor

openshift-merge-robot commented Oct 28, 2017

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 7e45849 into openshift:master Oct 28, 2017
@frobware frobware deleted the backpick-kubernetes-54597 branch October 30, 2017 07:44
frobware pushed a commit to frobware/origin that referenced this pull request Dec 11, 2017
@openshift-merge-robot @frobware
Merge pull request openshift#17051 from frobware/backpick-kubernetes-…
25292e6 
…54597

Automatic merge from submit-queue.

UPSTREAM: 54597: kubelet: check for illegal container state transition
@frobware frobware mentioned this pull request Dec 11, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@sjenning sjenning

@soltysh soltysh

@bparees bparees

@derekwaynecarr derekwaynecarr

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. vendor-update Touching vendor dir or related files

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@frobware @derekwaynecarr @openshift-merge-robot @sjenning @openshift-bot @soltysh @bparees @openshift-ci-robot