[PR #329] [MERGED] Move flags declaration to root #12484

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/329
Author: @mlsmaycon
Created: 5/26/2022
Status: Merged
Merged: 5/26/2022
Merged by: @mlsmaycon

Base: mainHead: fix-management-file-migration


📝 Commits (1)

  • 47f813c Move flags declaration to root

📊 Changes

2 files changed (+71 additions, -54 deletions)

View changed files

📝 management/cmd/management.go (+37 -38)
📝 management/cmd/root.go (+34 -16)

📄 Description

When we move the files to the new path(docker-compose deployment), there was no Management Data Dir being set, causing an issue with the configuration file

This allows for mgmtDataDir and mgmtConfig to be initialized properly

use handleMigration function for copying files


🔄 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/329 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 5/26/2022 **Status:** ✅ Merged **Merged:** 5/26/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `fix-management-file-migration` --- ### 📝 Commits (1) - [`47f813c`](https://github.com/netbirdio/netbird/commit/47f813cbc12119f01a90099ce726fa4f881478d4) Move flags declaration to root ### 📊 Changes **2 files changed** (+71 additions, -54 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/management.go` (+37 -38) 📝 `management/cmd/root.go` (+34 -16) </details> ### 📄 Description When we move the files to the new path(docker-compose deployment), there was no Management Data Dir being set, causing an issue with the configuration file This allows for mgmtDataDir and mgmtConfig to be initialized properly use handleMigration function for copying files --- <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:00 -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#12484