[PR #2708] [MERGED] Add support to envsub go management configurations #18061

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2708
Author: @mlsmaycon
Created: 10/8/2024
Status: Merged
Merged: 10/9/2024
Merged by: @mlsmaycon

Base: mainHead: add-support-to-envsub-for-management-config-file


📝 Commits (2)

  • 8d866e8 Add support to envsub go management configurations
  • c28dd72 return error

📊 Changes

4 files changed (+362 additions, -111 deletions)

View changed files

📝 management/cmd/management.go (+1 -1)
📝 util/file.go (+53 -0)
util/file_suite_test.go (+126 -0)
📝 util/file_test.go (+182 -110)

📄 Description

Describe your changes

This change allows users to reference environment variables using Go template format, like {{ .EnvName }}

I also moved the previous file test code to file_suite_test.go. I will migrate these tests to another PR.

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/2708 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 10/8/2024 **Status:** ✅ Merged **Merged:** 10/9/2024 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `add-support-to-envsub-for-management-config-file` --- ### 📝 Commits (2) - [`8d866e8`](https://github.com/netbirdio/netbird/commit/8d866e87f7e5f3fa2ecb7c9e25ed2c947fa594fc) Add support to envsub go management configurations - [`c28dd72`](https://github.com/netbirdio/netbird/commit/c28dd724bfd3cd1e6823a5879b312bbe6fbd6283) return error ### 📊 Changes **4 files changed** (+362 additions, -111 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/management.go` (+1 -1) 📝 `util/file.go` (+53 -0) ➕ `util/file_suite_test.go` (+126 -0) 📝 `util/file_test.go` (+182 -110) </details> ### 📄 Description ## Describe your changes This change allows users to reference environment variables using Go template format, like `{{ .EnvName }}` I also moved the previous file test code to file_suite_test.go. I will migrate these tests to another PR. ## 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 04:08:04 -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#18061