[WIP] use the default OS image from the OSImageStream#10406
[WIP] use the default OS image from the OSImageStream#10406cheesesashimi wants to merge 1 commit intoopenshift:mainfrom
Conversation
|
Skipping CI for Draft Pull Request. |
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Review skipped — only excluded labels are configured. (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
58db82b to
bc03ec7
Compare
This uses the new `machine-config-osimagestream` helper binary to inspect the release payload and determine what the default OS image for a given payload should be. Doing this allows a release payload to ship with multiple OS images while avoiding the need to hardcode those image names. Assisted-By: Claude Sonnet 4.5
bc03ec7 to
9db4532
Compare
|
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. |
This uses the new
machine-config-osimagestreamhelper binary to inspect the release payload and determine what the default OS image from a given payload should be. Doing this allows a release payload to ship with multiple OS images while avoiding the need to hardcode those image names.Right now, this is still a work in progress and should not be reviewed / merged until after openshift/machine-config-operator#5770 has merged.