-
Notifications
You must be signed in to change notification settings - Fork 292
Registry replacer: Update dockerfiles from ocp-build-data #1093
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
Registry replacer: Update dockerfiles from ocp-build-data #1093
Conversation
|
/assign @jupierce @stevekuznetsov |
|
/hold |
|
/lgtm |
|
/retest |
4607c07 to
cfbb475
Compare
| ignoredRepos sets.String, | ||
| ) { | ||
|
|
||
| // The tool only works for the current release |
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.
Technically we just need it to not be a branch that's being fast forwarded to right? So like release-4.2 would be fine
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.
Right. Currently the assumption is that we will only use this going forward. We should probably make this configurable but we won't need it for older releases.
a2cd2e3 to
9f1b048
Compare
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alvaroaleman, jupierce, 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 |
|
/hold cancel |
The diff this PR produces is here: openshift/release#10799
First commit is pure moving code around, second commit is the actual change.