Skip to content

Update RELATED_IMAGE_APACHE_IMAGE_URL_DEFAULT to ubi10/httpd-24:latest - #2086

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

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

Conversation

@stuggi

@stuggi stuggi commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
@openshift-ci-robot

Copy link
Copy Markdown

@stuggi: No Jira issue with key httpd-24 exists in the tracker at https://redhat.atlassian.net.
Once a valid jira issue is referenced in the title of this pull request, request a refresh with /jira refresh.

Details

In response to this:

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 fultonj and rebtoor September 18, 2026 15:05
@github-actions

Copy link
Copy Markdown

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 338314 bytes (330KB)
Base branch size 338314 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

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: 86d07873-b075-4359-8663-25ad1e96042e

📥 Commits

Reviewing files that changed from the base of the PR and between f85f1b4 and f2d1145.

📒 Files selected for processing (4)
  • api/dataplane/v1beta1/openstackdataplanenodeset_types.go
  • config/operator/default_images.yaml
  • docs/assemblies/con_provisioning-bare-metal-data-plane-nodes.adoc
  • hack/export_related_images.sh

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


📝 Summary

Summary by CodeRabbit

  • Updates
    • Updated the default Apache container image from UBI 9 to UBI 10.
    • Updated deployment configuration, provisioning examples, and related image settings to reference the UBI 10 Apache image.

Walkthrough

The default Apache container image changed from registry.redhat.io/ubi9/httpd-24:latest to registry.redhat.io/ubi10/httpd-24:latest in code, configuration, export tooling, and documentation.

Changes

Apache image update

Layer / File(s) Summary
Align Apache image defaults and references
api/dataplane/v1beta1/openstackdataplanenodeset_types.go, config/operator/default_images.yaml, hack/export_related_images.sh, docs/assemblies/con_provisioning-bare-metal-data-plane-nodes.adoc
The Apache image reference changed from UBI9 to UBI10 in the dataplane default, operator configuration, related-image export script, and provisioning example.

Priority: ⬇️ Low

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

Change: Other

Suggested reviewers: rabi

Merge Risk: ⚪ Minimal · up to f2d11

The UBI10 image update is synchronized across code, configuration, tooling, and documentation, with no demonstrated merge-blocking defect.

🚥 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 description does not convey information about the image update. Add a brief description stating that the default Apache image changes from UBI 9 to UBI 10.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the default Apache image 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 2…
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.

@stuggi
stuggi requested review from yazug and removed request for fultonj and rebtoor September 18, 2026 15:07

@yazug yazug 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 added the lgtm label Sep 18, 2026
@stuggi
stuggi requested review from abays and dprince September 18, 2026 15:36
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/8ca80fca95434a84adca807dab319ccb

✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 26m 48s
❌ podified-multinode-edpm-deployment-crc NODE_FAILURE Node(set) request 099-0000207283 failed in 0s
❌ cifmw-crc-podified-edpm-baremetal NODE_FAILURE Node(set) request 099-0000207284 failed in 0s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 35m 58s
✔️ openstack-operator-docs-preview SUCCESS in 2m 55s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 01m 02s

@stuggi

stuggi commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/d8f73059b6574009be14ea7186367bb7

✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 14m 42s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 18s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 36m 35s
❌ openstack-operator-tempest-multinode FAILURE in 1h 27m 11s
✔️ openstack-operator-docs-preview SUCCESS in 3m 03s
✔️ openstack-operator-edpm-baremetal-minor-update SUCCESS in 2h 01m 58s

@stuggi

stuggi commented Sep 21, 2026

Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

@abays abays 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 21, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, stuggi, yazug

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

@abays

abays commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

@stuggi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/openstack-operator-build-deploy-kuttl-4-20 f2d1145 link unknown /test openstack-operator-build-deploy-kuttl-4-20

Full PR test history. Your PR dashboard.
Details

    logger.go:42: 15:22:09 |  | from server for: "STDIN": Get "https://api.oko-09.oooci.ccitredhat.com:6443/api/v1/namespaces/openstack-kuttl-tests/secrets/dataplane-ansible-ssh-private-key-secret": dial tcp: lookup api.oko-09.oooci.ccitredhat.com on 172.30.0.10:53: read udp 10.128.205.30:52676->172.30.0.10:53: i/o timeout

/test openstack-operator-build-deploy-kuttl-4-20

@stuggi

stuggi commented Sep 22, 2026

Copy link
Copy Markdown
Contributor Author

/retest

@abays

abays commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@stuggi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/functional f2d1145 link unknown /test functional

Full PR test history. Your PR dashboard.
Details

[FAIL] OpenStackVersion controller when Multiple target versions exist [It] updating targetVersion triggers a minor update workflow [Serial]

Let's see if it reproduces.

/test functional

@stuggi

stuggi commented Sep 22, 2026

Copy link
Copy Markdown
Contributor Author

@stuggi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/openstack-operator-build-deploy-kuttl-4-22 f2d1145 link unknown /test openstack-operator-build-deploy-kuttl-4-22
Full PR test history. Your PR dashboard.

Details

  • skopeo copy docker://quay.io/openstack-k8s-operators/ironic-operator-bundle:9fef80436b32993d0e6d2b4992cadfa477b59bcf dir:tmp/bindata/tmp
    time="2026-09-22T08:04:48Z" level=fatal msg="initializing source docker://quay.io/openstack-k8s-operators/ironic-operator-bundle:9fef80436b32993d0e6d2b4992cadfa477b59bcf: pinging container registry quay.io: Get "https://quay.io/v2/\": dial tcp [2600:1f18:d77:9002:d6d8:1eed:e32a:e898]:443: connect: network is unreachable"

/test openstack-operator-build-deploy-kuttl-4-22

@abays

abays commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

@stuggi: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/openstack-operator-build-deploy-kuttl-4-22 f2d1145 link true /test openstack-operator-build-deploy-kuttl-4-22
ci/prow/precommit-check f2d1145 link unknown /test precommit-check

Full PR test history. Your PR dashboard.
Details

curl: (6) Could not resolve host: raw.githubusercontent.com

/test precommit-check

@stuggi

stuggi commented Sep 22, 2026

Copy link
Copy Markdown
Contributor Author

/test openstack-operator-build-deploy-kuttl-4-22

@openshift-merge-bot
openshift-merge-bot Bot merged commit 3654536 into openstack-k8s-operators:main Sep 22, 2026
11 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.

4 participants