Skip to content

Add jobs for feature promotion of AWS Dual Stack#76769

Merged
openshift-merge-bot[bot] merged 2 commits intoopenshift:mainfrom
patrickdillon:dualstack-promotion-tests
Apr 8, 2026
Merged

Add jobs for feature promotion of AWS Dual Stack#76769
openshift-merge-bot[bot] merged 2 commits intoopenshift:mainfrom
patrickdillon:dualstack-promotion-tests

Conversation

@patrickdillon
Copy link
Copy Markdown
Contributor

Adds periodics for promotion of AWSDualStackInstall feature gate.

@openshift-ci openshift-ci bot requested review from neisw and sosiouxme March 24, 2026 16:59
@patrickdillon
Copy link
Copy Markdown
Contributor Author

patrickdillon commented Mar 24, 2026

We are still working on ingress support, so the IPv6 Primary test will fail and the IPv4 Primary will install (but without dual stack support--it's a false positive).

@patrickdillon
Copy link
Copy Markdown
Contributor Author

/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@patrickdillon
Copy link
Copy Markdown
Contributor Author

/hold

@openshift-ci openshift-ci bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Mar 24, 2026
@patrickdillon patrickdillon force-pushed the dualstack-promotion-tests branch from 5b80e6f to 3240320 Compare March 31, 2026 14:52
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 31, 2026
@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Mar 31, 2026

Don't we need similar changes to ci-operator/config/openshift/release/openshift-release-main__nightly-4.22.yaml and ci-operator/config/openshift/release/openshift-release-main__nightly-5.0.yaml?

@patrickdillon patrickdillon force-pushed the dualstack-promotion-tests branch from 3240320 to d641c86 Compare March 31, 2026 20:45
@patrickdillon
Copy link
Copy Markdown
Contributor Author

Don't we need similar changes to ci-operator/config/openshift/release/openshift-release-main__nightly-4.22.yaml and ci-operator/config/openshift/release/openshift-release-main__nightly-5.0.yaml?

Good catch!

moved it to 4.22, and the generation copied the config to 4.23--not sure we need it in 5.0

@patrickdillon
Copy link
Copy Markdown
Contributor Author

/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 1, 2026

/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@patrickdillon
Copy link
Copy Markdown
Contributor Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 1, 2026
@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 7, 2026

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 7, 2026
@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 7, 2026

/approve

The jobs are setup correctly. Some of these failures in the e2e tests are expected.

@sadasu
Copy link
Copy Markdown
Contributor

sadasu commented Apr 7, 2026

/pj-rehearse ack

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Apr 7, 2026
Copy link
Copy Markdown
Member

@tthvo tthvo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

too!

@neisw
Copy link
Copy Markdown
Contributor

neisw commented Apr 8, 2026

/approve

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 8, 2026
@neisw
Copy link
Copy Markdown
Contributor

neisw commented Apr 8, 2026

/hold
Likely need 5.0 jobs

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 8, 2026
@patrickdillon patrickdillon force-pushed the dualstack-promotion-tests branch from d641c86 to 1df1301 Compare April 8, 2026 13:02
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 8, 2026
@openshift-ci-robot openshift-ci-robot removed the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Apr 8, 2026
@patrickdillon
Copy link
Copy Markdown
Contributor Author

/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

@tthvo tthvo left a comment

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 Apr 8, 2026
The test fails because it compares the hostname to the FQDN of the
node name. This is an upstream test, which is known issue and skipped
in upstream tests:

kubernetes/kubernetes#121018
kubernetes/kubernetes#123255
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 8, 2026
@patrickdillon
Copy link
Copy Markdown
Contributor Author

Added new commit with test skip for the granular checks. This is a known upstream issue. Details are in the commit message.

/pj-rehearse

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

@patrickdillon, pj-rehearse: unable to determine affected jobs. This could be due to a branch that needs to be rebased. ERROR:

could not load configuration from base revision of release repo: could not checkout worktree: '[git checkout 147c93bc9f5b084f66c8d841aa0ab52d10218277]' failed with out:  and error exec: Stdout already set
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.

@neisw
Copy link
Copy Markdown
Contributor

neisw commented Apr 8, 2026

/lgtm

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

openshift-ci bot commented Apr 8, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: neisw, patrickdillon, sadasu, tthvo

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

@patrickdillon
Copy link
Copy Markdown
Contributor Author

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Apr 8, 2026
@patrickdillon
Copy link
Copy Markdown
Contributor Author

/pj-rehearse ack

@openshift-ci-robot
Copy link
Copy Markdown
Contributor

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

@openshift-ci-robot openshift-ci-robot added the rehearsals-ack Signifies that rehearsal jobs have been acknowledged label Apr 8, 2026
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Apr 8, 2026

@patrickdillon: 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-release-main-nightly-4.23-e2e-aws-ovn-dualstack-ipv6-primary-techpreview d641c86 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-dualstack-ipv6-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-dualstack-ipv4-primary-techpreview d641c86 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-dualstack-ipv4-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-dualstack-ipv6-primary-techpreview d641c86 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-dualstack-ipv6-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-dualstack-ipv4-primary-techpreview d641c86 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-dualstack-ipv4-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-installer-dualstack-ipv6-primary-techpreview c26cf98 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-installer-dualstack-ipv6-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-5.0-e2e-aws-ovn-installer-dualstack-ipv6-primary-techpreview c26cf98 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-5.0-e2e-aws-ovn-installer-dualstack-ipv6-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview c26cf98 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.22-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview c26cf98 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-4.23-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview
ci/rehearse/periodic-ci-openshift-release-main-nightly-5.0-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview c26cf98 link unknown /pj-rehearse periodic-ci-openshift-release-main-nightly-5.0-e2e-aws-ovn-installer-dualstack-ipv4-primary-techpreview

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 547b080 into openshift:main Apr 8, 2026
15 of 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. 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.

5 participants