[PR #1235] [MERGED] feat: organizing infrastructure_files folder and adds new envs #15092

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1235
Author: @diegocn
Created: 10/20/2023
Status: Merged
Merged: 12/17/2023
Merged by: @mlsmaycon

Base: mainHead: main


📝 Commits (4)

  • d672b03 organizing infrastructure_files folder
  • b1a4a19 fix: update test working dorectory for artifacts
  • e3fb126 fix: update test working directory for docker compose run step
  • 85307ea fix: update test working directory for docker compose run step

📊 Changes

9 files changed (+71 additions, -40 deletions)

View changed files

📝 .github/workflows/test-infrastructure-files.yml (+6 -6)
📝 .gitignore (+5 -5)
infrastructure_files/artifacts/.gitkeep (+0 -0)
📝 infrastructure_files/base.setup.env (+17 -1)
📝 infrastructure_files/configure.sh (+18 -15)
📝 infrastructure_files/docker-compose.yml.tmpl (+5 -5)
📝 infrastructure_files/docker-compose.yml.tmpl.traefik (+5 -5)
📝 infrastructure_files/management.json.tmpl (+2 -2)
📝 infrastructure_files/setup.env.example (+13 -1)

📄 Description

Describe your changes

This PR aims to organize a little the files within infrastructure_files folder and adds some new ENV vars to the process.

  1. It creates the artifacts folder within the infrastructure_files folder, the idea behind it is to split templates from artifacts created after running ./configure.sh. It makes it easier to cp/rsync only artifacts content to the final server/destination.

  2. Creates NETBIRD_TURN_DOMAIN and TURN_DOMAIN ENV vars. The idea behind it is to make it possible to split the management/signal server from TURN server. If NETBIRD_TURN_DOMAIN is not set, then, TURN_DOMAIN will be set as NETBIRD_DOMAIN.

  3. Creates *_TAG ENVs for each component. The idea behind it is to give the users the choice to use latest tag as default or tie it to specific versions of each component in the stack.

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/1235 **Author:** [@diegocn](https://github.com/diegocn) **Created:** 10/20/2023 **Status:** ✅ Merged **Merged:** 12/17/2023 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (4) - [`d672b03`](https://github.com/netbirdio/netbird/commit/d672b0313a586f4a3e4d8c2fe761a9f0ffe5360c) organizing infrastructure_files folder - [`b1a4a19`](https://github.com/netbirdio/netbird/commit/b1a4a19cf68e829cdfc66b0fe34499802b3ed23c) fix: update test working dorectory for artifacts - [`e3fb126`](https://github.com/netbirdio/netbird/commit/e3fb126a8b8b374799a49a381098d8e8193b72d8) fix: update test working directory for docker compose run step - [`85307ea`](https://github.com/netbirdio/netbird/commit/85307ea48c833f99d22c451f7a67f6e0ffe8058c) fix: update test working directory for docker compose run step ### 📊 Changes **9 files changed** (+71 additions, -40 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-infrastructure-files.yml` (+6 -6) 📝 `.gitignore` (+5 -5) ➕ `infrastructure_files/artifacts/.gitkeep` (+0 -0) 📝 `infrastructure_files/base.setup.env` (+17 -1) 📝 `infrastructure_files/configure.sh` (+18 -15) 📝 `infrastructure_files/docker-compose.yml.tmpl` (+5 -5) 📝 `infrastructure_files/docker-compose.yml.tmpl.traefik` (+5 -5) 📝 `infrastructure_files/management.json.tmpl` (+2 -2) 📝 `infrastructure_files/setup.env.example` (+13 -1) </details> ### 📄 Description ## Describe your changes This PR aims to organize a little the files within `infrastructure_files` folder and adds some new ENV vars to the process. 1. It creates the `artifacts` folder within the `infrastructure_files` folder, the idea behind it is to split templates from artifacts created after running `./configure.sh`. It makes it easier to cp/rsync only `artifacts` content to the final server/destination. 2. Creates `NETBIRD_TURN_DOMAIN` and `TURN_DOMAIN` ENV vars. The idea behind it is to make it possible to split the management/signal server from TURN server. If `NETBIRD_TURN_DOMAIN` is not set, then, `TURN_DOMAIN` will be set as `NETBIRD_DOMAIN`. 3. Creates `*_TAG` ENVs for each component. The idea behind it is to give the users the choice to use `latest` tag as default or tie it to specific versions of each component in the stack. ## 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:06:57 -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#15092