Skip to content

Conversation

@bgilbert
Copy link
Contributor

No description provided.

@openshift-ci-robot openshift-ci-robot added bugzilla/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Aug 26, 2020
@openshift-ci-robot
Copy link
Contributor

@bgilbert: This pull request references Bugzilla bug 1871795, 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.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (4.6.0) matches configured target release for branch (4.6.0)
  • bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
Details

In response to this:

Bug 1871795: bump RHCOS images to fix SSH authentication

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.

@bgilbert
Copy link
Contributor Author

/assign @sdodson

@sdodson
Copy link
Member

sdodson commented Aug 26, 2020

/test e2e-gcp
/test e2e-azure
/test e2e-metal
/test e2e-openstack
/test e2e-ovirt

@bgilbert
Copy link
Contributor Author

/retest

1 similar comment
@bgilbert
Copy link
Contributor Author

/retest

@cgwalters
Copy link
Member

/approve

@bgilbert
Copy link
Contributor Author

/retest

2 similar comments
@bgilbert
Copy link
Contributor Author

/retest

@bgilbert
Copy link
Contributor Author

/retest

@cgwalters
Copy link
Member

In prior bootimage bumps we've had a rough rule of requiring aws/gcp/azure, best effort debugging vsphere, metal and others. You can usually take other ones as e2e-aws-workers-rhel7 as totally optional.

A good rule is to look at the latest periodics, e.g. the last currently accepted release image for 4.6 had e2e-metal passing, so we know that is working.

@cgwalters
Copy link
Member

The current e2e-aws failure looks like some sort of test infrastructure failure.

@bgilbert
Copy link
Contributor Author

/retest

@cgwalters
Copy link
Member

Hm, not clear to me what's going on here - e2e-aws for example seems good on other repositories. I don't see anything in recently merged openshift/release PRs that look relevant.

@praveenkumar
Copy link
Contributor

/retest

@cgwalters
Copy link
Member

/retest
The failures here were a CI infrastructure or release issue that should be fixed.

@cgwalters
Copy link
Member

/approve
/lgtm

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

/approve

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 27, 2020
@sdodson
Copy link
Member

sdodson commented Aug 27, 2020

/approve
even though i'm perpetually slower than abhinav

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abhinavdahiya, cgwalters, sdodson

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:
  • OWNERS [abhinavdahiya,sdodson]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-bot
Copy link
Contributor

/retest

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

@cgwalters
Copy link
Member

Blah, azure is hitting rate limits.

@cgwalters
Copy link
Member

It'd be good to get this to pass, even though it's very unlikely we somehow broke just Azure in this.
/test e2e-azure

@openshift-ci-robot
Copy link
Contributor

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

Test name Commit Details Rerun command
ci/prow/e2e-crc 9bd4545 link /test e2e-crc
ci/prow/e2e-azure 9bd4545 link /test e2e-azure

Full PR test history. Your PR dashboard.

Details

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.

@cgwalters
Copy link
Member

cgwalters commented Aug 27, 2020

Ugh looks like something fell over in...one of the centos7 repos?

No match for argument: pyOpenSSL
No match for argument: PyYAML 

Oh wait I think I see...this is fallout from the switch to ubi8 probably.

@cgwalters
Copy link
Member

Took a stab at fixing this in #4106

@openshift-merge-robot openshift-merge-robot merged commit ee89123 into openshift:master Aug 27, 2020
@openshift-ci-robot
Copy link
Contributor

@bgilbert: All pull requests linked via external trackers have merged:

Bugzilla bug 1871795 has been moved to the MODIFIED state.

Details

In response to this:

Bug 1871795: bump RHCOS images to fix SSH authentication

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.

@bgilbert bgilbert deleted the bootimage branch August 29, 2020 05:15
Prashanth684 added a commit to Prashanth684/installer that referenced this pull request Sep 1, 2020
vrutkovs pushed a commit to vrutkovs/installer that referenced this pull request Sep 10, 2020
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/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants