Skip to content
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

Write an enhancement proposal for OVN InterConnect support #142

Closed
Tracked by #186
aswinsuryan opened this issue Nov 15, 2022 · 3 comments · Fixed by #163
Closed
Tracked by #186

Write an enhancement proposal for OVN InterConnect support #142

aswinsuryan opened this issue Nov 15, 2022 · 3 comments · Fixed by #163
Assignees

Comments

@aswinsuryan
Copy link
Contributor

aswinsuryan commented Nov 15, 2022

What would you like to be added:

Add support in Submariner to support OVN interconnect

Why is this needed:

The OVN is moving to a new architecture using an OVN interconnect. This would have an impact on Submariner. We need to test Submariner with the new implementation and make the necessary changes.

Enhancement PR: #163

@maayanf24
Copy link
Contributor

@aswinsuryan - Can you add sizing information and an enhancement proposal for this?

@aswinsuryan
Copy link
Contributor Author

@maayanf24 Yes I will work on it.

@aswinsuryan
Copy link
Contributor Author

aswinsuryan commented Jan 30, 2023

Was able to deploy OVN with IC enabled using this branch. Followed the instructions in the readme and some of the scripts were modified to rectify issues that we face when multiple clusters deployed. @sridhargaddam documented the changes to be made here

A cluster with a single zone was tried by running the above steps with these flags

export KIND_NUM_ADDITIONAL_ZONES=0
export KIND_NUM_WORKER=2
export OVN_INTERCONNECT_ENABLE=true

The submariner e2e failed since the OVN transit switch that connects two different zone uses the same CIDR as the Submariner Router.

After changing the IP in OVN to a different CIDR, 244.254.0.0/16(changed in OVN as it was easier to test), Submariner tests passed

So the Submariner router needs to use a different CIDR.

Now need to verify submariner tests with multiple zones.

@aswinsuryan aswinsuryan changed the title OVN InterConnect support Write an enhancement proposal for OVN InterConnect support Apr 27, 2023
@nyechiel nyechiel moved this to In Review in Submariner 0.16 May 4, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Submariner 0.16 Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants