-
Notifications
You must be signed in to change notification settings - Fork 2k
Update template owners, sync owners files #1279
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
Update template owners, sync owners files #1279
Conversation
Signed-off-by: Steve Kuznetsov <[email protected]>
Signed-off-by: Steve Kuznetsov <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stevekuznetsov The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| @@ -0,0 +1,31 @@ | |||
| filters: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@smarterclayton FYI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may want to roll this back, because the approve plugin currently doesn't work with filters (kubernetes/test-infra#7690).
This reverts commit a3e9a3f (openshift#1279). Using filters for approvers and reviewers is the long-term goal, but the approve plugin can't handle this yet [1,2]. [1]: kubernetes/test-infra#7690 [2]: https://github.com/kubernetes/community/blame/19dc4bed5b05c23a69e631f96573f8ed2e0774cb/contributors/guide/owners.md#L86-L87
Add default definition for RHCOS_LIVE_IMAGES in common.sh
* Revert "Add support for Live CoreOS URLS (openshift#1279)" We no longer need to mirror these images, because they're supplied by machine-os-images in the release payload. This reverts commit 53e4b2e. * Don't pass clusterOSImage in install-config for 4.10+ We no longer need to pass a QCOW to the installer for the cluster OS (we still need it for the bootstrap), so don't do it any more starting with 4.10.
No description provided.