[PR #368] [MERGED] Add client status collection #12546

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/368
Author: @mlsmaycon
Created: 6/24/2022
Status: Merged
Merged: 7/2/2022
Merged by: @braginini

Base: mainHead: client-status


📝 Commits (10+)

  • f8d3267 Client Status package
  • 847bbc2 Merge branch 'main' into client-status
  • 1fdc0a0 Add status logic and tests
  • ff9e012 Use key string for peer maps and check previous existance
  • 72c08c3 Merge branch 'main' into client-status
  • b8590ee Update engine and peer package to use status recorder
  • fd899ad Add local peer state
  • 6a68b7c add WireguardModExists to windows and darwin
  • 95f5035 Compare to localPeer
  • a87a58b Minor comments corrections

📊 Changes

12 files changed (+524 additions, -53 deletions)

View changed files

📝 client/cmd/up.go (+2 -1)
📝 client/internal/connect.go (+55 -24)
📝 client/internal/engine.go (+29 -14)
📝 client/internal/engine_test.go (+5 -4)
📝 client/internal/peer/conn.go (+51 -3)
📝 client/internal/peer/conn_test.go (+6 -5)
📝 client/server/server.go (+13 -2)
client/status/status.go (+183 -0)
client/status/status_test.go (+169 -0)
📝 iface/iface_darwin.go (+5 -0)
📝 iface/iface_linux.go (+1 -0)
📝 iface/iface_windows.go (+5 -0)

📄 Description

Add client status recorder package.
We will update the peer's status on every state change.
Local peer, signal, and management states will be
updated when running or stopping the client.


🔄 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/368 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 6/24/2022 **Status:** ✅ Merged **Merged:** 7/2/2022 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `client-status` --- ### 📝 Commits (10+) - [`f8d3267`](https://github.com/netbirdio/netbird/commit/f8d3267db9a18b72024824bad3a963262876439f) Client Status package - [`847bbc2`](https://github.com/netbirdio/netbird/commit/847bbc22f9eaa3f7a31b2c6ddc9d5c6db3168142) Merge branch 'main' into client-status - [`1fdc0a0`](https://github.com/netbirdio/netbird/commit/1fdc0a00aa4efa1a20ab1881bec4cc2d08a9afec) Add status logic and tests - [`ff9e012`](https://github.com/netbirdio/netbird/commit/ff9e012d7569c65d9008ef774f0b5cd25fceacc1) Use key string for peer maps and check previous existance - [`72c08c3`](https://github.com/netbirdio/netbird/commit/72c08c39ca6fef241a09aac57d4d7b5e5846a220) Merge branch 'main' into client-status - [`b8590ee`](https://github.com/netbirdio/netbird/commit/b8590ee4dcdb9e0382f9f943da3784e3424fa3cc) Update engine and peer package to use status recorder - [`fd899ad`](https://github.com/netbirdio/netbird/commit/fd899ad4fec234e1ae6cc1cbca1e291838487c68) Add local peer state - [`6a68b7c`](https://github.com/netbirdio/netbird/commit/6a68b7c8a646cde43234bc4eca995b27486246cb) add WireguardModExists to windows and darwin - [`95f5035`](https://github.com/netbirdio/netbird/commit/95f5035da6f71099b978f0222644ca30535e5593) Compare to localPeer - [`a87a58b`](https://github.com/netbirdio/netbird/commit/a87a58b5de6f41c94c0bf435474ab6a9b5ffacd6) Minor comments corrections ### 📊 Changes **12 files changed** (+524 additions, -53 deletions) <details> <summary>View changed files</summary> 📝 `client/cmd/up.go` (+2 -1) 📝 `client/internal/connect.go` (+55 -24) 📝 `client/internal/engine.go` (+29 -14) 📝 `client/internal/engine_test.go` (+5 -4) 📝 `client/internal/peer/conn.go` (+51 -3) 📝 `client/internal/peer/conn_test.go` (+6 -5) 📝 `client/server/server.go` (+13 -2) ➕ `client/status/status.go` (+183 -0) ➕ `client/status/status_test.go` (+169 -0) 📝 `iface/iface_darwin.go` (+5 -0) 📝 `iface/iface_linux.go` (+1 -0) 📝 `iface/iface_windows.go` (+5 -0) </details> ### 📄 Description Add client status recorder package. We will update the peer's status on every state change. Local peer, signal, and management states will be updated when running or stopping the client. --- <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:09 -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#12546