[PR #2649] [MERGED] cmd/hi: fixes and qol #2782

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

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/2649
Author: @kradalby
Created: 6/19/2025
Status: Merged
Merged: 6/23/2025
Merged by: @kradalby

Base: mainHead: kradalby/hi-fixes


📝 Commits (10+)

  • e1c033f cmd/hi: fix extration of test data
  • c2190f9 cmd/hi: fix cleanup
  • 08de6a5 flake: add hi as cli in dev shell
  • 8fae7ed .github/workflows: hi is now a command
  • 3c8e194 cmd/hi: improve reliability and remove backwards compatibility
  • 4205d3c cmd/hi: extract artifacts as directories instead of tar files
  • e822877 integration/hsic: extract artifacts as directories instead of tar files
  • 3401c84 cmd/hi: simplify artifact extraction since hsic now handles it
  • 73a368b integration/hsic: improve database extraction with better error handling
  • 9d20a70 integration/hsic: handle PostgreSQL tests properly in SaveDatabase

📊 Changes

31 files changed (+1079 additions, -293 deletions)

View changed files

📝 .github/workflows/build.yml (+10 -10)
📝 .github/workflows/check-tests.yaml (+4 -4)
📝 .github/workflows/docs-deploy.yml (+3 -3)
📝 .github/workflows/docs-test.yml (+3 -3)
📝 .github/workflows/gh-actions-updater.yaml (+2 -2)
📝 .github/workflows/lint.yml (+11 -11)
📝 .github/workflows/release.yml (+5 -5)
📝 .github/workflows/stale.yml (+1 -1)
📝 .github/workflows/test-integration.yaml (+9 -12)
📝 .github/workflows/test.yml (+4 -4)
📝 .github/workflows/update-flake.yml (+3 -3)
📝 Makefile (+108 -31)
📝 README.md (+15 -2)
📝 cmd/hi/cleanup.go (+73 -10)
📝 cmd/hi/docker.go (+353 -23)
📝 cmd/hi/doctor.go (+1 -3)
cmd/hi/tar_utils.go (+101 -0)
📝 flake.nix (+16 -1)
📝 go.mod (+27 -28)
📝 go.sum (+64 -53)

...and 11 more files

📄 Description

fixes some extraction of artefacts and makes it a dev shell command


🔄 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/2649 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 6/19/2025 **Status:** ✅ Merged **Merged:** 6/23/2025 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `kradalby/hi-fixes` --- ### 📝 Commits (10+) - [`e1c033f`](https://github.com/juanfont/headscale/commit/e1c033ff411e2fd20a83cc8aec2ed60b111983cd) cmd/hi: fix extration of test data - [`c2190f9`](https://github.com/juanfont/headscale/commit/c2190f9f9ac5abe20431c08ef72b581c0a9d1bd3) cmd/hi: fix cleanup - [`08de6a5`](https://github.com/juanfont/headscale/commit/08de6a50e71f5bf78f348866cade28e6963827b8) flake: add hi as cli in dev shell - [`8fae7ed`](https://github.com/juanfont/headscale/commit/8fae7edd60dda6ba7520eb608a3f713dd6a7d449) .github/workflows: hi is now a command - [`3c8e194`](https://github.com/juanfont/headscale/commit/3c8e194d8bead1a74f73e49f59d88e51328a0c3a) cmd/hi: improve reliability and remove backwards compatibility - [`4205d3c`](https://github.com/juanfont/headscale/commit/4205d3c1b0dff2e10a8af0a2db640aa7ffd7ba55) cmd/hi: extract artifacts as directories instead of tar files - [`e822877`](https://github.com/juanfont/headscale/commit/e822877215549aa3824052394bdcd4179e8a64f9) integration/hsic: extract artifacts as directories instead of tar files - [`3401c84`](https://github.com/juanfont/headscale/commit/3401c84c2cc6c197cf6fe2a7d13d13a55b4a0d25) cmd/hi: simplify artifact extraction since hsic now handles it - [`73a368b`](https://github.com/juanfont/headscale/commit/73a368b98c250be5126deae5d122d6c0dd893580) integration/hsic: improve database extraction with better error handling - [`9d20a70`](https://github.com/juanfont/headscale/commit/9d20a706722b1603c21d82e6c7263bb3a673af61) integration/hsic: handle PostgreSQL tests properly in SaveDatabase ### 📊 Changes **31 files changed** (+1079 additions, -293 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build.yml` (+10 -10) 📝 `.github/workflows/check-tests.yaml` (+4 -4) 📝 `.github/workflows/docs-deploy.yml` (+3 -3) 📝 `.github/workflows/docs-test.yml` (+3 -3) 📝 `.github/workflows/gh-actions-updater.yaml` (+2 -2) 📝 `.github/workflows/lint.yml` (+11 -11) 📝 `.github/workflows/release.yml` (+5 -5) 📝 `.github/workflows/stale.yml` (+1 -1) 📝 `.github/workflows/test-integration.yaml` (+9 -12) 📝 `.github/workflows/test.yml` (+4 -4) 📝 `.github/workflows/update-flake.yml` (+3 -3) 📝 `Makefile` (+108 -31) 📝 `README.md` (+15 -2) 📝 `cmd/hi/cleanup.go` (+73 -10) 📝 `cmd/hi/docker.go` (+353 -23) 📝 `cmd/hi/doctor.go` (+1 -3) ➕ `cmd/hi/tar_utils.go` (+101 -0) 📝 `flake.nix` (+16 -1) 📝 `go.mod` (+27 -28) 📝 `go.sum` (+64 -53) _...and 11 more files_ </details> ### 📄 Description fixes some extraction of artefacts and makes it a dev shell command --- <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:18:57 +01:00
adam closed this issue 2025-12-29 04:18:57 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2782