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

simplify punt fake gateway #626

Merged
merged 1 commit into from
Oct 5, 2023
Merged

simplify punt fake gateway #626

merged 1 commit into from
Oct 5, 2023

Conversation

sknat
Copy link
Collaborator

@sknat sknat commented Aug 11, 2023

This patch simplifies the way we handle the punt routes when routing for-me traffic VPP cannot handle to the host. Previously, we were using different fake gateways for ip4 & ip6. We now use the ip4 indirection in both cases.
We also generate different hardware addresses using the uplink order in the configuration.

Finally, we also make the isMain parameter public in the config.

@sknat sknat requested a review from hedibouattour August 11, 2023 17:00
@sknat sknat force-pushed the dev/nsk-simplify-punt branch 2 times, most recently from 4be13ac to 82817c4 Compare August 22, 2023 16:45
calico-vpp-agent/cni/cni_server.go Outdated Show resolved Hide resolved
This patch simplifies the way we handle the punt routes
when routing for-me traffic VPP cannot handle to the host.
Previously, we were using different fake gateways for ip4 & ip6.
We now use the ip4 indirection in both cases.
We also generate different hardware addresses using the uplink order
in the configuration.

Finally, we also make the isMain parameter public in the config.

Signed-off-by: Nathan Skrzypczak <[email protected]>
@sknat sknat force-pushed the dev/nsk-simplify-punt branch from 82817c4 to 56fab64 Compare October 4, 2023 16:20
@sknat sknat merged commit 45a7dae into master Oct 5, 2023
3 checks passed
@sknat sknat deleted the dev/nsk-simplify-punt branch October 5, 2023 17:00
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