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

Implement distance functionality from fcm #190

Open
Honigeintopf opened this issue Nov 5, 2024 · 2 comments · May be fixed by #191
Open

Implement distance functionality from fcm #190

Honigeintopf opened this issue Nov 5, 2024 · 2 comments · May be fixed by #191
Assignees

Comments

@Honigeintopf
Copy link

The FCM is able to define a distance to the firewalls it creates.
The firewall controller isn't able to do that yet, which this issue aims to improve.

@Honigeintopf Honigeintopf self-assigned this Nov 5, 2024
@Honigeintopf
Copy link
Author

Honigeintopf commented Nov 5, 2024

Right now, firewalls are chosen randomly to send network requests. This random selection sometimes causes one firewall to send a request, but a different firewall to receive the response. This mismatch causes dropped packets and breaks network communication.

What Needs to Be Done

  1. Get the Distance: The firewall controller already retrieves a distance setting from the Firewall CRDs (custom resources). This distance setting is already in place and does not need to be changed.
  2. Update Routing Paths: Inside the metal-networker code, there is a function to update routing paths in the FRR (Free Range Routing) configuration. This function needs to use the distance setting from the firewall controller to adjust the routing paths so that requests and responses go through the same firewall.

Solution Summary

  1. Use the distance value to extend routing paths in the FRR configuration.
  2. This will make sure that the same firewall handles both the request and response, preventing packet drops and improving network stability.

@Honigeintopf
Copy link
Author

Honigeintopf commented Nov 6, 2024

AS path prepend

Image

@Honigeintopf Honigeintopf linked a pull request Nov 6, 2024 that will close this issue
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 a pull request may close this issue.

1 participant