Skip to content

ci: drop IBM Z heavy-build periodic; run full parallel suite in conformance#79333

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
ibm-adarsh:heavy-build-removal
May 15, 2026
Merged

ci: drop IBM Z heavy-build periodic; run full parallel suite in conformance#79333
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
ibm-adarsh:heavy-build-removal

Conversation

@ibm-adarsh
Copy link
Copy Markdown
Contributor

@ibm-adarsh ibm-adarsh commented May 15, 2026

Remove the separate ocp-heavy-build-ovn-remote-libvirt-s390x periodic jobs
from openshift/multiarch nightly configs (4.12–4.16) and regenerate Prow
periodics.

Delete the libvirt conf logic that excluded bandwidth-heavy build/image tests
from conformance-parallel on s390x (≤4.16) or materialized them as a dedicated
test list for heavy-build jobs.

Remove the heavy-build TEST_TYPE branch and heavy_build helper from the
libvirt e2e test step, since that job type is no longer used.

Rationale: Z conformance parallel jobs should run the full openshift/
conformance/parallel suite without a split heavy-build lane.

Summary

This PR removes the separate heavy-build test job for IBM Z (s390x) systems from OpenShift's CI infrastructure and consolidates the test workload into the standard conformance-parallel suite.

Changes

Multiarch CI Configuration (Versions 4.12–4.16)
Removed the ocp-heavy-build-ovn-remote-libvirt-s390x periodic job entries from nightly multiarch CI configurations for OpenShift versions 4.12, 4.13, 4.14, 4.15, and 4.16. This job type previously ran bandwidth-heavy build and image tests separately from the main conformance suite on s390x architectures.

Libvirt E2E Test Configuration
Deleted conditional logic in the libvirt configuration step (openshift-e2e-libvirt-conf-commands.sh) that excluded certain tests on s390x systems when running conformance-parallel tests. This logic previously maintained a separate "yellow zone" exclusion list for bandwidth-heavy tests on z/ppc64le architectures.

Libvirt Test Runner
Removed the heavy_build() function and its corresponding heavy-build case branch from the libvirt E2E test runner script (openshift-e2e-libvirt-test-commands.sh). The TEST_TYPE environment variable no longer recognizes the heavy-build value, as this job type is being deprecated.

Impact

s390x conformance jobs will now execute the complete openshift/conformance/parallel test suite without bandwidth-heavy tests being excluded or materialized as a separate periodic job. This simplifies the CI infrastructure for IBM Z systems while maintaining full test coverage.

…rmance

Signed-off-by: Adarsh Kumar Yadav <adarsh.kumar.yadav1@ibm.com>
@openshift-ci openshift-ci Bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label May 15, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 15, 2026

Hi @ibm-adarsh. 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.

Tip

We noticed you've done this a few times! Consider joining the org to skip this step and gain /lgtm and other bot rights. We recommend asking approvers on your previous PRs to sponsor you.

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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

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

Run ID: 7f03fbc2-b3f2-4c67-878f-71d1d70d21f3

📥 Commits

Reviewing files that changed from the base of the PR and between 4264ff9 and d9b2efb.

⛔ Files ignored due to path filters (1)
  • ci-operator/jobs/openshift/multiarch/openshift-multiarch-main-periodics.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (7)
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.12.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.13.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.14.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.15.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.16.yaml
  • ci-operator/step-registry/openshift/e2e/libvirt/conf/openshift-e2e-libvirt-conf-commands.sh
  • ci-operator/step-registry/openshift/e2e/libvirt/test/openshift-e2e-libvirt-test-commands.sh
💤 Files with no reviewable changes (7)
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.15.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.16.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.12.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.13.yaml
  • ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.14.yaml
  • ci-operator/step-registry/openshift/e2e/libvirt/conf/openshift-e2e-libvirt-conf-commands.sh
  • ci-operator/step-registry/openshift/e2e/libvirt/test/openshift-e2e-libvirt-test-commands.sh

Walkthrough

Removes the heavy-build test type and associated job definitions from OpenShift multiarch CI configurations across four nightly release branches (4.12, 4.13, 4.15, 4.16), migrates 4.14 to use conformance-parallel, and removes heavy-build execution logic from libvirt test scripts.

Changes

Remove heavy-build test type from multiarch CI

Layer / File(s) Summary
Remove/migrate heavy-build job definitions across nightly release configs
ci-operator/config/openshift/multiarch/openshift-multiarch-main__nightly-4.12.yaml, openshift-multiarch-main__nightly-4.13.yaml, openshift-multiarch-main__nightly-4.14.yaml, openshift-multiarch-main__nightly-4.15.yaml, openshift-multiarch-main__nightly-4.16.yaml
Deletes ocp-heavy-build-ovn-remote-libvirt-s390x job entries from nightly-4.12, 4.13, 4.15, and 4.16; replaces the job in nightly-4.14 with ocp-e2e-ovn-remote-libvirt-s390x running conformance-parallel.
Remove heavy-build test execution functions and dispatch
ci-operator/step-registry/openshift/e2e/libvirt/conf/openshift-e2e-libvirt-conf-commands.sh, ci-operator/step-registry/openshift/e2e/libvirt/test/openshift-e2e-libvirt-test-commands.sh
Removes the heavy_build() function, deletes the heavy-build) case from TEST_TYPE dispatch, and removes s390x yellow-zone test exclusion conditional logic that only applied to heavy-build and conformance-parallel on non-ppc64le systems ≤ 4.16.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested labels

lgtm, approved

Suggested reviewers

  • wking
  • neisw
🚥 Pre-merge checks | ✅ 12
✅ Passed checks (12 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing heavy-build periodic jobs for IBM Z (s390x) and switching to running the full parallel conformance suite instead.
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 This PR removes CI job definitions and bash infrastructure scripts. No Ginkgo test definitions (It(), Describe(), etc.) are introduced or modified. The check does not apply to this PR.
Test Structure And Quality ✅ Passed This PR does not contain Ginkgo test code. It only modifies YAML CI configurations and shell helper scripts. The custom check for Ginkgo test quality is inapplicable to this PR.
Microshift Test Compatibility ✅ Passed This PR does not add any new Ginkgo e2e tests. It removes infrastructure only (CI job configs and test execution scripts). The MicroShift compatibility check is not applicable.
Single Node Openshift (Sno) Test Compatibility ✅ Passed This PR is a removal/cleanup PR that deletes CI job definitions and test infrastructure code. No new Ginkgo e2e tests are added. The SNO compatibility check applies to newly added tests, not removals.
Topology-Aware Scheduling Compatibility ✅ Passed PR modifies CI configuration and test scripts only, not deployment manifests, operator code, or controllers. Topology-aware scheduling check only applies to those artifact types. Not applicable.
Ote Binary Stdout Contract ✅ Passed This PR modifies CI configuration and shell scripts that invoke OTE binary. The check applies to OTE source code itself. No OTE Go code is changed.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed This PR removes CI jobs and helpers, not adds Ginkgo tests. The check applies only to new test additions. No new test code is present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 and usage tips.

@openshift-ci openshift-ci Bot requested review from KeerthanaAP and alkleinrh May 15, 2026 06:37
@openshift-merge-bot
Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@ibm-adarsh: 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-installer-main-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-5.1-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-5.0-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.23-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.22-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.21-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.20-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.19-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.18-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.17-e2e-powervs-ovn openshift/installer presubmit Registry content changed
pull-ci-openshift-installer-release-4.16-altinfra-e2e-powervs-capi-ovn openshift/installer presubmit Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.20-ocp-e2e-ovn-remote-libvirt-s390x N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.11-ocp-e2e-ovn-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.23-ocp-e2e-ovn-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.14-ocp-jenkins-e2e-ovn-remote-libvirt-s390x N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.19-ocp-e2e-ovn-agent-remote-libvirt-multi-z-z N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-5.0-ocp-e2e-ovn-agent-remote-s2s-libvirt-multi-p-p N/A periodic Registry content changed
periodic-ci-openshift-operator-framework-operator-controller-release-5.0-periodics-e2e-compact-ovn-remote-libvirt-s390x-extended-f2 N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.8-ocp-image-ecosystem-remote-libvirt-s390x N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.14-ocp-image-ecosystem-ovn-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.10-ocp-e2e-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.11-ocp-e2e-serial-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.22-ocp-e2e-ovn-agent-remote-libvirt-multi-z-z N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.15-ocp-image-ecosystem-ovn-remote-s2s-libvirt-ppc64le N/A periodic Registry content changed
periodic-ci-openshift-multiarch-main-nightly-4.21-ocp-e2e-ovn-agent-remote-libvirt-multi-z-z N/A periodic Registry content changed

A total of 268 jobs have been affected by this change. The above listing is non-exhaustive and limited to 25 jobs.

A full list of affected jobs can be found here
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.

@ibm-adarsh
Copy link
Copy Markdown
Contributor Author

/rehearsal-ack

@sandisamp
Copy link
Copy Markdown
Contributor

/ok-to-test

@openshift-ci 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 May 15, 2026
@v78singh
Copy link
Copy Markdown

/ok-to-test

@v78singh
Copy link
Copy Markdown

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label May 15, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 15, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ibm-adarsh, v78singh

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 May 15, 2026
@ibm-adarsh
Copy link
Copy Markdown
Contributor Author

/rehearsals-ack

@ibm-adarsh
Copy link
Copy Markdown
Contributor Author

/pj-rehearse ack

@openshift-merge-bot
Copy link
Copy Markdown
Contributor

@ibm-adarsh: 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 May 15, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 15, 2026

@ibm-adarsh: 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 2638457 into openshift:main May 15, 2026
18 checks passed
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. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. rehearsals-ack Signifies that rehearsal jobs have been acknowledged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants