[PR #636] [MERGED] Add system activity tracking and event store #12818

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/636
Author: @braginini
Created: 12/27/2022
Status: Merged
Merged: 1/2/2023
Merged by: @braginini

Base: mainHead: feature/event-store


📝 Commits (10+)

  • 1cfb7c4 Add system activity tracking and event store
  • ff65a51 Cleanup sqlite event store in tests
  • da95c7b Use NoopEventStore in tests
  • 6c0e160 Add GetEvents test to account manager
  • 97c24c8 Add GetEvents HTTP API test
  • a42c930 Add GetEvents HTTP API test
  • d423468 Add SetupKey event tests
  • 0d04e80 Fix golint
  • 18548d9 Add more check for setup key events
  • dad2bf2 Add peer and account events test

📊 Changes

42 files changed (+1826 additions, -226 deletions)

View changed files

📝 .github/workflows/golang-test-linux.yml (+2 -2)
📝 client/cmd/testutil.go (+7 -1)
📝 client/internal/engine_test.go (+7 -1)
📝 go.mod (+1 -0)
📝 go.sum (+2 -0)
📝 management/client/client_test.go (+4 -1)
📝 management/cmd/management.go (+8 -1)
📝 management/server/account.go (+54 -10)
📝 management/server/account_test.go (+110 -25)
management/server/activity/codes.go (+202 -0)
management/server/activity/event.go (+42 -0)
management/server/activity/sqlite/sqlite.go (+149 -0)
management/server/activity/sqlite/sqlite_test.go (+53 -0)
management/server/activity/store.go (+54 -0)
management/server/event.go (+33 -0)
management/server/event_test.go (+63 -0)
📝 management/server/group.go (+92 -4)
📝 management/server/grpcserver.go (+2 -8)
📝 management/server/http/api/openapi.yml (+145 -78)
📝 management/server/http/api/types.gen.go (+49 -0)

...and 22 more files

📄 Description

Describe your changes

Add basic system activity tracking and SQLite storage to store the events

Checklist

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

🔄 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/636 **Author:** [@braginini](https://github.com/braginini) **Created:** 12/27/2022 **Status:** ✅ Merged **Merged:** 1/2/2023 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `feature/event-store` --- ### 📝 Commits (10+) - [`1cfb7c4`](https://github.com/netbirdio/netbird/commit/1cfb7c42c600818b44448a6c33d9c7b3e4920e36) Add system activity tracking and event store - [`ff65a51`](https://github.com/netbirdio/netbird/commit/ff65a51941fa2a9af90fc2aac87627d7fac2011f) Cleanup sqlite event store in tests - [`da95c7b`](https://github.com/netbirdio/netbird/commit/da95c7b1eb387b0c64429317c6dc0e6d91ddd305) Use NoopEventStore in tests - [`6c0e160`](https://github.com/netbirdio/netbird/commit/6c0e16070ec9d151986d32a95fe83a60700da2a9) Add GetEvents test to account manager - [`97c24c8`](https://github.com/netbirdio/netbird/commit/97c24c813449ee1048eab1e547a9a335fb75a4f4) Add GetEvents HTTP API test - [`a42c930`](https://github.com/netbirdio/netbird/commit/a42c9303dbd082f204d1ba40f05c71fbce8ecf48) Add GetEvents HTTP API test - [`d423468`](https://github.com/netbirdio/netbird/commit/d423468110d7a7beb5b254848bb260406b64a83f) Add SetupKey event tests - [`0d04e80`](https://github.com/netbirdio/netbird/commit/0d04e80b6b792a08bc2a33ecf366625a94bd2086) Fix golint - [`18548d9`](https://github.com/netbirdio/netbird/commit/18548d910aa02a77a6160359502d6c65881fad42) Add more check for setup key events - [`dad2bf2`](https://github.com/netbirdio/netbird/commit/dad2bf2a0a3b3e73000c6fae5fee3a3597c1b2ac) Add peer and account events test ### 📊 Changes **42 files changed** (+1826 additions, -226 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golang-test-linux.yml` (+2 -2) 📝 `client/cmd/testutil.go` (+7 -1) 📝 `client/internal/engine_test.go` (+7 -1) 📝 `go.mod` (+1 -0) 📝 `go.sum` (+2 -0) 📝 `management/client/client_test.go` (+4 -1) 📝 `management/cmd/management.go` (+8 -1) 📝 `management/server/account.go` (+54 -10) 📝 `management/server/account_test.go` (+110 -25) ➕ `management/server/activity/codes.go` (+202 -0) ➕ `management/server/activity/event.go` (+42 -0) ➕ `management/server/activity/sqlite/sqlite.go` (+149 -0) ➕ `management/server/activity/sqlite/sqlite_test.go` (+53 -0) ➕ `management/server/activity/store.go` (+54 -0) ➕ `management/server/event.go` (+33 -0) ➕ `management/server/event_test.go` (+63 -0) 📝 `management/server/group.go` (+92 -4) 📝 `management/server/grpcserver.go` (+2 -8) 📝 `management/server/http/api/openapi.yml` (+145 -78) 📝 `management/server/http/api/types.gen.go` (+49 -0) _...and 22 more files_ </details> ### 📄 Description ## Describe your changes Add basic system activity tracking and SQLite storage to store the events ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <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:45 -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#12818