-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add OVN BGP NoOverlay workflow for baremetal clusters #73335
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Add OVN BGP NoOverlay workflow for baremetal clusters #73335
Conversation
bf16951 to
93ee080
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: zhaozhanqi The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
1a93f8f to
9ea69da
Compare
|
/payload-job-with-prs pull-ci-openshift-cluster-network-operator-master-e2e-metal-ipi-ovn-dualstack-bgp-lgw-no-overlay-techpreview openshift/api#2537,openshift/cluster-network-operator#2844,openshift/ovn-kubernetes#2878,openshift/client-go#349 |
|
@zhaozhanqi: it appears that you have attempted to use some version of the payload command, but your comment was incorrectly formatted and cannot be acted upon. See the docs for usage info. |
This PR adds a new workflow 'baremetalds-e2e-ovn-bgp-dualstack-local-gw-no-overlay' based on the existing 'baremetalds-e2e-ovn-bgp-dualstack-local-gw' workflow with the following key differences: 1. Sets up FRR route reflector container on the packet server before cluster installation 2. Creates network operator manifest with NoOverlay transport configuration 3. Configures FRRConfiguration to peer with external route reflector 4. Adds RouteAdvertisements manifest for PodNetwork 5. Reorders pre-steps to run baremetalds-e2e-ovn-bgp-pre before baremetalds-ofcir-pre New components: - baremetalds-e2e-ovn-bgp-conf-no-overlay ref step - baremetalds-e2e-ovn-bgp-dualstack-local-gw-no-overlay workflow Network operator configuration: - defaultNetworkTransport: NoOverlay - routing: Unmanaged - outboundSNAT: Enabled - additionalRoutingCapabilities: FRR - routeAdvertisements: Enabled Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: zhaozhanqi <zzhao@redhat.com>
9ea69da to
8c04bc5
Compare
|
/payload-job-with-prs periodic-ci-openshift-cluster-network-operator-master-e2e-metal-ipi-ovn-dualstack-bgp-lgw-no-overlay-test openshift/cluster-network-operator#2844 openshift/ovn-kubernetes#2878 openshift/client-go#349 |
|
@zhaozhanqi: the repo openshift/release does not contribute to the OpenShift official images, or the base branch is not currently having images promoted |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@zhaozhanqi: This PR was included in a payload test run from openshift/cluster-network-operator#2844 |
|
@zhaozhanqi: This PR was included in a payload test run from openshift/ovn-kubernetes#2920 |
|
@zhaozhanqi: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
| "core-networking-reviewers" | ||
| ] | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this extra character
This PR adds a new workflow 'baremetalds-e2e-ovn-bgp-dualstack-local-gw-no-overlay' based on the existing 'baremetalds-e2e-ovn-bgp-dualstack-local-gw' workflow with the following key differences:
New components:
Network operator configuration: