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

ovn-k8s-overlay: Use the gateway router for North-South LB. #13

Merged
merged 1 commit into from
Sep 9, 2016
Merged

Conversation

shettyg
Copy link
Collaborator

@shettyg shettyg commented Sep 9, 2016

The load-balancer on the external switch does not work when
there is a SNAT rule in the gateway router.

Signed-off-by: Gurucharan Shetty guru@ovn.org

The load-balancer on the external switch does not work when
there is a SNAT rule in the gateway router.

Signed-off-by: Gurucharan Shetty <guru@ovn.org>
@salv-orlando
Copy link
Contributor

I guess this changes little to nothing in terms of packet flow, but fixes the issue you observed.
Is this a bug in OVN or is it just that the SNAT rule will mess up with the return traffic?

@shettyg
Copy link
Collaborator Author

shettyg commented Sep 9, 2016

The SNAT rule will mess up with the return traffic. And when it eventually comes to the place where load-balancing happened it sees a different packet and hence it will not unDNAT it. With this, since we load-balance in the router, the SNAT rule does not happen before unDNAT.

This support just got merged in OVN (i.e ability to add LB to gateway router with a fresh schema change)

@salv-orlando
Copy link
Contributor

cool. lgtm then.

does this also mean the issue preventing us from testing N/S connectivity is gone?

@shettyg
Copy link
Collaborator Author

shettyg commented Sep 9, 2016

does this also mean the issue preventing us from testing N/S connectivity is gone?

Yes.

@shettyg shettyg merged commit 902ae64 into ovn-kubernetes:master Sep 9, 2016
pecameron pushed a commit to pecameron/ovn-kubernetes that referenced this pull request Sep 12, 2019
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.

2 participants