[PR #884] [MERGED] Integration test v2, hostname ping test #1705

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

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/884
Author: @kradalby
Created: 10/23/2022
Status: Merged
Merged: 10/23/2022
Merged by: @kradalby

Base: mainHead: integration-v2-ping-by-hostname


📝 Commits (8)

  • d242cea Make hostname dns safe, allow string in ping command
  • fa3d21c Rename pingall test to signal ip
  • d8144ee Add initial pingallbyhostname
  • cbbf9fb Use FQDN from tailscale client
  • d706c35 Remove 1.16 from FQDN, bump 1.32.1
  • 53b4bb2 Fixup after ts interface
  • 40c048f Fix lint
  • 7155b22 Factor out some commonly used patterns

📊 Changes

6 files changed (+156 additions, -39 deletions)

View changed files

📝 integration/control.go (+4 -2)
📝 integration/general_test.go (+67 -29)
📝 integration/hsic/hsic.go (+1 -1)
📝 integration/scenario.go (+68 -1)
📝 integration/tailscale.go (+2 -1)
📝 integration/tsic/tsic.go (+14 -5)

📄 Description

Signed-off-by: Kristoffer Dalby kristoffer@tailscale.com


🔄 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/884 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 10/23/2022 **Status:** ✅ Merged **Merged:** 10/23/2022 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `integration-v2-ping-by-hostname` --- ### 📝 Commits (8) - [`d242cea`](https://github.com/juanfont/headscale/commit/d242ceac46d1fd7a17de8785c22ee59344ff9a6c) Make hostname dns safe, allow string in ping command - [`fa3d21c`](https://github.com/juanfont/headscale/commit/fa3d21cbc07489aacad8bdb18ccd6a610b93a7db) Rename pingall test to signal ip - [`d8144ee`](https://github.com/juanfont/headscale/commit/d8144ee2ed9edef2a8396b4f44314c523ee289fd) Add initial pingallbyhostname - [`cbbf9fb`](https://github.com/juanfont/headscale/commit/cbbf9fbdefc2f94bbf6558d10f20a279b78ee2c0) Use FQDN from tailscale client - [`d706c35`](https://github.com/juanfont/headscale/commit/d706c3516dca8f65f837c88288e1e1cb8b668a5f) Remove 1.16 from FQDN, bump 1.32.1 - [`53b4bb2`](https://github.com/juanfont/headscale/commit/53b4bb220dd0cafc7f77cf1b3f41b57c8489881a) Fixup after ts interface - [`40c048f`](https://github.com/juanfont/headscale/commit/40c048fb45b490632a5be61cc7ddc61a97afc7ae) Fix lint - [`7155b22`](https://github.com/juanfont/headscale/commit/7155b22043ee830a845c91f1fc18ffb58bfcc447) Factor out some commonly used patterns ### 📊 Changes **6 files changed** (+156 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `integration/control.go` (+4 -2) 📝 `integration/general_test.go` (+67 -29) 📝 `integration/hsic/hsic.go` (+1 -1) 📝 `integration/scenario.go` (+68 -1) 📝 `integration/tailscale.go` (+2 -1) 📝 `integration/tsic/tsic.go` (+14 -5) </details> ### 📄 Description Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com> --- <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:31:21 +01:00
adam closed this issue 2025-12-29 02:31:21 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1705