[PR #6407] [infra] Unify self-hosted deployment paths in getting-started.sh #28165

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/6407
Author: @jnfrati
Created: 6/11/2026
Status: 🔄 Open

Base: mainHead: feat/getting-started-unified-wizard


📝 Commits (2)

  • 8e05834 [infra] Unify self-hosted deployment paths in getting-started.sh
  • f74d66c fix getting started to generate IdpManagerConfig on external IdP

📊 Changes

7 files changed (+2129 additions, -214 deletions)

View changed files

📝 .github/workflows/test-infrastructure-files.yml (+14 -0)
📝 infrastructure_files/configure.sh (+14 -0)
📝 infrastructure_files/getting-started-with-dex.sh (+9 -0)
📝 infrastructure_files/getting-started-with-zitadel.sh (+11 -0)
📝 infrastructure_files/getting-started.sh (+1750 -117)
📝 infrastructure_files/setup.env.example (+98 -97)
infrastructure_files/tests/test-render.sh (+233 -0)

📄 Description

Make getting-started.sh the single entry point for all self-hosted deployments. The wizard now asks two independent questions - identity provider (built-in vs standalone OIDC) and architecture (combined netbird-server vs separate management/signal/relay containers) - and renders the matching Docker Compose deployment with full reverse-proxy parity (built-in Traefik, external Traefik, Nginx, NPM, Caddy, manual).

Highlights:

  • setup.env contract: every wizard answer is persisted; --non-interactive re-renders idempotently from the file (IaC), --render-only generates without starting services. Secrets are generated once and appended so re-renders never rotate them.
  • Split architecture renders a modern management.json (embedded Dex or external OIDC via PKCE), drops coturn entirely (the relay container serves STUN via NB_ENABLE_STUN), and optionally adds a PostgreSQL container when the postgres engine is selected without a DSN.
  • The standalone-IdP path is framed around its real differentiator: multi-account support. The built-in IdP supports external SSO connectors but enforces single account mode.
  • configure.sh, getting-started-with-dex.sh and getting-started-with-zitadel.sh print deprecation banners; their templates are frozen pending removal.
  • New tests/test-render.sh validates all 8 combos (JSON validity, compose config, idempotent re-render, combined+external rejection) and runs in CI as the test-render-matrix job.

Describe your changes

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)
  • This change does not modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — OR I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See CONTRIBUTING.md.

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 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/6407 **Author:** [@jnfrati](https://github.com/jnfrati) **Created:** 6/11/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `feat/getting-started-unified-wizard` --- ### 📝 Commits (2) - [`8e05834`](https://github.com/netbirdio/netbird/commit/8e0583438414624a51add08378d0484cc6d003b5) [infra] Unify self-hosted deployment paths in getting-started.sh - [`f74d66c`](https://github.com/netbirdio/netbird/commit/f74d66c9ad3e235101dd56ccd33afa20e90a75a0) fix getting started to generate IdpManagerConfig on external IdP ### 📊 Changes **7 files changed** (+2129 additions, -214 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-infrastructure-files.yml` (+14 -0) 📝 `infrastructure_files/configure.sh` (+14 -0) 📝 `infrastructure_files/getting-started-with-dex.sh` (+9 -0) 📝 `infrastructure_files/getting-started-with-zitadel.sh` (+11 -0) 📝 `infrastructure_files/getting-started.sh` (+1750 -117) 📝 `infrastructure_files/setup.env.example` (+98 -97) ➕ `infrastructure_files/tests/test-render.sh` (+233 -0) </details> ### 📄 Description Make getting-started.sh the single entry point for all self-hosted deployments. The wizard now asks two independent questions - identity provider (built-in vs standalone OIDC) and architecture (combined netbird-server vs separate management/signal/relay containers) - and renders the matching Docker Compose deployment with full reverse-proxy parity (built-in Traefik, external Traefik, Nginx, NPM, Caddy, manual). Highlights: - setup.env contract: every wizard answer is persisted; --non-interactive re-renders idempotently from the file (IaC), --render-only generates without starting services. Secrets are generated once and appended so re-renders never rotate them. - Split architecture renders a modern management.json (embedded Dex or external OIDC via PKCE), drops coturn entirely (the relay container serves STUN via NB_ENABLE_STUN), and optionally adds a PostgreSQL container when the postgres engine is selected without a DSN. - The standalone-IdP path is framed around its real differentiator: multi-account support. The built-in IdP supports external SSO connectors but enforces single account mode. - configure.sh, getting-started-with-dex.sh and getting-started-with-zitadel.sh print deprecation banners; their templates are frozen pending removal. - New tests/test-render.sh validates all 8 combos (JSON validity, compose config, idempotent re-render, combined+external rejection) and runs in CI as the test-render-matrix job. ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-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) - [ ] This change does **not** modify the public API, gRPC protocols, functionality behavior, CLI / service flags, or introduce a new feature — **OR** I have discussed it with the NetBird team beforehand (link the issue / Slack thread in the description). See [CONTRIBUTING.md](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTING.md#discuss-changes-with-the-netbird-team-first). > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [ ] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <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 08:05:54 -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#28165