fix: unpin hypershift operator image after registry-override regression fix (AROSLSRE-1325) - #5785
Conversation
…35c2a83c923019b169c33d23b070e Signed-off-by: Ashish <asnaraya@redhat.com>
There was a problem hiding this comment.
Pull request overview
Updates the pinned Hypershift operator image digest used by the repo’s configuration defaults and the derived (materialized) dev environment configs.
Changes:
- Bump
defaults.hypershift.image.digestinconfig/config.yamltosha256:9775a90c…d23b070e. - Propagate the same digest bump into the rendered dev configs for westus3 and centralus variants.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| config/config.yaml | Updates the canonical/default Hypershift operator image digest (with updated provenance comment). |
| config/rendered/dev/pers/westus3.yaml | Updates rendered dev pers westus3 Hypershift digest to match defaults. |
| config/rendered/dev/perf/westus3.yaml | Updates rendered dev perf westus3 Hypershift digest to match defaults. |
| config/rendered/dev/dev/westus3.yaml | Updates rendered dev dev westus3 Hypershift digest to match defaults. |
| config/rendered/dev/cspr/westus3.yaml | Updates rendered dev cspr westus3 Hypershift digest to match defaults. |
| config/rendered/dev/ci01/centralus.yaml | Updates rendered dev ci01 centralus Hypershift digest to match defaults. |
| config/rendered/dev/ci00/centralus.yaml | Updates rendered dev ci00 centralus Hypershift digest to match defaults. |
| registry: quay.io | ||
| repository: redhat-services-prod/crt-redhat-acm-tenant/hypershift/hypershift-operator | ||
| digest: sha256:e30b97d22230e8a06d0079c3862d0e5510868a4cd755df9e4396ddbd095956e5 # 80198108f879b26a7e990bc621b490c50fe1c196 (2026-06-19 15:51) | ||
| digest: sha256:9775a90c9bf7affe17341f04769b4817b8835c2a83c923019b169c33d23b070e # 9acec4759a16c86f0f87792c18a1d738b3b85129 (2026-06-25 05:28) |
Manyanda Chitimbo (machi1990)
left a comment
There was a problem hiding this comment.
Don't we need to revert the pinning as well #5776 ?
Signed-off-by: Ashish <asnaraya@redhat.com>
Rael Garcia (raelga)
left a comment
There was a problem hiding this comment.
/lgtm
/approve
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ashishmax31, 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 |
What
Updates the HyperShift operator image to the build that includes the upstream
registry-override fix (openshift/hypershift#8824) and
removes the temporary pin.
Why
Reverts the temporary HyperShift operator image pin from AROSLSRE-1318 now
that the upstream registry-override regression (OCPBUGS-92034) is fixed and a
corrected image is published, so the automated bumper can resume.
JIRA: AROSLSRE-1325 (revert)
· parent AROSLSRE-1318
Testing
Testing is required for feature completion and tests should be part of the pull
request along with the feature changes.
Describe the testing provided. If you did not add tests, provide a clear
justification.
Special notes for your reviewer
PR Checklist
If E2E tests are included:
demonstrate that the test is able to detect a defect/error and fail with
proper error message and logs which communicates nature of the problem.