Skip to content

Switch Dockerfile base images from UBI 9 to UBI 10 - #468

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
dprince:ubi10
Sep 22, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
dprince:ubi10

Conversation

@dprince

@dprince dprince commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@openshift-ci
openshift-ci Bot requested review from hjensas and rabi September 21, 2026 19:37
@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

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

Review profile: CHILL

Plan: Enterprise

Run ID: 4fa53a0f-c07c-4fda-8e37-f7c483ebc853

📥 Commits

Reviewing files that changed from the base of the PR and between c6b4240 and d4428fb.

📒 Files selected for processing (1)
  • Dockerfile

Included review availability: Your plan provides up to 12 included reviews per hour; 0 remain after this review.


📝 Summary

Summary by CodeRabbit

  • Chores
    • Updated the application’s build and runtime environments to use UBI 10 base images.
    • Changed the runtime image from a pinned version to the latest available release.

Walkthrough

The Dockerfile changes the Go builder image from UBI 9 to UBI 10 and changes the runtime image from pinned UBI 9.6 to UBI 10 latest.

Changes

Container image updates

Layer / File(s) Summary
Update container base images
Dockerfile
The builder image changes to UBI 10 with Go 1.26. The runtime image changes to UBI 10 with the latest tag.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: stuggi

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so the change cannot be evaluated from the description. Add a brief description of the Dockerfile base image updates, including the UBI 10 build and runtime images.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the Dockerfile change from UBI 9 to UBI 10.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

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

@centosinfra-prod-github-app

Copy link
Copy Markdown

@stuggi stuggi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci

openshift-ci Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprince, stuggi

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

@openshift-merge-bot
openshift-merge-bot Bot merged commit cc4e4ff into openstack-k8s-operators:main Sep 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants