Skip to content

MULTIARCH-6023: Add PowerVS dal14 lease with 4 quota slices - #82273

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
Neha-dot-Yadav:powervs-add-dal14
Jul 23, 2026
Merged

MULTIARCH-6023: Add PowerVS dal14 lease with 4 quota slices#82273
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
Neha-dot-Yadav:powervs-add-dal14

Conversation

@Neha-dot-Yadav

@Neha-dot-Yadav Neha-dot-Yadav commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Adds PowerVS dal14 as a new IPI cluster profile slot (powervs-10) with 4 quota slices for ipi-powervs workflow.
Dependent PR for addition in cluster-profile-config #82274 to be added after it is merged.

Summary by CodeRabbit

Adds the powervs-10 IPI cluster profile for PowerVS dal14 with four quota slices. Updates CI workflows, region handling, secret bootstrapping, and Boskos resource generation so the new leased infrastructure can be provisioned and used by OpenShift CI.

Signed-off-by: Neha Yadav <neha.yadav3@ibm.com>
@openshift-ci-robot

openshift-ci-robot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

@Neha-dot-Yadav: This pull request references MULTIARCH-6023 which is a valid jira issue.

Details

In response to this:

Adds PowerVS dal14 as a new IPI cluster profile slot (powervs-10) with 4 quota slices for ipi-powervs workflow

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-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 22, 2026
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

Adds four DAL14 PowerVS quota-slice resources, configures Boskos generation and secret bootstrap mappings, and updates PowerVS cluster setup scripts to derive DAL14 and us-south region values.

Changes

PowerVS DAL14 quota slices

Layer / File(s) Summary
Quota-slice resource inventory
core-services/prow/02_config/generate-boskos.py, core-services/prow/02_config/_boskos.yaml
Defines and generates four dal14-powervs-10-quota-slice resources in the free Boskos inventory.
Leased resource configuration
ci-operator/step-registry/ipi/conf/powervs/ipi-conf-powervs-commands.sh, core-services/ci-secret-bootstrap/_config.yaml
Adds DAL14 leased-resource cluster names, service-instance secret lookups, placement values, and the cluster-secrets-powervs-10 bootstrap mapping.
Installation region mapping
ci-operator/step-registry/upi/install/powervs/cluster/upi-install-powervs-cluster-commands.sh
Maps dal14 to the dal PowerVS region and us-south to the matching VPC region.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • openshift/release#82263: Adds a different quota-slice resource family through the same Boskos generator and generated inventory.

Suggested reviewers: amisstea, roivaz

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
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 No Ginkgo test titles were added or changed; the PR only edits shell scripts and YAML/config generators.
Test Structure And Quality ✅ Passed PR touches only shell scripts, YAML, and a Python generator; no Ginkgo test files or test code were changed, so the test-quality check is not applicable.
Microshift Test Compatibility ✅ Passed No new Ginkgo/e2e tests were added; the PR only changes shell scripts and YAML config, so MicroShift compatibility isn't impacted.
Single Node Openshift (Sno) Test Compatibility ✅ Passed HEAD only changes CI/config scripts and Boskos data; no Ginkgo e2e test additions or SNO-sensitive test logic were added.
Topology-Aware Scheduling Compatibility ✅ Passed Only CI/bootstrap and Boskos resource-mapping files changed; no deployment manifests, controllers, or scheduling fields (affinity, nodeSelector, topology spread, PDBs) were added.
Ote Binary Stdout Contract ✅ Passed The PR only adds config/assignment lines in shell/Python generators; no new process-level stdout writes, klog, or print calls were added.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR only updates PowerVS scripts/YAML for dal14 quota slices; no new Ginkgo e2e tests or new external-connectivity/IP-family assumptions were added.
No-Weak-Crypto ✅ Passed Touched files only add dal14/powervs-10 plumbing; exact weak-primitive searches found no MD5/SHA1/DES/RC4/ECB/custom-crypto or secret-compare usage.
Container-Privileges ✅ Passed No changed manifest sets privileged/hostPID/hostNetwork/hostIPC/allowPrivilegeEscalation; PR only updates PowerVS/Boskos/secret mapping logic.
No-Sensitive-Data-In-Logs ✅ Passed No new logging was introduced; the dal14 changes only add case mappings and resource/secret references, while existing echo statements were unchanged.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding the PowerVS dal14 lease with four quota slices.
✨ 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 droslean and sudeeshjohn July 22, 2026 08:46
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@Neha-dot-Yadav: pj-rehearse could not automatically process this event because the request waited in queue for longer than 5 minutes. Use /pj-rehearse to trigger rehearsals manually.

@prb112 prb112 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 Indicates that a PR is ready to be merged. label Jul 22, 2026
@danilo-gemoli

Copy link
Copy Markdown
Contributor

/lgtm

@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: danilo-gemoli, Neha-dot-Yadav, prb112

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 23, 2026
@Neha-dot-Yadav

Copy link
Copy Markdown
Contributor Author

/pj-rehearse ack

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

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

@openshift-merge-bot openshift-merge-bot Bot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Jul 23, 2026
@openshift-ci

openshift-ci Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@Neha-dot-Yadav: 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-merge-bot
openshift-merge-bot Bot merged commit 3268e14 into openshift:main Jul 23, 2026
19 checks passed
@openshift-ci

openshift-ci Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

@Neha-dot-Yadav: Updated the following 3 configmaps:

  • 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
  • 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
Details

In response to this:

Adds PowerVS dal14 as a new IPI cluster profile slot (powervs-10) with 4 quota slices for ipi-powervs workflow.
Dependent PR for addition in cluster-profile-config #82274 to be added after it is merged.

Summary by CodeRabbit

Adds the powervs-10 IPI cluster profile for PowerVS dal14 with four quota slices. Updates CI workflows, region handling, secret bootstrapping, and Boskos resource generation so the new leased infrastructure can be provisioned and used by OpenShift 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 kubernetes-sigs/prow repository.

apahim pushed a commit to apahim/release that referenced this pull request Jul 24, 2026
mrniranjan pushed a commit to mrniranjan/release that referenced this pull request Jul 28, 2026
amogh-redhat pushed a commit to amogh-redhat/release that referenced this pull request Aug 5, 2026
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. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants