medik8s/system-tests: use Go 1.27 builder image as build_root - #85308
openshift-merge-bot[bot] merged 1 commit into
Conversation
|
Important Review skippedWe couldn't safely recover the incremental review. No full review was started, and the last reviewed checkpoint was preserved. Retry later, or explicitly request a full review by commenting You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (5)
Included review availability: Your plan provides up to 2 included reviews per hour; 0 remain after this review. WalkthroughThe five Medik8s system-test configurations now use the ChangesMedik8s build root configuration
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The CI configurations use the intended Go 1.27 builder image, with no identified merge-blocking impact. 🚥 Pre-merge checks | ✅ 15✅ Passed checks (15 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
openshift-ci
Bot
added
the
approved
|
/pj-rehearse pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-far-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-far-nhc-interop-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-mdr-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-nhc-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-nmo-aws |
|
@jmontleon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
The system-tests suites now require Go >= 1.27.1 to compile the operators under test, but the newest official CI builder is golang-1.26, so jobs fail with "go.mod requires go >= 1.27.1 (running go 1.26.7)". Point the system-tests build_root at medik8s/builder:go1.27 (imported separately) and regenerate jobs. Revert once an official golang-1.27 builder exists in the ocp/openshift namespaces. Depends on the medik8s/builder:go1.27 imagestream being applied first. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
0786c78 to
b9b49e4
Compare
|
[REHEARSALNOTIFIER]
A total of 31 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs. A full list of affected jobs can be found here Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
/pj-rehearse pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-far-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-far-nhc-interop-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-mdr-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-nhc-aws pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-nmo-aws |
|
@jmontleon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
/pj-rehearse pull-ci-medik8s-system-tests-main-4.22-konflux-e2e-far-aws |
|
@jmontleon: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jmontleon, weshayutin 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 |
|
/pj-rehearse ack |
|
@weshayutin: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel. |
openshift-merge-bot
Bot
added
the
rehearsals-ack
cc83957
into
openshift:main
2 participants
The system-tests suites now require Go >= 1.27.1 to compile the operators under test, but the newest official CI builder is golang-1.26, so jobs fail with "go.mod requires go >= 1.27.1 (running go 1.26.7)".
Point the system-tests build_root at medik8s/builder:go1.27 (imported separately) and regenerate jobs. Revert once an official golang-1.27 builder exists in the ocp/openshift namespaces.
Depends on the medik8s/builder:go1.27 imagestream being applied first.
Summary by CodeRabbit
The Medik8s system-test CI jobs now use the
medik8s/builder:go1.27image as theirbuild_rootfor the 4.21, 4.22, and 5.0 configurations. This provides the Go 1.27.1 toolchain required by the test suites.Apply the
medik8s/builder:go1.27ImageStream before using these jobs. Revert this workaround when an official Go 1.27 builder becomes available.