Fix ovn-failover #2865
Merged
Fix ovn-failover #2865
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🤖 Created branch: z_pr2865/aswinsuryan/ovn-failover |
aswinsuryan
force-pushed
the
ovn-failover
branch
2 times, most recently
from
cd6e3db to
40fb429
Compare
aswinsuryan
requested review from
Oats87,
skitt,
sridhargaddam and
tpantelis
as code owners
sridhargaddam
requested changes
Jan 5, 2024
pkg/routeagent_driver/handlers/ovn/gateway_route_handler_test.go
Outdated
Show resolved
Hide resolved
pkg/routeagent_driver/handlers/ovn/non_gateway_route_handler.go
Outdated
Show resolved
Hide resolved
tpantelis
requested changes
Jan 5, 2024
pkg/routeagent_driver/handlers/ovn/non_gateway_route_handler.go
Outdated
Show resolved
Hide resolved
aswinsuryan
force-pushed
the
ovn-failover
branch
from
40fb429 to
d23bf64
Compare
tpantelis
approved these changes
Jan 5, 2024
pkg/routeagent_driver/handlers/ovn/non_gateway_route_handler.go
Outdated
Show resolved
Hide resolved
aswinsuryan
force-pushed
the
ovn-failover
branch
from
d23bf64 to
c08956b
Compare
In OVN-Kubernetes out of order event lead to stale gatewayroutes and nongatewayroutes. This lead to incorrect flow in datatpath. Now gatewayroutes and nongatewayroutes use cluster ID instead of endpoint name. This ensures that there is only one resource and it gets updated when remote endpoint is created and updated and preventing stale endpoints. The gatewayroutes only care about remoteCIDRs does not matter if the remote gateway switches. Signed-off-by: Aswin Suryanarayanan <[email protected]>
aswinsuryan
force-pushed
the
ovn-failover
branch
from
c08956b to
22c5f40
Compare
sridhargaddam
approved these changes
Jan 8, 2024
|
🤖 Closed branches: [z_pr2865/aswinsuryan/ovn-failover] |
aswinsuryan
added a commit
to aswinsuryan/submariner-website
that referenced
this pull request
Jan 10, 2024
Signed-off-by: Aswin Suryanarayanan <[email protected]>
tpantelis
pushed a commit
to submariner-io/submariner-website
that referenced
this pull request
Jan 11, 2024
Signed-off-by: Aswin Suryanarayanan <[email protected]>
tpantelis
pushed a commit
to tpantelis/submariner-website
that referenced
this pull request
Feb 27, 2024
Signed-off-by: Aswin Suryanarayanan <[email protected]>
tpantelis
pushed a commit
to submariner-io/submariner-website
that referenced
this pull request
Feb 29, 2024
Signed-off-by: Aswin Suryanarayanan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In OVN-Kubernetes out of order event leads to stale gatewayroutes and
nongatewayroutes. This leads to incorrect flow in datatpath. Now gatewayroutes
and nongatewayroutes use cluster ID instead of endpoint name. This ensures that
there is only one resource and it gets updated when remote endpoint is created
and updated and preventing stale endpoints. The gatewayroutes only care about
remoteCIDRs do not matter if the remote gateway switches.