Skip to content

Conversation

@hongkailiu
Copy link
Member

This is a temporary one before #1277 gets in.

@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

Walkthrough

The changes upgrade the Go runtime version from 1.24 to 1.25 and the OpenShift base images from version 4.21 to 4.22 across build and runtime configurations. The Dockerfile also adds COPY operations for CRD manifests and introduces an ENTRYPOINT instruction.

Changes

Cohort / File(s) Summary
Build and Runtime Environment Upgrade
.ci-operator.yaml, Dockerfile.rhel
Updated Go builder image from 1.24 to 1.25 and OpenShift base images from 4.21 to 4.22. Added COPY steps for zz\_generated.crd-manifests and bootstrap directory. Introduced ENTRYPOINT instruction for cluster-version-operator.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes



📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 76d5b2f and 42e945c.

📒 Files selected for processing (2)
  • .ci-operator.yaml
  • Dockerfile.rhel
🧰 Additional context used
📓 Path-based instructions (1)
**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

  • Dockerfile.rhel
🔇 Additional comments (2)
Dockerfile.rhel (1)

1-1: Verify new builder/base image tags are available and compatible.

These tag bumps are external dependencies; please confirm the images are promoted/accessible in registry.ci.openshift.org and that Go 1.25 builds cleanly for this repo.

Also applies to: 8-8

.ci-operator.yaml (1)

4-4: Aligned CI build root tag; confirm imagestream tag exists.

Looks consistent with the Dockerfile updates—please verify the rhel-9-release-golang-1.25-openshift-4.22 tag is available in the CI imagestream to prevent build failures.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 16, 2026
@hongkailiu hongkailiu changed the title Pr1277 Bump to builder:rhel-9-golang-1.25-openshift-4.22 Jan 16, 2026
@Ximinhan
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 16, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 16, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hongkailiu, Ximinhan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hongkailiu
Copy link
Member Author

/test e2e-aws-ovn-techpreview

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 16, 2026

@hongkailiu: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@hongkailiu hongkailiu changed the title Bump to builder:rhel-9-golang-1.25-openshift-4.22 NO-JIRA: Bump to builder:rhel-9-golang-1.25-openshift-4.22 Jan 16, 2026
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jan 16, 2026
@openshift-ci-robot
Copy link
Contributor

@hongkailiu: This pull request explicitly references no jira issue.

Details

In response to this:

This is a temporary one before #1277 gets in.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@hongkailiu
Copy link
Member Author

CI is all green.

/verified by @hongkailiu

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jan 16, 2026
@openshift-ci-robot
Copy link
Contributor

@hongkailiu: This PR has been marked as verified by @hongkailiu.

Details

In response to this:

CI is all green.

/verified by @hongkailiu

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-merge-bot openshift-merge-bot bot merged commit 62ee569 into openshift:main Jan 16, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants