Skip to content

[NO-JIRA] Sync Dockerfile.ocp image#1127

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
jhadvig:builder_release-4.22
Mar 17, 2026
Merged

[NO-JIRA] Sync Dockerfile.ocp image#1127
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
jhadvig:builder_release-4.22

Conversation

@jhadvig
Copy link
Copy Markdown
Member

@jhadvig jhadvig commented Mar 17, 2026

Summary by CodeRabbit

  • Chores
    • Updated Docker builder and runtime base images to newer versions for improved compatibility and security.
    • Enhanced build configuration with additional manifest and CRD file inclusion for better deployment packaging.

@jhadvig jhadvig changed the title Sync Dockerfile.ocp image [NO-JIRA] Sync Dockerfile.ocp image Mar 17, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 0095ee38-4fb9-4075-ace7-602cb5270c51

📥 Commits

Reviewing files that changed from the base of the PR and between 3d173bc and ccac969.

📒 Files selected for processing (1)
  • Dockerfile.ocp
📜 Recent review details
🧰 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.ocp
🔇 Additional comments (2)
Dockerfile.ocp (2)

1-1: LGTM!

Builder image correctly updated to Go 1.25 and OpenShift 4.22.


8-8: LGTM!

Runtime base image correctly updated to 4.22, consistent with the builder stage.


Walkthrough

Updated the Dockerfile.ocp base images from OpenShift 4.20 to 4.22 with Go 1.25, and added COPY directives for manifests, quickstarts, and CRD resources to the image build stages.

Changes

Cohort / File(s) Summary
Dockerfile Updates
Dockerfile.ocp
Updated builder image from rhel-9-golang-1.24-openshift-4.20 to rhel-9-golang-1.25-openshift-4.22 and runtime image from 4.20:base-rhel9 to 4.22:base-rhel9. Added COPY directives for manifests/, quickstarts/*.yaml, and CRD manifest files from vendor/.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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

@jhadvig jhadvig added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. verified Signifies that the PR passed pre-merge verification criteria labels Mar 17, 2026
@openshift-ci openshift-ci Bot requested review from TheRealJon and spadgett March 17, 2026 14:08
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 17, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jhadvig

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

@jhadvig
Copy link
Copy Markdown
Member Author

jhadvig commented Mar 17, 2026

Overriding since the actual change is done in openshift/release#76399
/override ci/prow/e2e-aws-console ci/prow/e2e-aws-operator ci/prow/e2e-azure-ovn-upgrade ci/prow/e2e-gcp-ovn

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 17, 2026

@jhadvig: Overrode contexts on behalf of jhadvig: ci/prow/e2e-aws-console, ci/prow/e2e-aws-operator, ci/prow/e2e-azure-ovn-upgrade, ci/prow/e2e-gcp-ovn

Details

In response to this:

Overriding since the actual change is done in openshift/release#76399
/override ci/prow/e2e-aws-console ci/prow/e2e-aws-operator ci/prow/e2e-azure-ovn-upgrade ci/prow/e2e-gcp-ovn

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.

@jhadvig jhadvig added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Mar 17, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit f5ba579 into openshift:main Mar 17, 2026
11 checks passed
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Mar 17, 2026

@jhadvig: 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.

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-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. 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.

1 participant