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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 1 commit into from
Oct 28, 2017
Merged

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

merged 1 commit into from
Oct 28, 2017

Conversation

frobware
Copy link
Contributor

@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

/assign @sjenning

@derekwaynecarr
Copy link
Member

/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

/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

[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.

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

@frobware can you pick this to stage as well?

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

/retest

@openshift-bot
Copy link
Contributor

/retest

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

@frobware
Copy link
Contributor Author

/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

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

Images should be pushed now

/retest

@sjenning
Copy link
Contributor

/retest

@openshift-bot
Copy link
Contributor

/retest

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

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

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

@openshift-merge-robot
Copy link
Contributor

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