Skip to content

TRT-2776: Replace ubi9/nginx-120 with ubi9-minimal + nginx package#14

Merged
openshift-merge-bot[bot] merged 3 commits into
openshift:mainfrom
jhadvig:update_dockerfile
Jul 10, 2026
Merged

TRT-2776: Replace ubi9/nginx-120 with ubi9-minimal + nginx package#14
openshift-merge-bot[bot] merged 3 commits into
openshift:mainfrom
jhadvig:update_dockerfile

Conversation

@jhadvig

@jhadvig jhadvig commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace ubi9/nginx-120:latest base image with ubi9-minimal + microdnf install nginx
  • Add directory permissions for OpenShift restricted SCC
  • Send nginx error log to stderr for container log visibility

Why

ART's OCP 5.0 build rebases container images onto RHEL 10 base layers. The ubi9/nginx-120 pre-built image loses its nginx binary after this rebase, causing CreateContainerErrorProgressDeadlineExceeded → CVO Failing=True on all TechPreview clusters.

This was one of two issues that caused the revert of CVO#1398 and blocked 5.0 nightly payloads for 69+ hours (TRT-2776).

Installing nginx explicitly via microdnf ensures the package is pulled from the target RHEL repos during the build, surviving the ART rebase. This follows the pattern used by openshift/lightspeed-agentic-console.

Test plan

  • docker build . succeeds locally
  • Container starts and serves static files on port 9001
  • CI image build passes

🤖 Generated with Claude Code

ART's OCP 5.0 build rebases container images onto RHEL 10 base
layers. The ubi9/nginx-120 pre-built image loses its nginx binary
after this rebase, causing CreateContainerError and CVO Failing=True
on all TechPreview clusters.

Switch to ubi9-minimal and install nginx explicitly via microdnf so
the package is pulled from the target RHEL repos during the build,
surviving the ART rebase. This follows the pattern used by
openshift/lightspeed-agentic-console.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 9, 2026
@openshift-ci-robot

openshift-ci-robot commented Jul 9, 2026

Copy link
Copy Markdown

@jhadvig: This pull request references TRT-2776 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

  • Replace ubi9/nginx-120:latest base image with ubi9-minimal + microdnf install nginx
  • Add directory permissions for OpenShift restricted SCC
  • Send nginx error log to stderr for container log visibility

Why

ART's OCP 5.0 build rebases container images onto RHEL 10 base layers. The ubi9/nginx-120 pre-built image loses its nginx binary after this rebase, causing CreateContainerErrorProgressDeadlineExceeded → CVO Failing=True on all TechPreview clusters.

This was one of two issues that caused the revert of CVO#1398 and blocked 5.0 nightly payloads for 69+ hours (TRT-2776).

Installing nginx explicitly via microdnf ensures the package is pulled from the target RHEL repos during the build, surviving the ART rebase. This follows the pattern used by openshift/lightspeed-agentic-console.

Test plan

  • docker build . succeeds locally
  • Container starts and serves static files on port 9001
  • CI image build passes

🤖 Generated with Claude Code

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-ci openshift-ci Bot requested review from fao89 and wking July 9, 2026 14:54
@openshift-ci

openshift-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

[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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 9, 2026
@racheljpg

Copy link
Copy Markdown

/lgtm

The OKD CI config overrides the base image to base-stream9 (CentOS
Stream) which has dnf but not microdnf. Switch to dnf which works
on both ubi9-minimal and base-stream9, following the pattern from
openshift/networking-console-plugin.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@racheljpg

Copy link
Copy Markdown

I see the okd-scos-images test is failing because of /bin/sh: line 1: microdnf: command not found - anything you've seen before? Is that something that needs to be updated in this test because you are using microdnf in this PR?

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 9, 2026
The OCP CI build uses ubi9-minimal (has microdnf), while the OKD CI
overrides the base to base-stream9 (has dnf). Use a fallback chain
so the install works on both.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 9, 2026
@openshift-ci

openshift-ci Bot commented Jul 9, 2026

Copy link
Copy Markdown

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

@jhadvig

jhadvig commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

/verified by CI

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jul 9, 2026
@openshift-ci-robot

Copy link
Copy Markdown

@jhadvig: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI

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.

@racheljpg

Copy link
Copy Markdown

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 10, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit cb8968f into openshift:main Jul 10, 2026
4 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