[PR #1939] [MERGED] Implement PostgreSQL store #14478

Closed
opened 2026-08-05 03:05:42 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1939
Author: @bcmmbaga
Created: 5/7/2024
Status: Merged
Merged: 5/16/2024
Merged by: @bcmmbaga

Base: mainHead: implement-postgres-store


📝 Commits (10+)

📊 Changes

22 files changed (+723 additions, -148 deletions)

View changed files

📝 .github/workflows/golang-test-linux.yml (+1 -1)
📝 client/cmd/testutil.go (+3 -1)
📝 client/internal/engine_test.go (+1 -1)
📝 client/server/server_test.go (+2 -1)
📝 go.mod (+31 -5)
📝 go.sum (+98 -7)
📝 management/client/client_test.go (+2 -1)
📝 management/cmd/migration_down.go (+6 -5)
📝 management/server/account_test.go (+14 -2)
📝 management/server/dns_test.go (+3 -2)
📝 management/server/file_store.go (+3 -3)
📝 management/server/file_store_test.go (+6 -0)
📝 management/server/management_proto_test.go (+3 -1)
📝 management/server/management_test.go (+2 -1)
📝 management/server/nameserver_test.go (+2 -1)
📝 management/server/policy.go (+1 -1)
📝 management/server/posture_checks_test.go (+2 -1)
📝 management/server/route_test.go (+2 -1)
📝 management/server/sql_store.go (+106 -101)
📝 management/server/sql_store_test.go (+298 -2)

...and 2 more files

📄 Description

Describe your changes

Add experimental support for PostgreSQL as store engine.

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)
  • 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/1939 **Author:** [@bcmmbaga](https://github.com/bcmmbaga) **Created:** 5/7/2024 **Status:** ✅ Merged **Merged:** 5/16/2024 **Merged by:** [@bcmmbaga](https://github.com/bcmmbaga) **Base:** `main` ← **Head:** `implement-postgres-store` --- ### 📝 Commits (10+) - [`e6a25d1`](https://github.com/netbirdio/netbird/commit/e6a25d13f1c828794695fb5da6cafc9b01042449) migrate sqlite store to - [`6b2684a`](https://github.com/netbirdio/netbird/commit/6b2684a31d65ba16d493b793feb89ec11cd08715) Merge branch 'refs/heads/main' into implement-postgres-store - [`66a7108`](https://github.com/netbirdio/netbird/commit/66a71089ffe77c5545a4bb7d87e6d1a8c656f051) fix conflicts - [`da1d162`](https://github.com/netbirdio/netbird/commit/da1d16291d9550452afae1ebf8fdae1d1c8e71cc) init postgres store - [`6cea7cb`](https://github.com/netbirdio/netbird/commit/6cea7cbedfd1050ea36cb16d19a36ad511e927d6) Add postgres store tests - [`a64ac87`](https://github.com/netbirdio/netbird/commit/a64ac87311e3ff2032c73c177683def8f359057e) Refactor postgres store engine name - [`c9c887c`](https://github.com/netbirdio/netbird/commit/c9c887cc0f61c1b2a3d735efdcf2702ef26ded6d) fix tests - [`606a7e7`](https://github.com/netbirdio/netbird/commit/606a7e7c375f73f41ead1b59cdf43aa64fa6d348) Run postgres store tests on linux only - [`29825d6`](https://github.com/netbirdio/netbird/commit/29825d69285dc0d0700d0db2214f49183b7cd384) fix tests - [`0478811`](https://github.com/netbirdio/netbird/commit/0478811416deb553f51d3d8a7b0580d1c56670af) Refactor ### 📊 Changes **22 files changed** (+723 additions, -148 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/golang-test-linux.yml` (+1 -1) 📝 `client/cmd/testutil.go` (+3 -1) 📝 `client/internal/engine_test.go` (+1 -1) 📝 `client/server/server_test.go` (+2 -1) 📝 `go.mod` (+31 -5) 📝 `go.sum` (+98 -7) 📝 `management/client/client_test.go` (+2 -1) 📝 `management/cmd/migration_down.go` (+6 -5) 📝 `management/server/account_test.go` (+14 -2) 📝 `management/server/dns_test.go` (+3 -2) 📝 `management/server/file_store.go` (+3 -3) 📝 `management/server/file_store_test.go` (+6 -0) 📝 `management/server/management_proto_test.go` (+3 -1) 📝 `management/server/management_test.go` (+2 -1) 📝 `management/server/nameserver_test.go` (+2 -1) 📝 `management/server/policy.go` (+1 -1) 📝 `management/server/posture_checks_test.go` (+2 -1) 📝 `management/server/route_test.go` (+2 -1) 📝 `management/server/sql_store.go` (+106 -101) 📝 `management/server/sql_store_test.go` (+298 -2) _...and 2 more files_ </details> ### 📄 Description ## Describe your changes Add experimental support for PostgreSQL as store engine. ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [ ] It is a refactor - [ ] 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 03:05:42 -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#14478