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

Add latency based routing #1732

Merged
merged 8 commits into from
Apr 9, 2024
Merged

Add latency based routing #1732

merged 8 commits into from
Apr 9, 2024

Conversation

pascal-fischer
Copy link
Contributor

Describe your changes

Now that we have the latency between peers available we can use this data to consider when choosing the beste route. This way the route with the routing peer with the lowes latency will be prefered over others with the same target network.

Issue ticket number and link

#1554

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

Copy link
Collaborator

@mlsmaycon mlsmaycon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pls see other comments

client/internal/routemanager/client.go Outdated Show resolved Hide resolved
lixmal
lixmal previously approved these changes Mar 25, 2024
@mlsmaycon mlsmaycon merged commit 3ed2f08 into main Apr 9, 2024
16 checks passed
@mlsmaycon mlsmaycon deleted the feature/latency-based-routing branch April 9, 2024 19:20
Foosec pushed a commit to Foosec/netbird that referenced this pull request May 8, 2024
Now that we have the latency between peers available we can use this data to consider when choosing the best route. This way the route with the routing peer with the lower latency will be preferred over others with the same target network.
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.

3 participants