Kueue downstream ci for s390x by Meghagaur · Pull Request #83236 · openshift/release

Meghagaur · 2026-08-11T09:32:31Z

Summary by CodeRabbit

This PR adds downstream CI support for the Kueue operator release-1.4 on s390x. It configures nightly release inputs, resource requests, and daily downstream end-to-end tests.

It also adds a libvirt-s390x workflow that provisions temporary OpenShift 4.21 clusters on remote IBM Z hypervisors. The workflow includes setup, cleanup, log collection, ownership, and workflow metadata.

…ease-1.4

openshift-ci · 2026-08-11T09:33:06Z

Hi @Meghagaur. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

openshift-ci · 2026-08-11T09:33:35Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Meghagaur
Once this PR has been reviewed and has the lgtm label, please assign rphillips for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

coderabbitai · 2026-08-11T09:36:12Z

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: 76db104e-4c59-4726-85dc-618b7ee91732

📥 Commits

Reviewing files that changed from the base of the PR and between 1948eac and 1b35c38.

⛔ Files ignored due to path filters (1)

ci-operator/jobs/openshift/kueue-operator/openshift-kueue-operator-release-1.4-periodics.yaml is excluded by !ci-operator/jobs/**

📒 Files selected for processing (1)

ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4__libvirt-s390x.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4__libvirt-s390x.yaml

Walkthrough

The PR adds a remote s390x libvirt workflow for the Kueue operator and connects it to the release-1.4 OpenShift CI configuration. It also adds workflow ownership and metadata.

Changes

Kueue s390x libvirt CI

Layer / File(s)	Summary
Remote libvirt workflow `ci-operator/step-registry/kueue-operator/libvirt-s390x/*`	The workflow provisions an ephemeral s390x OpenShift cluster on IBM Z libvirt hypervisors. It adds setup, diagnostics, cleanup, and environment settings.
Release CI integration `ci-operator/config/openshift/kueue-operator/openshift-kueue-operator-release-1.4__libvirt-s390x.yaml`, `ci-operator/step-registry/kueue-operator/libvirt-s390x/*`	The release configuration adds images, release inputs, resource requests, repository metadata, a daily downstream end-to-end test, workflow metadata, and ownership.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Prow
  participant Boskos
  participant KueueOperatorWorkflow
  participant IBMZLibvirtHypervisors
  participant DownstreamE2ETest
  Prow->>Boskos: Allocate libvirt resources
  Prow->>KueueOperatorWorkflow: Start s390x workflow
  KueueOperatorWorkflow->>IBMZLibvirtHypervisors: Provision ephemeral OpenShift cluster
  KueueOperatorWorkflow->>DownstreamE2ETest: Run daily downstream test
  KueueOperatorWorkflow->>IBMZLibvirtHypervisors: Gather logs and clean up resources

Possibly related PRs

openshift/release#82603: Introduces the libvirt-s390x-vpn profile used by this workflow.
openshift/release#83138: Registers the libvirt-s390x-vpn-hcp profile used by related s390x workflow configuration.
openshift/release#83139: Configures another s390x libvirt workflow to use the libvirt-s390x-vpn profile.

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Ote Binary Stdout Contract	❓ Inconclusive	Tool connection failed before the changed files and OTE-related code could be fully inspected.	Retry repository inspection to verify whether this pull request changes any process-level OTE stdout behavior.

✅ Passed checks (14 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 Kueue downstream CI changes for the s390x platform.
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 PR adds CI YAML, workflow metadata, and ownership files only; no Ginkgo It, Describe, Context, or When test titles, dynamic or otherwise, are introduced.
Test Structure And Quality	✅ Passed	The PR changes only CI YAML, OWNERS, and metadata; no Ginkgo test code, It blocks, or cluster assertions are added or modified.
Microshift Test Compatibility	✅ Passed	The patch adds CI configuration, workflow YAML, OWNERS, and metadata only; it adds no Ginkgo e2e tests or test declarations requiring MicroShift API review.
Single Node Openshift (Sno) Test Compatibility	✅ Passed	The pull request changes CI configuration and workflow wiring only; no new Ginkgo It/Describe/Context/When tests or multi-node test logic were added.
Topology-Aware Scheduling Compatibility	✅ Passed	The PR changes CI configuration and a libvirt workflow only; they add no operator manifests, controllers, replicas, affinity, topology spread, selectors, or PDB scheduling constraints.
Ipv6 And Disconnected Network Test Compatibility	✅ Passed	The pull request changes CI YAML and metadata only; it adds no Ginkgo test declarations or test code with IPv4 assumptions or external connectivity.
No-Weak-Crypto	✅ Passed	The checked-out patch only changes the cluster profile labels. No MD5, SHA1, DES, RC4, Blowfish, ECB, custom crypto, or secret comparisons are added.
Container-Privileges	✅ Passed	The four added files contain no privileged:true, hostPID, hostNetwork, hostIPC, SYS_ADMIN, allowPrivilegeEscalation, or root securityContext declarations; capabilities: intranet is a job capability.
No-Sensitive-Data-In-Logs	✅ Passed	The added YAML and metadata contain no credentials, tokens, PII, or log statements; they only reference standard gather steps and a job-log URL.

✨ 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.}

sivaramsingana · 2026-08-11T10:02:46Z

/ok-to-test

openshift-ci · 2026-08-11T10:03:17Z

@sivaramsingana: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

Details

In response to this:

/ok-to-test

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.

veera-damisetti · 2026-08-11T10:23:51Z

/ok-to-test

bj2122 · 2026-08-11T10:33:03Z

/ok-to-test

openshift-merge-bot · 2026-08-11T11:59:17Z

[REHEARSALNOTIFIER]
@Meghagaur: 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-openshift-kueue-operator-release-1.4-libvirt-s390x-images	openshift/kueue-operator	presubmit	Presubmit changed
periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22	N/A	periodic	Periodic changed

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.

Meghagaur · 2026-08-11T12:22:22Z

/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22

openshift-merge-bot · 2026-08-11T12:22:25Z

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

Meghagaur · 2026-08-11T13:13:07Z

/ok-to-test

openshift-ci · 2026-08-11T13:36:25Z

@Meghagaur: 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/periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22	`1b35c38`	link	unknown	`/pj-rehearse periodic-ci-openshift-kueue-operator-release-1.4-libvirt-s390x-test-e2e-downstream-4-22`
ci/prow/config	`1b35c38`	link	true	`/test config`

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.

Meghagaur and others added 3 commits August 11, 2026 14:27

ci(kueue-operator): add libvirt-s390x e2e workflow and config for rel…

f2dcbab

…ease-1.4

Merge branch 'openshift:main' into kueue-oz-s390x

9c4965f

generated jobs for kueue-1.4 downstream

1948eac

openshift-ci Bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 11, 2026

openshift-ci Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Aug 11, 2026

Updated the cluster profile to oz profile

1b35c38

Meghagaur force-pushed the kueue-oz-s390x branch from 2c27259 to 1b35c38 Compare August 11, 2026 11:54

Navigation Menu

Uh oh!

Kueue downstream ci for s390x - #83236

Kueue downstream ci for s390x#83236
Meghagaur wants to merge 4 commits into
openshift:mainfrom
Meghagaur:kueue-oz-s390x

Meghagaur commented Aug 11, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

coderabbitai Bot commented Aug 11, 2026 •

edited

Loading

Pre-merge checks failed

Uh oh!

sivaramsingana commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

veera-damisetti commented Aug 11, 2026

Uh oh!

bj2122 commented Aug 11, 2026

Uh oh!

openshift-merge-bot Bot commented Aug 11, 2026

Uh oh!

Meghagaur commented Aug 11, 2026

Uh oh!

openshift-merge-bot Bot commented Aug 11, 2026

Uh oh!

Meghagaur commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Meghagaur commented Aug 11, 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!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

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

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Pre-merge checks failed

❌ Failed checks (1 inconclusive)

Uh oh!

sivaramsingana commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

veera-damisetti commented Aug 11, 2026

Uh oh!

bj2122 commented Aug 11, 2026

Uh oh!

openshift-merge-bot Bot commented Aug 11, 2026

Uh oh!

Meghagaur commented Aug 11, 2026

Uh oh!

openshift-merge-bot Bot commented Aug 11, 2026

Uh oh!

Meghagaur commented Aug 11, 2026

Uh oh!

openshift-ci Bot commented Aug 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Meghagaur commented Aug 11, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Aug 11, 2026 •

edited

Loading