[WIP] AGENT-1326: Add InternalReleaseImage templates generation during the bootstrap (with RenderContext)#5419
Conversation
|
@andfasano: This pull request references AGENT-1326 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.21.0" version, but no target version was set. DetailsIn response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: andfasano The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@andfasano: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
|
This would be easier to review if the refactoring to use RenderContext were in a separate commit from the changes to add IRI registry templates. |
|
PR needs rebase. DetailsInstructions 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-sigs/prow repository. |
|
Addressed by github.com//pull/5452 |
What I did
Enhances the machine-config-controller bootstrap command to add conditionally the InternalReleaseImage related templates (based on the presence on the InternalReleaseImage resource and if the related feature gate has been enabled)
How to verify it
Add the InternalReleaseImage resource in the bootstrap manifests dir (/etc/mcc/bootstrap) and enable the feature gate.
Description for the changelog
Add support for InternalReleaseImage resource