Skip to content

OCPEDGE-2355: feat: skip unsupported commatrix tests for two node#30720

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
eggfoobar:skip-commatrix-twonode
Feb 3, 2026
Merged

OCPEDGE-2355: feat: skip unsupported commatrix tests for two node#30720
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
eggfoobar:skip-commatrix-twonode

Conversation

@eggfoobar
Copy link
Contributor

Adding simple skip for commatrix tests that were determined to not be supported on two node.

Original skip was reverted, #30670, this is a much simpler implementation that only targets two node.

/assign @aabughosh

@openshift-ci-robot
Copy link

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: automatic mode

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

openshift-ci-robot commented Jan 23, 2026

@eggfoobar: This pull request references OCPEDGE-2355 which is a valid jira issue.

Details

In response to this:

Adding simple skip for commatrix tests that were determined to not be supported on two node.

Original skip was reverted, #30670, this is a much simpler implementation that only targets two node.

/assign @aabughosh

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
Copy link

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-fips
/test e2e-aws-ovn-microshift
/test e2e-aws-ovn-microshift-serial
/test e2e-aws-ovn-serial-1of2
/test e2e-aws-ovn-serial-2of2
/test e2e-gcp-csi
/test e2e-gcp-ovn
/test e2e-gcp-ovn-upgrade
/test e2e-metal-ipi-ovn-ipv6
/test e2e-vsphere-ovn
/test e2e-vsphere-ovn-upi

@eggfoobar
Copy link
Contributor Author

/retest-required

1 similar comment
@eggfoobar
Copy link
Contributor Author

/retest-required

@eggfoobar eggfoobar force-pushed the skip-commatrix-twonode branch from 4a049c8 to ca130b5 Compare January 29, 2026 14:55
@openshift-ci-robot
Copy link

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-fips
/test e2e-aws-ovn-microshift
/test e2e-aws-ovn-microshift-serial
/test e2e-aws-ovn-serial-1of2
/test e2e-aws-ovn-serial-2of2
/test e2e-gcp-csi
/test e2e-gcp-ovn
/test e2e-gcp-ovn-upgrade
/test e2e-metal-ipi-ovn-ipv6
/test e2e-vsphere-ovn
/test e2e-vsphere-ovn-upi

@eggfoobar eggfoobar force-pushed the skip-commatrix-twonode branch from ca130b5 to 91b5baa Compare February 2, 2026 03:10
@openshift-ci-robot
Copy link

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-fips
/test e2e-aws-ovn-microshift
/test e2e-aws-ovn-microshift-serial
/test e2e-aws-ovn-serial-1of2
/test e2e-aws-ovn-serial-2of2
/test e2e-gcp-csi
/test e2e-gcp-ovn
/test e2e-gcp-ovn-upgrade
/test e2e-metal-ipi-ovn-ipv6
/test e2e-vsphere-ovn
/test e2e-vsphere-ovn-upi

@eggfoobar
Copy link
Contributor Author

/retest-required

@aabughosh
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 2, 2026
@eggfoobar
Copy link
Contributor Author

@kyrtapz Would you be able to approve this PR?

@eggfoobar
Copy link
Contributor Author

/payload-job periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-serial-techpreview

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 2, 2026

@eggfoobar: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-serial-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/b726e0d0-0078-11f1-936b-e089079830b0-0

Signed-off-by: ehila <ehila@redhat.com>
@eggfoobar eggfoobar force-pushed the skip-commatrix-twonode branch from 91b5baa to 17e6dd1 Compare February 3, 2026 05:01
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Feb 3, 2026
@eggfoobar
Copy link
Contributor Author

/verified by https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-origin-30720-nightly-4.22-e2e-metal-ovn-two-node-fencing-serial-techpreview/2018426725842554880

Did a force push to update the log statements so the formatting was more readable, otherwise this is good to go.

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Feb 3, 2026
@openshift-ci-robot
Copy link

@eggfoobar: This PR has been marked as verified by https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-origin-30720-nightly-4.22-e2e-metal-ovn-two-node-fencing-serial-techpreview/2018426725842554880.

Details

In response to this:

/verified by https://prow.ci.openshift.org/view/gs/test-platform-results/logs/openshift-origin-30720-nightly-4.22-e2e-metal-ovn-two-node-fencing-serial-techpreview/2018426725842554880

Did a force push to update the log statements so the formatting was more readable, otherwise this is good to go.

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
Copy link

Scheduling required tests:
/test e2e-aws-csi
/test e2e-aws-ovn-fips
/test e2e-aws-ovn-microshift
/test e2e-aws-ovn-microshift-serial
/test e2e-aws-ovn-serial-1of2
/test e2e-aws-ovn-serial-2of2
/test e2e-gcp-csi
/test e2e-gcp-ovn
/test e2e-gcp-ovn-upgrade
/test e2e-metal-ipi-ovn-ipv6
/test e2e-vsphere-ovn
/test e2e-vsphere-ovn-upi

@eggfoobar
Copy link
Contributor Author

/retest-required

@aabughosh
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 3, 2026
@dgoodwin
Copy link
Contributor

dgoodwin commented Feb 3, 2026

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aabughosh, dgoodwin, eggfoobar

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 Feb 3, 2026
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 2dfa4fd and 2 for PR HEAD 17e6dd1 in total

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 3, 2026

@eggfoobar: 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 2b2156b into openshift:main Feb 3, 2026
20 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. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments