Skip to content

WINC-2139: Remove unstable legacy Nutanix Windows jobs - #85219

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
rrasouli:winc-2139-remove-nutanix-winc
Sep 15, 2026
Merged

openshift-merge-bot[bot] merged 1 commit into
openshift:mainfrom
rrasouli:winc-2139-remove-nutanix-winc

Conversation

@rrasouli

@rrasouli rrasouli commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Remove the unstable legacy Nutanix Windows Containers Prow jobs tracked in WINC-2139. The reported rehearsal failed before Windows tests ran: bastion provisioning waited about 10 minutes for an IP and exited with Failed to get host IP.

This removes 17 legacy test definitions and their generated Prow jobs:

  • 10 openshift-tests-private periodics on main, releases 4.16–4.22, 5.0, and 5.1.
  • Six debug-winc-nutanix-ipi presubmits in the public/private openshift-tests-private configurations.
  • The nutanix-ipi-ovn-winc FBC postsubmit.

Removal is approved. The OTE replacement is tracked in #85079, whose main-branch run was reported successful. That PR is still open at submission; its optional nutanix-e2e-ote job is triggered with /test nutanix-e2e-ote. It does not automatically replace periodic scheduling or establish equivalent upgrade coverage.

Existing WMCO operator jobs and reusable step-registry workflows remain available.

Validation: ci-operator-checkconfig and ci-operator-prowgen completed successfully. A semantic comparison verified exactly 17 source definitions and 17 generated jobs were removed, with all other configuration unchanged. git diff --check passed; no legacy Nutanix Windows job references remain in source or generated job configurations.

Full make update was attempted but could not complete because of local Podman storage/runtime failures. Generation succeeded using isolated storage; the remaining sanitizer invocation failed with Exec format error. The full repository checks need to complete in CI.

Summary by CodeRabbit

This PR removes 17 unstable legacy Nutanix Windows Containers jobs from OpenShift CI:

  • 10 periodic jobs across main and releases 4.16–4.22, 5.0, and 5.1.
  • Six debug-winc-nutanix-ipi presubmits.
  • One nutanix-ipi-ovn-winc FBC postsubmit.

The change removes the failing Nutanix workflow while leaving existing WMCO jobs and reusable workflows unchanged. Configuration checks and Prow job generation passed. Full make update did not complete because of local Podman and sanitizer runtime failures.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Sep 15, 2026
@openshift-ci-robot

openshift-ci-robot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

@rrasouli: This pull request references WINC-2139 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "5.1.0" version, but no target version was set.

Details

In response to this:

Remove the unstable legacy Nutanix Windows Containers Prow jobs tracked in WINC-2139. The reported rehearsal failed before Windows tests ran: bastion provisioning waited about 10 minutes for an IP and exited with Failed to get host IP.

This removes 17 legacy test definitions and their generated Prow jobs:

  • 10 openshift-tests-private periodics on main, releases 4.16–4.22, 5.0, and 5.1.
  • Six debug-winc-nutanix-ipi presubmits in the public/private openshift-tests-private configurations.
  • The nutanix-ipi-ovn-winc FBC postsubmit.

Removal is approved. The OTE replacement is tracked in #85079, whose main-branch run was reported successful. That PR is still open at submission; its optional nutanix-e2e-ote job is triggered with /test nutanix-e2e-ote. It does not automatically replace periodic scheduling or establish equivalent upgrade coverage.

Existing WMCO operator jobs and reusable step-registry workflows remain available.

Validation: ci-operator-checkconfig and ci-operator-prowgen completed successfully. A semantic comparison verified exactly 17 source definitions and 17 generated jobs were removed, with all other configuration unchanged. git diff --check passed; no legacy Nutanix Windows job references remain in source or generated job configurations.

Full make update was attempted but could not complete because of local Podman storage/runtime failures. Generation succeeded using isolated storage; the remaining sanitizer invocation failed with Exec format error. The full repository checks need to complete in 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 openshift-eng/jira-lifecycle-plugin repository.

@coderabbitai

coderabbitai Bot commented Sep 15, 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: Advanced

Run ID: d9b4cdde-e004-4aad-bffe-7936da859e43

📥 Commits

Reviewing files that changed from the base of the PR and between edadb51 and 2cc7d92.

⛔ Files ignored due to path filters (17)
  • ci-operator/jobs/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.21-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.22-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-main-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.18-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.19-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.20-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22-presubmits.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.0-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.1-periodics.yaml is excluded by !ci-operator/jobs/**
  • ci-operator/jobs/openshift/windows-machine-config-operator-fbc/openshift-windows-machine-config-operator-fbc-main-postsubmits.yaml is excluded by !ci-operator/jobs/**
📒 Files selected for processing (17)
  • ci-operator/config/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.21.yaml
  • ci-operator/config/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.22.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-main.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.18__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.19__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.20__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.0__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.1__amd64-nightly.yaml
  • ci-operator/config/openshift/windows-machine-config-operator-fbc/openshift-windows-machine-config-operator-fbc-main.yaml
💤 Files with no reviewable changes (17)
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16.yaml
  • ci-operator/config/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.21.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.0__amd64-nightly.yaml
  • ci-operator/config/openshift-priv/openshift-tests-private/openshift-priv-openshift-tests-private-release-4.22.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.19__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.21__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.20__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.18__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.17.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22.yaml
  • ci-operator/config/openshift/windows-machine-config-operator-fbc/openshift-windows-machine-config-operator-fbc-main.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-5.1__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-main.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.16__amd64-nightly.yaml
  • ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-release-4.22__amd64-nightly.yaml

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


Walkthrough

The change removes Nutanix Windows Containers test jobs from OpenShift release and nightly configurations, plus a Windows Machine Config Operator postsubmit configuration. No replacement jobs are added.

Changes

Nutanix Windows CI removal

Layer / File(s) Summary
Optional and postsubmit test removal
ci-operator/config/openshift-priv/..., ci-operator/config/openshift/openshift-tests-private/..., ci-operator/config/openshift/windows-machine-config-operator-fbc/...
Removed optional Nutanix Windows Containers jobs from OpenShift release configurations and the postsubmit job from the Windows Machine Config Operator configuration.
Scheduled test removal
ci-operator/config/openshift/openshift-tests-private/openshift-openshift-tests-private-*.yaml
Removed scheduled Nutanix Windows Containers jobs across the main and release CI configurations, including their schedules, workflows, profiles, and test settings.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to 2cc7d

The change removes the specified legacy CI jobs, with no supported evidence of an additional broken workflow or release risk.

🚥 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 and concisely describes the main change: removing unstable legacy Nutanix Windows jobs tracked by WINC-2139.
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.
Stable And Deterministic Test Names ✅ Passed PASS: The reviewed range changes only YAML CI configuration and generated Prow job files. All 34 files contain deletions only, with zero added lines. No Ginkgo test titles or dynamic values are introd…
Test Structure And Quality ✅ Passed The check is not applicable to this pull request. The authoritative diff changes only CI configuration and generated Prow job YAML files, with 2,078 deletions and no additions. The changed paths conta…
Microshift Test Compatibility ✅ Passed The custom check applies only when the pull request adds Ginkgo e2e tests. The reviewed diff changes 34 YAML configuration/job files and has zero added lines. It removes existing Nutanix job definitio…
Single Node Openshift (Sno) Test Compatibility ✅ Passed The pull-request diff contains only YAML configuration and generated-job changes. All 34 files have zero added lines; the changes remove existing Nutanix job definitions. No Ginkgo tests or other test…
Topology-Aware Scheduling Compatibility ✅ Passed The check is not applicable. The authoritative PR diff changes only ci-operator/config/** and ci-operator/jobs/** YAML files. It removes 17 CI test definitions and their generated Prow jobs, with …
Ote Binary Stdout Contract ✅ Passed The pull request changes only YAML CI configuration and generated Prow job files. The authoritative diff contains 34 changed files, all with zero added lines and only deletions. No Go source or proces…
Ipv6 And Disconnected Network Test Compatibility ✅ Passed The pull-request diff contains only deletions from 17 CI configuration files and 17 generated job files. The authoritative diff has zero added lines and no changed Go, test, or e2e source files. There…
No-Weak-Crypto ✅ Passed The pull-request diff contains only YAML configuration changes. Git reports 0 added lines and 2,078 deleted lines across 34 files, with the changes removing Nutanix Windows job definitions and generat…
Container-Privileges ✅ Passed PASS. The reviewed range contains 0 added lines and only removes 17 CI definitions plus their generated Prow jobs. No added patch text contains privileged, hostPID, hostNetwork, hostIPC, SYS_ADMIN, al…
No-Sensitive-Data-In-Logs ✅ Passed The review-scoped diff adds 0 lines and removes 2,078 lines across CI YAML configuration and generated Prow job files. The removed job definitions contain no new logging statements or sensitive-data o…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

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

@openshift-ci

openshift-ci Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

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

@jrvaldes jrvaldes 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 Sep 15, 2026
@openshift-ci

openshift-ci Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jrvaldes, rrasouli

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 Sep 15, 2026
@jrvaldes

Copy link
Copy Markdown
Contributor

/pj-rehearse ack

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@jrvaldes: 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 Sep 15, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit 6e64e62 into openshift:main Sep 15, 2026
17 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. 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.

3 participants