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

✨ feat(route): allows duplicate routes across users #1

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

amitsingh21
Copy link

@amitsingh21 amitsingh21 commented Nov 16, 2023

Description

Headscale follows tailscale control server architecture where if more than one subnet router irrespective of the user it belongs to, tries to advertise the same subnet range, only one of them can become a primary router. This is problematic for rapyuta.io VPN use cases where multiple warehouses aka users/namespaces in control server terminology can use the same subnet ranges.

This PR tries to solve this use case by checking conflicting subnet ranges within a namespace/user scope instead of all the users/namespaces.

Wrike:

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

@pallabpain pallabpain force-pushed the feat/user-subnets branch 4 times, most recently from 5cb177e to 741eb57 Compare November 29, 2023 13:07
@pallabpain pallabpain removed the request for review from RomilShah November 29, 2023 13:21
@pallabpain pallabpain added the enhancement New feature or request label Nov 29, 2023
@pallabpain pallabpain changed the title feat(route): allows conflicting subnet ranges across users feat(route): allows duplicate routes across users Nov 29, 2023
@pallabpain pallabpain changed the title feat(route): allows duplicate routes across users ✨ feat(route): allows duplicate routes across users Nov 29, 2023
@pallabpain pallabpain force-pushed the feat/user-subnets branch 4 times, most recently from e7359f2 to 1be2446 Compare December 7, 2023 23:55
@pallabpain pallabpain merged commit ec93707 into rr-release-0.22.3 Dec 11, 2023
8 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants