chore: bump platform-api and hyperfleet-operator images to c882180 - #749
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe regional cluster configuration updates the HyperFleet Argo CD source revision and operator image tag to ChangesRegional cluster deployment revisions
Estimated code review effort: 1 (Trivial) | ~2 minutes Mergeability Score: ⚪ Minimal · up to This PR updates two image references to align deployed components with the corresponding API changes; no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
/retest-required |
|
/test on-demand-e2e |
|
The Failing tests:
Passing suites:
Both failures are environment/infrastructure timeouts unrelated to image tag changes. /retest-required AI-generated. Review for accuracy. |
|
Second run, same two environment flakes — both unrelated to this PR:
Passing suites (unchanged): e2e-api 12/12 ✅, e2e-platform-monitoring 13/13 ✅ These two tests are flaking consistently in the on-demand environment. Retesting. /retest-required AI-generated. Review for accuracy. |
|
Third consecutive
These two tests have failed identically across all 3 runs. The ZOA execution environment appears unable to transition executions out of Not retesting again — the pattern is established. This PR may need AI-generated. Review for accuracy. |
platform-api: f7ec7d5 → c882180 (picks up PR openshift-online#331 typed error codes) hyperfleet-operator: f7ec7d5 → f58b3a3 (latest available Konflux-built image) The two components use separate Konflux pipelines with different path triggers, so their latest built images may reference different commits.
92920e8 to
e0a15cc
Compare
typeid
left a comment
There was a problem hiding this comment.
/lgtm
/approve
/test on-demand-e2e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: typeid 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 |
|
Updated: the hyperfleet-operator image tag has been corrected. Root cause of CI failures: the hyperfleet-operator image didn't exist at Fix: hyperfleet-operator Verification:
This commit was amended and force-pushed — will need re-approval. AI-generated. Review for accuracy. |
|
/test on-demand-e2e |
d940eed
into
openshift-online:main
Bump platform-api and hyperfleet-operator image references to the latest rosa-hyperfleet-api main commit (
c882180ff9a18fd79f8974cd5a026df0a6cb6b5a).This picks up changes from rosa-hyperfleet-api including:
RATE-LIMIT-001, etc.)Why
The Aug 13 nightly CI runs (ephemeral + integration) both fail on
should return correct 429 response body and Retry-After headerbecause the e2e test binary is built from latestmain(which includes PR #331's updated assertions expectingRATE-LIMIT-001), but the deployed platform-api image still predates PR #331 and returns the old format.Bumping the image references resolves the mismatch.
Changes
argocd/config/regional-cluster/platform-api/values.yaml— image tagargocd/config/regional-cluster/hyperfleet/values.yaml— image tag +targetRevisionAll updated from
f7ec7d5...→c882180....AI-generated. Review for accuracy.
@typeid requested in Slack thread
Summary by CodeRabbit