Skip to content

Add new cluster profiles: metal-dpf-doca4 and metal-dpf-doca5 - #83086

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
josecastillolema:doca4-doca5
Aug 7, 2026
Merged

Add new cluster profiles: metal-dpf-doca4 and metal-dpf-doca5#83086
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
josecastillolema:doca4-doca5

Conversation

@josecastillolema

@josecastillolema josecastillolema commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Adds metal-dpf-doca4 and metal-dpf-doca5 cluster profiles for OpenShift CI.
  • Configures CI secret targets for both profiles in the ci namespace.
  • Adds one free Metal DPF quota slice in RDU2 for each profile.

Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>

    gitleaks.version: 8.30.0
    gitleaks.check-secrets: ENABLED
@coderabbitai

coderabbitai Bot commented Aug 7, 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: 48911507-96a9-476a-a33d-c4a45275fee8

📥 Commits

Reviewing files that changed from the base of the PR and between f85f4e1 and 25541ad.

📒 Files selected for processing (3)
  • core-services/ci-secret-bootstrap/_config.yaml
  • core-services/prow/02_config/_boskos.yaml
  • core-services/prow/02_config/generate-boskos.py

Walkthrough

The change adds CI secret targets and Boskos quota-slice resources for Metal DPF Doca4 and Doca5 in RDU2.

Changes

Metal DPF cluster provisioning

Layer / File(s) Summary
Boskos quota-slice definitions
core-services/prow/02_config/generate-boskos.py, core-services/prow/02_config/_boskos.yaml
Adds quota-slice definitions and free RDU2 resources for metal-dpf-doca4 and metal-dpf-doca5.
CI secret target registration
core-services/ci-secret-bootstrap/_config.yaml
Adds both clusters to the non_app_ci target group in the ci namespace.

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

Possibly related PRs

Suggested labels: lgtm, approved, rehearsals-ack

Suggested reviewers: davo911, amisstea

🚥 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 addition of the two new cluster profiles and matches the changes in the pull request.
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 The pull request changes only YAML and Python Boskos configuration; it adds no Ginkgo tests or test titles with dynamic values.
Test Structure And Quality ✅ Passed The commit changes only YAML and Python configuration; it adds no Ginkgo tests or It blocks, so the listed test-quality requirements are not applicable.
Microshift Test Compatibility ✅ Passed The commit changes only CI secret and Boskos configuration plus a Python generator; it adds no Ginkgo tests or test declarations, so MicroShift compatibility checks do not apply.
Single Node Openshift (Sno) Test Compatibility ✅ Passed The commit changes only YAML and Python configuration; it adds no Ginkgo tests or test-source paths, so SNO compatibility checks do not apply.
Topology-Aware Scheduling Compatibility ✅ Passed The PR only adds secret-bootstrap entries, Boskos resources, and quota-slice generator data; it modifies no deployment manifests, operators, controllers, or scheduling constraints.
Ote Binary Stdout Contract ✅ Passed The patch changes only YAML declarations and a Python quota map; it adds no OTE binary, Go suite setup, or process-level stdout/logging write.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The pull request changes only YAML configuration and Python quota data; it adds no Ginkgo tests, IPv4 assumptions, or external connectivity.
No-Weak-Crypto ✅ Passed The PR only adds cluster-secret and Boskos quota-slice configuration; added lines contain no weak-crypto algorithms, crypto APIs, custom crypto, or secret comparisons.
Container-Privileges ✅ Passed The commit adds only secret-group, Boskos resource, and quota mappings; added lines contain no privileged, host PID/network/IPC, SYS_ADMIN, root, or allowPrivilegeEscalation settings.
No-Sensitive-Data-In-Logs ✅ Passed The diff adds only cluster-secret resource names and quota identifiers; it adds no logging or print calls, and the generator writes YAML without emitting sensitive values.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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-ci
openshift-ci Bot requested review from bear-redhat and deepsm007 August 7, 2026 08:37
@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Aug 7, 2026
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@josecastillolema: no rehearsable tests are affected by this change

Note: If this PR includes changes to step registry files (ci-operator/step-registry/) and you expected jobs to be found, try rebasing your PR onto the base branch. This helps pj-rehearse accurately detect changes when the base branch has moved forward.

@josecastillolema

Copy link
Copy Markdown
Member Author

cc @wabouhamad

@openshift-ci

openshift-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

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

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

openshift-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: josecastillolema, Prucek

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 Aug 7, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit ba19e84 into openshift:main Aug 7, 2026
15 checks passed
@openshift-ci

openshift-ci Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

@josecastillolema: Updated the following 3 configmaps:

  • resources configmap in namespace ci at cluster app.ci using the following files:
    • key boskos.yaml using file core-services/prow/02_config/_boskos.yaml
  • ci-secret-bootstrap configmap in namespace ci at cluster app.ci using the following files:
    • key _config.yaml using file core-services/ci-secret-bootstrap/_config.yaml
  • ci-secret-bootstrap configmap in namespace ci at cluster core-ci using the following files:
    • key _config.yaml using file core-services/ci-secret-bootstrap/_config.yaml
Details

In response to this:

Summary by CodeRabbit

  • Adds metal-dpf-doca4 and metal-dpf-doca5 cluster profiles for OpenShift CI.
  • Configures CI secret targets for both profiles in the ci namespace.
  • Adds one free Metal DPF quota slice in RDU2 for each profile.

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.

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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants