Skip to content

(release/25.1) ci(workflows): retry the DragonFlyBSD VM step on transient boot flakes - #3939

Merged
metux merged 1 commit into
release/25.1from
wt/dfly-retry-251
Oct 13, 2026
Merged

metux merged 1 commit into
release/25.1from
wt/dfly-retry-251

Conversation

@metux

@metux metux commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Retry der DragonFlyBSD-VM-Lane bei transientem Boot-Flake.

xserver-build-dragonflybsd führte den vmactions-VM-Step exakt einmal aus.
Bootet die VM transient nicht hoch ("ssh: Could not resolve hostname
dragonflybsd"), ist die Lane rot und nichts retriert. Das ist ein reines
Infra-Problem (VM-Image/Boot), kein Code-Defekt.

Änderung: 3-Versuche-Retry mit continue-on-error auf Attempt 1+2; jeder
Attempt nutzt einen eigenen guest-Namen, damit ein von einem fehlgeschlagenen
Boot hinterlassenes Domain nicht mit dem nächsten Attempt kollidiert
("Guest name ... is already in use") — auf diesem Zweig gibt es keine
libvirt-cleanup-Action, daher die Kollisionsvermeidung über die Namen.

Wichtig: Die Retry-if:-Bedingungen nutzen !cancelled() statt der
impliziten success()-Statusfunktion — nach dem continue-on-error-Fehlschlag
von Attempt 1 ist success() false und würde die späteren Attempts still
überspringen (derselbe Fehler wie #3937 auf master).

Kein Code-/Build-Verhalten geändert — nur die Lane-Robustheit.

The DragonFlyBSD lane runs the vmactions VM step exactly once. When the VM
fails to boot/provision transiently ("ssh: Could not resolve hostname
dragonflybsd"), the lane is red and nothing retries it. Add a 3-attempt
retry with continue-on-error on attempts 1-2; each attempt uses its own
guest name so a leftover domain from a failed boot does not collide with
the next attempt.

The retry `if:` uses !cancelled() instead of the implicit success() status
function - after attempt 1 fails, success() is false and would silently
skip the later attempts.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
@metux
metux force-pushed the wt/dfly-retry-251 branch from d5b92a1 to 723abac Compare October 13, 2026 09:10
@metux
metux merged commit 9f167c3 into release/25.1 Oct 13, 2026
@metux
metux deleted the wt/dfly-retry-251 branch October 13, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant