fix: unpin hypershift image and bump hypershift-stack (AROSLSRE-921) - #5401
Conversation
Unpin hypershift operator now that the domain shadowing revert has landed upstream (openshift/hypershift#8585). The new image (89e19f83) includes the fix. Also bumps ACM operator v2.16.2-440 and MCE v2.11.2-480.
There was a problem hiding this comment.
Pull request overview
This PR updates the HyperShift/ACM “hypershift-stack” image set to restore automated tracking of the HyperShift operator image (by unpinning the image-updater tag) and bumps the corresponding digests across defaults and rendered dev configs.
Changes:
- Unpin HyperShift operator in
tooling/image-updaterby switching from a pinned commit SHA tag back totag: "latest". - Bump HyperShift operator digest and ACM operator/MCE bundle digests in
config/config.yamland regenerated rendered dev environment configs. - Refresh multicluster-engine Helm chart metadata and operator deployment operand image digests to match the new MCE bundle.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tooling/image-updater/config.yaml | Restores HyperShift operator tracking to latest for future automated digest bumps. |
| config/config.yaml | Updates default digests for HyperShift operator, ACM operator bundle, and MCE bundle. |
| config/rendered/dev/prow/centralus.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| config/rendered/dev/pers/westus3.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| config/rendered/dev/perf/westus3.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| config/rendered/dev/dev/westus3.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| config/rendered/dev/cspr/westus3.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| config/rendered/dev/ci01/centralus.yaml | Regenerated rendered dev config with updated ACM/MCE/HyperShift digests. |
| acm/deploy/helm/multicluster-engine/templates/multicluster-engine-operator.deployment.yaml | Updates operand image digests used by the MCE operator deployment. |
| acm/deploy/helm/multicluster-engine/Chart.yaml | Updates chart sources digest reference for the MCE bundle. |
| acm/deploy/helm/multicluster-engine-crds/Chart.yaml | Updates chart sources digest reference for the MCE bundle (CRDs chart). |
Manyanda Chitimbo (machi1990)
left a comment
There was a problem hiding this comment.
/lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: machi1990, raelga 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 |
AROSLSRE-921
What
Unpin the HyperShift operator image and bump the full
hypershift-stackgroup:cf2b91fb(May 18, pinned) →89e19f83(May 27, latest) — includes the domain shadowing revertv2.16.2-438→v2.16.2-440v2.11.2-462→v2.11.2-480Why
The HyperShift image was [pinned in AROSLSRE-919](https://redhat.atlassian.net/browse/AROSLSRE-919) because commit
be263214added a CEL validation that rejected Azure HostedClusters when service hostnames shadow the cluster base domain, breaking all e2e cluster creation.The upstream fix (openshift/hypershift#8585) reverted that validation and was merged on May 26. The new
latestimage (89e19f83) is 4 commits ahead of the revert — confirmed to include the fix.Testing
make -C config materializeregenerates cleanlySpecial notes for your reviewer
The
tooling/image-updater/config.yamlchange restores the hypershift tag from a pinned commit SHA back to"latest", which re-enables automated bumps for future PRs.PR Checklist