-
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
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. |
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 |
|
@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 |
| "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: 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>
8c04bc5 to
e9ff88f
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 |
|
[REHEARSALNOTIFIER]
Interacting with pj-rehearseComment: Once you are satisfied with the results of the rehearsals, comment: |
|
@zhaozhanqi: all tests passed! 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. |
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: