[PR #2686] [CLOSED] cmd/hi: prevent concurrent runs and add force flag #2801

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

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/2686
Author: @kradalby
Created: 7/14/2025
Status: Closed

Base: mainHead: kradalby/hi-only-run-once


📝 Commits (3)

  • 9603c36 cmd/hi: prevent concurrent runs and add force flag
  • e51d319 cmd/hi: handle multiple runs
  • 615c8c0 *ignore: add node stuff

📊 Changes

3 files changed (+61 additions, -0 deletions)

View changed files

📝 .dockerignore (+4 -0)
📝 .gitignore (+4 -0)
📝 cmd/hi/run.go (+53 -0)

📄 Description

Add mechanism to check for active test runs and prevent concurrent execution with helpful error message. Add --force flag to kill existing containers and force new run.

Fixes duplicate run conflicts in integration testing.


🔄 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/2686 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 7/14/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `kradalby/hi-only-run-once` --- ### 📝 Commits (3) - [`9603c36`](https://github.com/juanfont/headscale/commit/9603c36616e6771a0da9113e5d5674771f957b61) cmd/hi: prevent concurrent runs and add force flag - [`e51d319`](https://github.com/juanfont/headscale/commit/e51d319d0b64ba3e48a6aa70f284d944890b7fed) cmd/hi: handle multiple runs - [`615c8c0`](https://github.com/juanfont/headscale/commit/615c8c04201c3126b8c3b9338b9668386083f648) *ignore: add node stuff ### 📊 Changes **3 files changed** (+61 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.dockerignore` (+4 -0) 📝 `.gitignore` (+4 -0) 📝 `cmd/hi/run.go` (+53 -0) </details> ### 📄 Description Add mechanism to check for active test runs and prevent concurrent execution with helpful error message. Add --force flag to kill existing containers and force new run. Fixes duplicate run conflicts in integration testing. --- <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 04:19:02 +01:00
adam closed this issue 2025-12-29 04:19:02 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2801