[PR #6866] [MERGED] [management] component types #27329

Closed
opened 2026-08-05 07:08:34 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/6866
Author: @pascal-fischer
Created: 7/22/2026
Status: Merged
Merged: 7/23/2026
Merged by: @pascal-fischer

Base: mainHead: fix/component-types


📝 Commits (1)

📊 Changes

35 files changed (+611 additions, -428 deletions)

View changed files

📝 management/internals/shared/grpc/components_encoder.go (+11 -14)
📝 management/internals/shared/grpc/components_encoder_test.go (+50 -54)
📝 management/internals/shared/grpc/components_envelope_response_test.go (+5 -5)
📝 management/server/groups/manager.go (+7 -2)
📝 management/server/http/handlers/peers/peers_handler.go (+13 -8)
📝 management/server/networks/resources/types/resource.go (+22 -0)
📝 management/server/networks/routers/types/router.go (+31 -0)
📝 management/server/peer.go (+2 -2)
📝 management/server/peer/peer.go (+30 -0)
📝 management/server/peer_test.go (+2 -2)
📝 management/server/posture/nb_version.go (+2 -28)
📝 management/server/posture/nb_version_test.go (+0 -65)
📝 management/server/types/account.go (+8 -7)
📝 management/server/types/account_components.go (+32 -28)
📝 management/server/types/account_private_netmap_test.go (+1 -2)
📝 management/server/types/account_test.go (+1 -1)
📝 management/server/types/aliases.go (+16 -13)
📝 management/server/types/group.go (+28 -8)
📝 management/server/types/ipv6_endtoend_test.go (+2 -1)
📝 management/server/types/networkmap_components_test.go (+1 -1)

...and 15 more files

📄 Description

Describe your changes

Stack

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)
  • This change does not modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — OR I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See CONTRIBUTING.md.

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Improvements
    • Improved network map generation and synchronization for peers, routers, groups, and network resources.
    • Enhanced IPv4/IPv6 firewall and routing rule handling based on peer capabilities.
    • Improved accessible-peer results by using complete peer status and metadata.
    • Standardized minimum-version checks, including clearer handling of pre-release versions.
    • Improved event details when adding or removing resources from groups.

🔄 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/6866 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 7/22/2026 **Status:** ✅ Merged **Merged:** 7/23/2026 **Merged by:** [@pascal-fischer](https://github.com/pascal-fischer) **Base:** `main` ← **Head:** `fix/component-types` --- ### 📝 Commits (1) - [`eebafde`](https://github.com/netbirdio/netbird/commit/eebafde7842c55ba3d80502250d1295a3a0a080f) component types ### 📊 Changes **35 files changed** (+611 additions, -428 deletions) <details> <summary>View changed files</summary> 📝 `management/internals/shared/grpc/components_encoder.go` (+11 -14) 📝 `management/internals/shared/grpc/components_encoder_test.go` (+50 -54) 📝 `management/internals/shared/grpc/components_envelope_response_test.go` (+5 -5) 📝 `management/server/groups/manager.go` (+7 -2) 📝 `management/server/http/handlers/peers/peers_handler.go` (+13 -8) 📝 `management/server/networks/resources/types/resource.go` (+22 -0) 📝 `management/server/networks/routers/types/router.go` (+31 -0) 📝 `management/server/peer.go` (+2 -2) 📝 `management/server/peer/peer.go` (+30 -0) 📝 `management/server/peer_test.go` (+2 -2) 📝 `management/server/posture/nb_version.go` (+2 -28) 📝 `management/server/posture/nb_version_test.go` (+0 -65) 📝 `management/server/types/account.go` (+8 -7) 📝 `management/server/types/account_components.go` (+32 -28) 📝 `management/server/types/account_private_netmap_test.go` (+1 -2) 📝 `management/server/types/account_test.go` (+1 -1) 📝 `management/server/types/aliases.go` (+16 -13) 📝 `management/server/types/group.go` (+28 -8) 📝 `management/server/types/ipv6_endtoend_test.go` (+2 -1) 📝 `management/server/types/networkmap_components_test.go` (+1 -1) _...and 15 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [x] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] This change does **not** modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — **OR** I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See [CONTRIBUTING.md](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTING.md#discuss-changes-with-the-netbird-team-first). > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ <!-- codesmith:footer --> --- <a href="https://app.blacksmith.sh/netbirdio/codesmith/netbird/pr/6866"><picture><source media="(prefers-color-scheme: dark)" srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg"><source media="(prefers-color-scheme: light)" srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-light-v2.svg"><img alt="View with Codesmith" src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg"></picture></a> <a href="https://backend.blacksmith.sh/track/enable-autofix?expires=1787348392&installation_model_id=427504&pr_number=6866&repository=netbirdio%2Fnetbird&return_to=https%3A%2F%2Fgithub.com%2Fnetbirdio%2Fnetbird%2Fpull%2F6866&signature=ae8b97debc23a78ba86dd5a5ea9fc68851f62b72fb616504ff0fb1cfc79a5ab2"><picture><source media="(prefers-color-scheme: dark)" srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-light.svg"><img alt="Autofix with Codesmith" src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"></picture></a> <sup>Need help on this PR? Tag <code>/codesmith</code> with what you need. Autofix is disabled.</sup> <!-- codesmith:autofix:disabled --> <!-- /codesmith:footer --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Improvements** * Improved network map generation and synchronization for peers, routers, groups, and network resources. * Enhanced IPv4/IPv6 firewall and routing rule handling based on peer capabilities. * Improved accessible-peer results by using complete peer status and metadata. * Standardized minimum-version checks, including clearer handling of pre-release versions. * Improved event details when adding or removing resources from groups. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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 07:08:34 -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#27329