coreos/rhel-coreos-config: specify pull credentials namespace by Roshan-R · Pull Request #85279 · openshift/release

Roshan-R · 2026-09-16T09:04:18Z

Use the test-credentials namespace for the RHCOS 9 and RHCOS 10 build tests as without it the tests are failing and blocking CI.

ref: coreos/rhel-coreos-config#324

Summary by CodeRabbit

The RHCOS 9 and RHCOS 10 build tests now use pull credentials from the test-credentials namespace. This fixes credential lookup failures that block OpenShift CI.

Use the `test-credentials` namespace for the RHCOS 9 and RHCOS 10 build tests as without it the tests are failing and blocking CI. ref: coreos/rhel-coreos-config#324

coderabbitai · 2026-09-16T09:04:28Z

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: Advanced

Run ID: 6f7087a2-a736-4054-8fbd-8cdfaa87de81

📥 Commits

Reviewing files that changed from the base of the PR and between 0843076 and 3853150.

📒 Files selected for processing (1)

ci-operator/config/coreos/rhel-coreos-config/coreos-rhel-coreos-config-main.yaml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

Walkthrough

The RHEL CoreOS 9 and 10 build test steps now set namespace: test-credentials for their ci-pull-credentials secret mounts.

Changes

RHEL CoreOS test credential namespaces

Layer / File(s)	Summary
Configure test credential mounts `ci-operator/config/coreos/rhel-coreos-config/coreos-rhel-coreos-config-main.yaml`	The `rhcos-9-build-test` and `rhcos-10-build-test` steps now specify `namespace: test-credentials` on their `ci-pull-credentials` secret mounts.

Priority: ➖ Normal

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 38531

The credential namespace updates match the stated CI test objective and are ready to merge.

🚥 Pre-merge checks | ✅ 5 | ❌ 10

❌ Failed checks (10 inconclusive)

Check name	Status	Explanation	Resolution
Stable And Deterministic Test Names	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Test Structure And Quality	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Microshift Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Single Node Openshift (Sno) Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Topology-Aware Scheduling Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Ote Binary Stdout Contract	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Ipv6 And Disconnected Network Test Compatibility	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
No-Weak-Crypto	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
Container-Privileges	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.
No-Sensitive-Data-In-Logs	❓ Inconclusive	Repository clone failed, so this custom check could not run with code access.	Retry the review run. If this persists, inspect pre-merge custom-check logs for infrastructure or agent runtime failures.

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the affected configuration and the main change: specifying the pull credentials namespace. It matches the PR objective and changed files.
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 PR with unit tests

Warning

Git: CodeRabbit could not clone the repository, so clone-backed analysis was skipped and this review may be incomplete. Verify repository clone access, such as SSH credentials, before requesting another full review. If clone access is intentionally unavailable, use path_filters to narrow the review scope.

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

openshift-merge-bot · 2026-09-16T09:07:18Z

[REHEARSALNOTIFIER]
@Roshan-R: 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-rhel-coreos-config-main-rhcos-10-build-test	coreos/rhel-coreos-config	presubmit	Ci-operator config changed
pull-ci-coreos-rhel-coreos-config-main-rhcos-9-build-test	coreos/rhel-coreos-config	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.

jbtrystram

/lgtm

jbtrystram · 2026-09-16T09:23:02Z

/pj-rehearse pull-ci-coreos-rhel-coreos-config-main-rhcos-10-build-test

openshift-merge-bot · 2026-09-16T09:23:05Z

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

openshift-ci · 2026-09-16T09:23:12Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jbtrystram, Roshan-R

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:

~~ci-operator/config/coreos/rhel-coreos-config/OWNERS~~ [Roshan-R,jbtrystram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

jbtrystram · 2026-09-16T09:23:15Z

/pj-rehearse pull-ci-coreos-rhel-coreos-config-main-rhcos-9-build-test

openshift-merge-bot · 2026-09-16T09:23:19Z

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

openshift-ci · 2026-09-16T12:09:50Z

@Roshan-R: 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.

coreos/rhel-coreos-config: specify pull credentials namespace

3853150

Use the `test-credentials` namespace for the RHCOS 9 and RHCOS 10 build tests as without it the tests are failing and blocking CI. ref: coreos/rhel-coreos-config#324

openshift-ci Bot requested review from angelcerveraroldan and cverna September 16, 2026 09:04

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

jbtrystram approved these changes Sep 16, 2026

View reviewed changes

openshift-ci Bot assigned jbtrystram Sep 16, 2026

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

Navigation Menu

Uh oh!

coreos/rhel-coreos-config: specify pull credentials namespace - #85279

coreos/rhel-coreos-config: specify pull credentials namespace#85279
Roshan-R wants to merge 1 commit into
openshift:mainfrom
Roshan-R:fix-coreos-ci

Roshan-R commented Sep 16, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Sep 16, 2026 •

edited

Loading

❌ Failed checks (10 inconclusive)

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

jbtrystram left a comment

Uh oh!

jbtrystram commented Sep 16, 2026

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

openshift-ci Bot commented Sep 16, 2026

Uh oh!

jbtrystram commented Sep 16, 2026

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

openshift-ci Bot commented Sep 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Roshan-R commented Sep 16, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Sep 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

❌ Failed checks (10 inconclusive)

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

jbtrystram left a comment

Choose a reason for hiding this comment

Uh oh!

jbtrystram commented Sep 16, 2026

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

openshift-ci Bot commented Sep 16, 2026

Uh oh!

jbtrystram commented Sep 16, 2026

Uh oh!

openshift-merge-bot Bot commented Sep 16, 2026

Uh oh!

openshift-ci Bot commented Sep 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Roshan-R commented Sep 16, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Sep 16, 2026 •

edited

Loading