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.

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

[release-4.4] Bug 1806576: bump rhcos for crio fix #3179

Merged

Conversation

rphillips
Copy link
Contributor

@rphillips rphillips commented Feb 24, 2020

backport #3173 to 4.4

@openshift-ci-robot openshift-ci-robot added the bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. label Feb 24, 2020
@openshift-ci-robot
Copy link
Contributor

@rphillips: This pull request references Bugzilla bug 1806576, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

[release-4.4] Bug 1806576: bump rhcos for kubelet fix

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 24, 2020
@openshift-ci-robot
Copy link
Contributor

@rphillips: This pull request references Bugzilla bug 1806576, which is valid.

In response to this:

[release-4.4] Bug 1806576: bump rhcos for kubelet fix

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@abhinavdahiya
Copy link
Contributor

/test e2e-gcp
/test e2e-metal
/test e2e-azure
/test e2e-vsphere

@rphillips
Copy link
Contributor Author

/retest

1 similar comment
@abhinavdahiya
Copy link
Contributor

/retest

@rphillips
Copy link
Contributor Author

aws throttle, we may want to pause for a little bit

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/test e2e-vsphere
/test e2e-gcp

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/test e2e-vsphere

@rphillips
Copy link
Contributor Author

rphillips commented Feb 25, 2020

Sorry all, new branches were cut in the middle of this pr chain... I need to backport the original patch into 4.4. My bad.

QE is going to need another day to test the kubelet fix... We should bump rhcos anyway for the crio fix.

@rphillips rphillips closed this Feb 25, 2020
@rphillips
Copy link
Contributor Author

openshift/origin#24596

@rphillips rphillips reopened this Feb 25, 2020
@rphillips rphillips changed the title [release-4.4] Bug 1806576: bump rhcos for kubelet fix [release-4.4] Bug 1806576: bump rhcos for crio fix Feb 25, 2020
@rphillips
Copy link
Contributor Author

/retest

@openshift-ci-robot
Copy link
Contributor

@rphillips: This pull request references Bugzilla bug 1806576, which is valid.

In response to this:

[release-4.4] Bug 1806576: bump rhcos for kubelet fix

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rphillips
Copy link
Contributor Author

QE is going to take another day to test the Kubelet fix... We need to get this into 4.4 for the crio fix.

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/test e2e-metal
/test e2e-vsphere
/test e2e-openstack

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

Nevermind, openstack looks good.
vsphere looks dire but that doesn't appear to be a regression as everything vsphere 4.4 is red https://prow.svc.ci.openshift.org/?job=*vsphere*4.4
I'd like to see another e2e-metal run before we push this through, not sure why the bot is being unresponsive.

@rphillips
Copy link
Contributor Author

/test e2e-metal

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/test e2e-vsphere

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/lgtm
/approve
/override ci/prow/e2e-aws-upgrade

1 similar comment
@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/lgtm
/approve
/override ci/prow/e2e-aws-upgrade

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

@sdodson: Overrode contexts on behalf of sdodson: ci/prow/e2e-aws-upgrade

In response to this:

/lgtm
/approve
/override ci/prow/e2e-aws-upgrade

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sdodson

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

The pull request process is described here

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 Feb 25, 2020
@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.

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/test e2e-gcp-upi

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Feb 25, 2020

@rphillips: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-aws-upgrade 9c05de4 link /test e2e-aws-upgrade
ci/prow/e2e-gcp-upi 9c05de4 link /test e2e-gcp-upi
ci/prow/e2e-vsphere 9c05de4 link /test e2e-vsphere

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

/override ci/prow/e2e-gcp-upi
normal e2e-gcp has passed

@openshift-ci-robot
Copy link
Contributor

@sdodson: Overrode contexts on behalf of sdodson: ci/prow/e2e-gcp-upi

In response to this:

/override ci/prow/e2e-gcp-upi
normal e2e-gcp has passed

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@sdodson
Copy link
Member

sdodson commented Feb 25, 2020

vsphere failed on e2e tests, which shouldn't be a direct result of this change due to the pivot

@openshift-merge-robot openshift-merge-robot merged commit a686af7 into openshift:release-4.4 Feb 25, 2020
@openshift-ci-robot
Copy link
Contributor

@rphillips: All pull requests linked via external trackers have merged. Bugzilla bug 1806576 has been moved to the MODIFIED state.

In response to this:

[release-4.4] Bug 1806576: bump rhcos for crio fix

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants