[PR #2799] [MERGED] integration: eventually fixups #2865

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

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/2799
Author: @kradalby
Created: 10/17/2025
Status: Merged
Merged: 10/17/2025
Merged by: @kradalby

Base: mainHead: kradalby/depupdate-027


📝 Commits (2)

  • afca4ce integration: wrap external calls in EventuallyWithT
  • 54f0620 integration: fix traceroute timeout in TestHASubnetRouterFailover

📊 Changes

2 files changed (+59 additions, -39 deletions)

View changed files

📝 integration/acl_test.go (+50 -30)
📝 integration/route_test.go (+9 -9)

📄 Description

harden some more tests for timing issues.


🔄 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/2799 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 10/17/2025 **Status:** ✅ Merged **Merged:** 10/17/2025 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `kradalby/depupdate-027` --- ### 📝 Commits (2) - [`afca4ce`](https://github.com/juanfont/headscale/commit/afca4ceac3714701458e685c2b0660d747f6e3d9) integration: wrap external calls in EventuallyWithT - [`54f0620`](https://github.com/juanfont/headscale/commit/54f0620f776871973d418deb69263738da8e91e7) integration: fix traceroute timeout in TestHASubnetRouterFailover ### 📊 Changes **2 files changed** (+59 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `integration/acl_test.go` (+50 -30) 📝 `integration/route_test.go` (+9 -9) </details> ### 📄 Description harden some more tests for timing issues. --- <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:26 +01:00
adam closed this issue 2025-12-29 04:19:26 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2865