[PR #891] [MERGED] Integration, remove retry #1713

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

📋 Pull Request Information

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

Base: mainHead: integration-ditch-retry


📝 Commits (2)

📊 Changes

5 files changed (+5 additions, -30 deletions)

View changed files

📝 .github/workflows/test-integration-cli.yml (+1 -6)
📝 .github/workflows/test-integration-derp.yml (+1 -6)
📝 .github/workflows/test-integration-general.yml (+1 -6)
📝 .github/workflows/test-integration-oidc.yml (+1 -6)
📝 .github/workflows/test-integration-v2-general.yml (+1 -6)

📄 Description

The retry has no real function as it will just fail on "container exists" on the old tests and the new test will just try forever before it eventually fails.

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/891 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 10/24/2022 **Status:** ✅ Merged **Merged:** 10/25/2022 **Merged by:** [@juanfont](https://github.com/juanfont) **Base:** `main` ← **Head:** `integration-ditch-retry` --- ### 📝 Commits (2) - [`23a0946`](https://github.com/juanfont/headscale/commit/23a0946e76e80b5674fd76096cf6e0edd9f00556) Integration, remove retry - [`2b10226`](https://github.com/juanfont/headscale/commit/2b10226618402d148f3e4aff6a2475ef6eb34213) Remove extra line ### 📊 Changes **5 files changed** (+5 additions, -30 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test-integration-cli.yml` (+1 -6) 📝 `.github/workflows/test-integration-derp.yml` (+1 -6) 📝 `.github/workflows/test-integration-general.yml` (+1 -6) 📝 `.github/workflows/test-integration-oidc.yml` (+1 -6) 📝 `.github/workflows/test-integration-v2-general.yml` (+1 -6) </details> ### 📄 Description The retry has no real function as it will just fail on "container exists" on the old tests and the new test will just try forever before it eventually fails. 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:24 +01:00
adam closed this issue 2025-12-29 02:31:24 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1713