chore(config): bump ACM/MCE operator bundles (bisect e2e #5789)#5911
Conversation
|
Skipping CI for Draft Pull Request. |
|
/hold |
There was a problem hiding this comment.
Pull request overview
This PR bumps the ACM operator bundle and MCE operator bundle digests (and the derived ACM/MCE Helm chart content + fixtures) to isolate whether that specific bump is responsible for the e2e-parallel CreateHCPCluster timeout regression being bisected from PR #5789.
Changes:
- Update default ACM operator bundle digest and MCE operator bundle digest in
config/config.yaml. - Propagate the updated digests into rendered environment configs under
config/rendered/dev/.... - Update ACM/MCE Helm charts and the Helm template golden fixture to reflect the bumped bundle/app versions and operand image digests.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| config/config.yaml | Bumps the default ACM and MCE bundle digests (and version comments) used for materialization. |
| config/rendered/dev/pers/westus3.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| config/rendered/dev/perf/westus3.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| config/rendered/dev/dev/westus3.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| config/rendered/dev/cspr/westus3.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| config/rendered/dev/ci00/centralus.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| config/rendered/dev/ci01/centralus.yaml | Updates rendered ACM/MCE bundle digests for this environment. |
| acm/deploy/helm/multicluster-engine/Chart.yaml | Updates MCE chart appVersion/version and sources digest to match the bumped MCE bundle. |
| acm/deploy/helm/multicluster-engine-crds/Chart.yaml | Updates MCE CRDs chart appVersion/version and sources digest to match the bumped MCE bundle. |
| acm/deploy/helm/multicluster-engine/templates/multicluster-engine-operator.deployment.yaml | Updates operator/operand image digests and operator version env var consistent with the new MCE bundle. |
| acm/deploy/helm/multicluster-engine-config/charts/policy/Chart.yaml | Updates policy chart version pins to the newer ACM version. |
| acm/deploy/helm/multicluster-engine-config/charts/policy/charts/grc/Chart.yaml | Updates grc chart app/version to the newer ACM version. |
| acm/deploy/helm/multicluster-engine-config/charts/policy/charts/cluster-lifecycle/Chart.yaml | Updates cluster-lifecycle chart app/version to the newer ACM version. |
| acm/deploy/helm/multicluster-engine-config/charts/policy/values.yaml | Updates ACM governance-related image override digests used by the policy chart. |
| acm/zz_fixture_TestHelmTemplate_dev_westus3_mgmt_1_mce.yaml | Updates the Helm template golden fixture to reflect the updated MCE operator version. |
|
/unhold |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: inbharajmani, 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 |
Bisecting the e2e-parallel regression on the automated image-digest PR #5789, which flipped from green to red at commit
128feb1.What
Bumps only the ACM operator bundle and MCE operator bundle digests (plus the regenerated
acm/helm charts and fixtures) to the latest values from #5789, on top of currentmain. Hypershift and all other components stay at theirmaindigests.Why
At the pass→fail boundary of #5789, ACM/MCE bundles were among the 6 changed digests. The failure is
CreateHCPClustertiming out across ~16 tests. This PR isolates the ACM/MCE bump so e2e-parallel tells us whether it alone breaks provisioning. Theacm/chart files were reused from #5789 head (no skopeo pull) and confirmed byte-identical, thenmake -C config/ materializeregenerated rendered config + fixtures.Testing
Relies on
ci/prow/e2e-parallel.make -C config/ detect-changeis clean. Diagnostic bisect PR — not intended to merge as-is.Special notes for your reviewer
Do not merge. Diagnostic bisect of #5789. Companion PRs isolate hypershift and the remaining components.