Skip to content

add ci/ovn-evpn-serial-metal-e2e#78585

Open
jechen0648 wants to merge 5 commits intoopenshift:mainfrom
jechen0648:ci/ovn-evpn-serial-metal-e2e
Open

add ci/ovn-evpn-serial-metal-e2e#78585
jechen0648 wants to merge 5 commits intoopenshift:mainfrom
jechen0648:ci/ovn-evpn-serial-metal-e2e

Conversation

@jechen0648
Copy link
Copy Markdown
Contributor

@jechen0648 jechen0648 commented Apr 29, 2026

Adds an optional, on-demand metal IPI presubmit for openshift/ovn-kubernetes that runs the dual-stack BGP local-GW workflow with the serial conformance suite, so EVPN disruptive / serial cases (e.g. from openshift/ovn-kubernetes#3161) can be exercised in CI when requested.

Branches
Ci-operator config and regenerated presubmits updated for:
master
release-4.22
release-4.23
release-5.0

how to trigger the job after this PR is merged:
/test e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial

Summary by CodeRabbit

  • Tests
    • Added new optional serial end-to-end test job for OVN-Kubernetes bare-metal deployments featuring dual-stack BGP configuration with local gateway and EVPN support. Test runs across master and multiple OpenShift release branches.

Add e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial using
baremetalds-e2e-ovn-bgp-dualstack-local-gw with openshift/conformance/serial,
TechPreviewNoUpgrade, and FRR 10.4.1 aligned with the techpreview lane.

always_run: false, optional: true, same TEST_SKIPS and doc-only skip as sibling jobs.

Signed-off-by: Jean Chen <jechen@redhat.com>
Mirror master e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial settings.

Signed-off-by: Jean Chen <jechen@redhat.com>
Same settings as master/release-4.22 (TechPreviewNoUpgrade, serial suite,
FRR 10.4.1, baremetalds-e2e-ovn-bgp-dualstack-local-gw).

Signed-off-by: Jean Chen <jechen@redhat.com>
Same settings as other branches (TechPreviewNoUpgrade, serial suite,
FRR 10.4.1, baremetalds-e2e-ovn-bgp-dualstack-local-gw).

Signed-off-by: Jean Chen <jechen@redhat.com>
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 29, 2026

@jechen0648: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test app-ci-config-dry
/test boskos-config
/test boskos-config-generation
/test build03-dry
/test build04-dry
/test build05-dry
/test build06-dry
/test build07-dry
/test build08-dry
/test build09-dry
/test build10-dry
/test build11-dry
/test check-gh-automation
/test check-gh-automation-tide
/test check-trigger-trusted-apps
/test ci-operator-config
/test ci-operator-config-metadata
/test ci-operator-registry
/test ci-secret-bootstrap-config-validation
/test ci-testgrid-allow-list
/test clusterimageset-validate
/test config
/test core-ci-config-dry
/test core-valid
/test generated-config
/test generated-dashboards
/test hosted-mgmt-dry
/test image-mirroring-config-validation
/test jira-lifecycle-config
/test labels
/test openshift-image-mirror-mappings
/test ordered-prow-config
/test owners
/test pr-reminder-config
/test prow-config
/test prow-config-filenames
/test prow-config-semantics
/test pylint
/test release-config
/test release-controller-config
/test rover-groups-config-validation
/test secret-generator-config-valid
/test services-valid
/test stackrox-stackrox-stackrox-stackrox-check
/test step-registry-metadata
/test step-registry-shellcheck
/test sync-rover-groups
/test verified-config
/test vsphere02-dry
/test yamllint

The following commands are available to trigger optional jobs:

/test build12-dry
/test check-cluster-profiles-config

Use /test all to run the following jobs that were automatically triggered:

pull-ci-openshift-release-check-gh-automation
pull-ci-openshift-release-main-ci-operator-config
pull-ci-openshift-release-main-ci-operator-config-metadata
pull-ci-openshift-release-main-ci-operator-registry
pull-ci-openshift-release-main-config
pull-ci-openshift-release-main-core-valid
pull-ci-openshift-release-main-generated-config
pull-ci-openshift-release-main-ordered-prow-config
pull-ci-openshift-release-main-owners
pull-ci-openshift-release-main-prow-config-filenames
pull-ci-openshift-release-main-prow-config-semantics
pull-ci-openshift-release-main-release-controller-config
pull-ci-openshift-release-openshift-image-mirror-mappings
pull-ci-openshift-release-yamllint
Details

In response to this:

Adds an optional, on-demand metal IPI presubmit for openshift/ovn-kubernetes that runs the dual-stack BGP local-GW workflow with the serial conformance suite, so EVPN disruptive / serial cases (e.g. from openshift/ovn-kubernetes#3161) can be exercised in CI when requested.

Branches
Ci-operator config and regenerated presubmits updated for:
master
release-4.22
release-4.23
release-5.0

how to trigger the job after this PR is merged:
/test e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial

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 openshift-ci Bot requested review from jcaamano and tssurya April 29, 2026 18:48
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 29, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jechen0648
Once this PR has been reviewed and has the lgtm label, please assign knobunc 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
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

Warning

Rate limit exceeded

@jechen0648 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 37 minutes and 52 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Central YAML (inherited)

Review profile: CHILL

Plan: Enterprise

Run ID: 4deddd33-ee0c-4e8f-8f05-c0e3233629e7

📥 Commits

Reviewing files that changed from the base of the PR and between 4ad5b69 and 9f33ade.

📒 Files selected for processing (4)
  • ci-operator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master.yaml
  • ci-operator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.22.yaml
  • ci-operator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-4.23.yaml
  • ci-operator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-release-5.0.yaml

Walkthrough

A new serial E2E test job for bare-metal OVN dual-stack BGP with local gateway and EVPN is added across four OpenShift OVN-Kubernetes configuration files. The job is configured as optional and non-always-run, targeting Equinix metal with specific environment variables and workflow parameters.

Changes

Cohort / File(s) Summary
OVN-Kubernetes Test Job Configuration
ci-operator/config/openshift/ovn-kubernetes/openshift-ovn-kubernetes-master.yaml, openshift-ovn-kubernetes-release-4.22.yaml, openshift-ovn-kubernetes-release-4.23.yaml, openshift-ovn-kubernetes-release-5.0.yaml
Added new test job e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial configured with TechPreviewNoUpgrade feature set, FRR image 10.4.1, serial conformance suite, intranet capability requirement, and baremetalds-e2e-ovn-bgp-dualstack-local-gw workflow execution on equinix-ocp-metal.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 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 pull request title 'add ci/ovn-evpn-serial-metal-e2e' accurately describes the main change: adding a new CI job configuration for OVN EVPN serial metal E2E testing across multiple release branches.
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 PR only modifies CI configuration YAML files and does not contain any Ginkgo test code changes.
Test Structure And Quality ✅ Passed The custom check for Test Structure and Quality is not applicable to this PR as it only modifies CI/CD configuration YAML files, not Ginkgo test implementations.
Microshift Test Compatibility ✅ Passed This PR does not add new Ginkgo e2e tests; changes are purely CI configuration in YAML files that define how existing test workflows should be scheduled and executed.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PR modifies only CI operator configuration files (YAML) and does not add any new Ginkgo e2e test code.
Topology-Aware Scheduling Compatibility ✅ Passed PR adds CI test job configurations, not deployment manifests or operator code that would affect cluster topology-aware scheduling.
Ote Binary Stdout Contract ✅ Passed PR modifies only YAML CI/CD configuration files, adding a new test job definition with no executable source code or process-level code.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed PR only modifies CI-operator YAML configuration files defining test job configs, not actual Ginkgo test code with It(), Describe(), Context(), When() statements.

✏️ 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
Review rate limit: 0/1 reviews remaining, refill in 37 minutes and 52 seconds.

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

…adata

determinize-ci-operator orders env keys; TEST_SUITE must follow the
multiline TEST_SKIPS entry.

Signed-off-by: Jean Chen <jechen@redhat.com>
@openshift-merge-bot
Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@jechen0648: 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-ovn-kubernetes-master-e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial openshift/ovn-kubernetes presubmit Presubmit changed
pull-ci-openshift-ovn-kubernetes-release-4.22-e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial openshift/ovn-kubernetes presubmit Presubmit changed
pull-ci-openshift-ovn-kubernetes-release-4.23-e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial openshift/ovn-kubernetes presubmit Presubmit changed
pull-ci-openshift-ovn-kubernetes-release-5.0-e2e-metal-ipi-ovn-dualstack-bgp-local-gw-evpn-serial openshift/ovn-kubernetes presubmit Presubmit 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.

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 29, 2026

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant