Skip to content

coreos/coreos-assembler: bump to f44#82346

Open
c4rt0 wants to merge 1 commit into
openshift:mainfrom
c4rt0:apiaseck/bumpToF44
Open

coreos/coreos-assembler: bump to f44#82346
c4rt0 wants to merge 1 commit into
openshift:mainfrom
c4rt0:apiaseck/bumpToF44

Conversation

@c4rt0

@c4rt0 c4rt0 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

bump the fedora container in the buildroot image to Fedora 44 for both main and the rhel-9.6 configuration.

Summary by CodeRabbit

Updates coreos/coreos-assembler buildroot images in both the main and rhel-9.6 OpenShift CI configurations to use Fedora 44 instead of Fedora 43.

bump the fedora container in the buildroot image to Fedora 44 for both
main and the rhel-9.6 configuration.
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 98707540-9683-4415-baae-31fe8baab3b7

📥 Commits

Reviewing files that changed from the base of the PR and between f7232c0 and c90134a.

📒 Files selected for processing (2)
  • ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-main.yaml
  • ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-rhel-9.6.yaml

Walkthrough

Updates the CoreOS assembler project image Dockerfile definitions in the main and RHEL 9.6 configurations from Fedora 43 to Fedora 44.

Changes

CoreOS assembler image configuration

Layer / File(s) Summary
Update project image base images
ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-main.yaml, ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-rhel-9.6.yaml
Embedded Dockerfile FROM directives now use quay.io/fedora/fedora:44 instead of quay.io/fedora/fedora:43.

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

Suggested reviewers: dustymabe

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: bumping coreos/coreos-assembler buildroot images to Fedora 44.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Stable And Deterministic Test Names ✅ Passed Only two ci-operator YAML configs changed (Fedora 43→44); no Ginkgo test titles or test code were added or modified.
Test Structure And Quality ✅ Passed PR only updates two ci-operator YAML build-root images from Fedora 43 to 44; no Ginkgo/test code or test patterns are touched.
Microshift Test Compatibility ✅ Passed PASS: The PR only updates ci-operator build_root Dockerfile base images in two YAML configs; no new Ginkgo tests or OpenShift API/resource usage were added.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: The PR only changes ci-operator YAML build-root base images; no Ginkgo e2e tests were added or modified, so the SNO check is not applicable.
Topology-Aware Scheduling Compatibility ✅ Passed Touched YAMLs only bump the build root Fedora base image; no replicas, affinity, node selectors, tolerations, or spread constraints were added.
Ote Binary Stdout Contract ✅ Passed Only two ci-operator YAML build-root base-image pins changed; no process-level OTE binary code or stdout writes were modified.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed Only ci-operator YAML changed Fedora base images; no Ginkgo e2e tests or network-dependent test code were added.
No-Weak-Crypto ✅ Passed Diff only bumps Fedora 43 to 44 in two build_root Dockerfile literals; no weak-crypto, custom crypto, or secret-comparison code was added.
Container-Privileges ✅ Passed Only the build_root Dockerfile base image changed to Fedora 44; no privileged, hostPID/Network/IPC, SYS_ADMIN, or allowPrivilegeEscalation settings are present.
No-Sensitive-Data-In-Logs ✅ Passed The PR only bumps the buildroot base image in two YAML configs; no logging code or sensitive-data exposure was added.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Checkov (3.3.8)
ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-main.yaml

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'

ci-operator/config/coreos/coreos-assembler/coreos-coreos-assembler-rhel-9.6.yaml

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@c4rt0: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-coreos-coreos-assembler-main-images coreos/coreos-assembler presubmit Ci-operator config changed
pull-ci-coreos-coreos-assembler-main-rhcos coreos/coreos-assembler presubmit Ci-operator config changed
pull-ci-coreos-coreos-assembler-rhel-9.6-images coreos/coreos-assembler presubmit Ci-operator config changed
pull-ci-coreos-coreos-assembler-rhel-9.6-rhcos coreos/coreos-assembler presubmit Ci-operator config changed

Prior to this PR being merged, you will need to either run and acknowledge or opt to skip these rehearsals.

Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@openshift-ci
openshift-ci Bot requested review from PeaceRebel and yasminvalim July 23, 2026 12:16
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 23, 2026
@dustymabe

Copy link
Copy Markdown
Member

/pj-rehearse auto-ack
/lgtm

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@dustymabe: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 23, 2026
@openshift-ci

openshift-ci Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: c4rt0, dustymabe

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 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@c4rt0: 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/rehearse/coreos/coreos-assembler/rhel-9.6/images c90134a link unknown /pj-rehearse pull-ci-coreos-coreos-assembler-rhel-9.6-images
ci/rehearse/coreos/coreos-assembler/rhel-9.6/rhcos c90134a link unknown /pj-rehearse pull-ci-coreos-coreos-assembler-rhel-9.6-rhcos
ci/rehearse/coreos/coreos-assembler/main/rhcos c90134a link unknown /pj-rehearse pull-ci-coreos-coreos-assembler-main-rhcos
ci/rehearse/coreos/coreos-assembler/main/images c90134a link unknown /pj-rehearse pull-ci-coreos-coreos-assembler-main-images

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. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants