[PR #606] [MERGED] Add network routes distribution groups #12790

Closed
opened 2026-08-05 02:06:41 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/606
Author: @mlsmaycon
Created: 12/4/2022
Status: Merged
Merged: 12/6/2022
Merged by: @mlsmaycon

Base: mainHead: feature/RouteDistributionGroups


📝 Commits (7)

  • de2c7c5 Add network routes distribution groups
  • ca99702 add empty list and group id string tests
  • 83f7b0a sync routes to peers in the distribution groups
  • ada5243 Merge branch 'main' into feature/RouteDistributionGroups
  • 0bf32f2 fix note from review
  • 7d7e4be refactor get peers routes method and update doc
  • 55c7a40 remove libappindicator3-dev from the workflows

📊 Changes

16 files changed (+385 additions, -100 deletions)

View changed files

📝 .github/workflows/golang-test-linux.yml (+2 -2)
📝 .github/workflows/golangci-lint.yml (+1 -1)
📝 .github/workflows/release.yml (+1 -1)
📝 management/server/account.go (+41 -18)
📝 management/server/account_test.go (+8 -2)
📝 management/server/dns.go (+1 -9)
📝 management/server/file_store.go (+13 -0)
📝 management/server/http/api/openapi.yml (+7 -1)
📝 management/server/http/api/types.gen.go (+7 -0)
📝 management/server/http/routes.go (+13 -1)
📝 management/server/http/routes_test.go (+19 -10)
📝 management/server/mock_server/account_mock.go (+3 -3)
📝 management/server/peer.go (+9 -9)
📝 management/server/route.go (+22 -1)
📝 management/server/route_test.go (+214 -41)
📝 route/route.go (+24 -1)

📄 Description

Describe your changes

Updated tests, API, and account manager methods

Sync routes to peers in the distribution groups

Added store upgrade by adding the All group to routes that don't have them

Relates to #463

Checklist

  • Is a feature enhancement
  • Created tests that fail without the change (if possible)
  • Will update documentation in another PR

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbirdio/netbird/pull/606 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 12/4/2022 **Status:** ✅ Merged **Merged:** 12/6/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `feature/RouteDistributionGroups` --- ### 📝 Commits (7) - [`de2c7c5`](https://github.com/netbirdio/netbird/commit/de2c7c5a85616f5d560a514f7f217a3601415677) Add network routes distribution groups - [`ca99702`](https://github.com/netbirdio/netbird/commit/ca997028965fc6e0d36036bacd305fd40d2767e0) add empty list and group id string tests - [`83f7b0a`](https://github.com/netbirdio/netbird/commit/83f7b0a169d1f26b2ee0a499719cd33318b67f64) sync routes to peers in the distribution groups - [`ada5243`](https://github.com/netbirdio/netbird/commit/ada5243179c8329aa097801e05e3c5140a98d839) Merge branch 'main' into feature/RouteDistributionGroups - [`0bf32f2`](https://github.com/netbirdio/netbird/commit/0bf32f28772e84ec4aac530e8c1c81a870cb9cb0) fix note from review - [`7d7e4be`](https://github.com/netbirdio/netbird/commit/7d7e4be30cf64cfaf94a870efc1c919cec6eda9d) refactor get peers routes method and update doc - [`55c7a40`](https://github.com/netbirdio/netbird/commit/55c7a40a02cd77a6dd5c9642d92f10213f4335be) remove libappindicator3-dev from the workflows ### 📊 Changes **16 files changed** (+385 additions, -100 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golang-test-linux.yml` (+2 -2) 📝 `.github/workflows/golangci-lint.yml` (+1 -1) 📝 `.github/workflows/release.yml` (+1 -1) 📝 `management/server/account.go` (+41 -18) 📝 `management/server/account_test.go` (+8 -2) 📝 `management/server/dns.go` (+1 -9) 📝 `management/server/file_store.go` (+13 -0) 📝 `management/server/http/api/openapi.yml` (+7 -1) 📝 `management/server/http/api/types.gen.go` (+7 -0) 📝 `management/server/http/routes.go` (+13 -1) 📝 `management/server/http/routes_test.go` (+19 -10) 📝 `management/server/mock_server/account_mock.go` (+3 -3) 📝 `management/server/peer.go` (+9 -9) 📝 `management/server/route.go` (+22 -1) 📝 `management/server/route_test.go` (+214 -41) 📝 `route/route.go` (+24 -1) </details> ### 📄 Description ## Describe your changes Updated tests, API, and account manager methods Sync routes to peers in the distribution groups Added store upgrade by adding the All group to routes that don't have them Relates to #463 ### Checklist - [x] Is a feature enhancement - [x] Created tests that fail without the change (if possible) - [x] Will update documentation in another PR --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 02:06:41 -04:00
saavagebueno changed title from [PR #606] Add network routes distribution groups to [PR #606] [MERGED] Add network routes distribution groups 2026-08-05 03:07:03 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#12790