[PR #612] [MERGED] Override config file via cli #1524

Closed
opened 2025-12-29 02:30:25 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/612
Author: @huskyii
Created: 6/5/2022
Status: Merged
Merged: 6/10/2022
Merged by: @kradalby

Base: mainHead: enhance_cli_config


📝 Commits (7)

  • 0363e58 cli.LoadConfig accepts config file now
  • 402a29e impl heascale -c to specify config file
  • ce13596 add integration test for headscale -c
  • 8744eee ExecuteCommand set HEADSCALE_LOG_LEVEL to disabled
  • 0c5a402 add changelog
  • bcb04d3 Merge branch 'main' into enhance_cli_config
  • fdefe46 Merge branch 'main' into enhance_cli_config

📊 Changes

10 files changed (+304 additions, -55 deletions)

View changed files

📝 CHANGELOG.md (+1 -0)
cmd/headscale/cli/dump_config.go (+28 -0)
📝 cmd/headscale/cli/root.go (+58 -0)
📝 cmd/headscale/headscale.go (+0 -38)
📝 cmd/headscale/headscale_test.go (+49 -4)
📝 config.go (+12 -13)
📝 integration_cli_test.go (+40 -0)
integration_test/etc/alt-config.dump.gold.yaml (+46 -0)
integration_test/etc/alt-config.yaml (+24 -0)
integration_test/etc/config.dump.gold.yaml (+46 -0)

📄 Description

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

🔄 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/juanfont/headscale/pull/612 **Author:** [@huskyii](https://github.com/huskyii) **Created:** 6/5/2022 **Status:** ✅ Merged **Merged:** 6/10/2022 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `enhance_cli_config` --- ### 📝 Commits (7) - [`0363e58`](https://github.com/juanfont/headscale/commit/0363e58467eafd067c705b1fa617186df3256381) cli.LoadConfig accepts config file now - [`402a29e`](https://github.com/juanfont/headscale/commit/402a29e50cfe7203e51b5365fea2031528f13ba7) impl heascale -c to specify config file - [`ce13596`](https://github.com/juanfont/headscale/commit/ce13596077ff19233655efc79c4097d710382803) add integration test for headscale -c - [`8744eee`](https://github.com/juanfont/headscale/commit/8744eeeb190ce1d91bd192f3224c997a47a386bb) ExecuteCommand set HEADSCALE_LOG_LEVEL to disabled - [`0c5a402`](https://github.com/juanfont/headscale/commit/0c5a402206657797389afe7b67b9ae5bff535803) add changelog - [`bcb04d3`](https://github.com/juanfont/headscale/commit/bcb04d38a57cd270843393cc93e4d54c9723e195) Merge branch 'main' into enhance_cli_config - [`fdefe46`](https://github.com/juanfont/headscale/commit/fdefe46c40dda5e8d34df003bde5ff3520cc5cbd) Merge branch 'main' into enhance_cli_config ### 📊 Changes **10 files changed** (+304 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -0) ➕ `cmd/headscale/cli/dump_config.go` (+28 -0) 📝 `cmd/headscale/cli/root.go` (+58 -0) 📝 `cmd/headscale/headscale.go` (+0 -38) 📝 `cmd/headscale/headscale_test.go` (+49 -4) 📝 `config.go` (+12 -13) 📝 `integration_cli_test.go` (+40 -0) ➕ `integration_test/etc/alt-config.dump.gold.yaml` (+46 -0) ➕ `integration_test/etc/alt-config.yaml` (+24 -0) ➕ `integration_test/etc/config.dump.gold.yaml` (+46 -0) </details> ### 📄 Description <!-- Please tick if the following things apply. You… --> - [x] read the [CONTRIBUTING guidelines](README.md#contributing) - [x] raised a GitHub issue or discussed it on the projects chat beforehand - [x] added unit tests - [x] added integration tests - [ ] updated documentation if needed - [x] updated CHANGELOG.md <!-- If applicable, please reference the issue using `Fixes #XXX` and add tests to cover your new code. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 02:30:25 +01:00
adam closed this issue 2025-12-29 02:30:25 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1524