Volg Forgejo-workflowruns per poging, niet runner-taken #2294

New issue

Closed

opened 2026-10-05 23:41:34 +00:00 by brenno · 0 comments

brenno commented

2026-10-05 23:41:34 +00:00

Owner

Aanleiding

De release van v0.6.14 stopte in fase 2 met de melding dat geen ci-image-scans-taak was aangemaakt. De dispatch had wel degelijk een workflowrun aangemaakt, maar die stond nog in de runnerwachtrij en had daarom nog geen entry in /actions/tasks.

Dezelfde aanname zit in de scans-imagewacht, de drie PR-poorten en de release-CI. Bovendien groepeert release_ci_snapshot taken van verschillende pogingen alleen op jobnaam. Daardoor kan een oude downstream-success samen met nieuwe upstream-success ten onrechte een complete groene poging vormen.

Bewijs

scripts/release_auto.sh:331-342 erkent zelf dat /actions/tasks alleen toegewezen runner-taken toont.
scripts/release_auto.sh:1400-1429 verklaart een niet-zichtbare queued run na circa 50 seconden ten onrechte niet aangemaakt.
scripts/release_auto.sh:1458-1528 gebruikt dezelfde taskwaarneming voor de PR-poorten.
scripts/release_auto.sh:325-358 en 1682-1684 mengen taken per naam zonder workflowrun-identiteit.
Bij v0.6.14 bestond workflowrun 9655 al; job 23603 had nog task_id 0 en werd later normaal uitgevoerd en groen.

Gewenst gedrag

Volg de Forgejo workflowrun met een vaste run-id/attempt-id vanaf de dispatch.
queued, waiting en nog niet aan een runner toegewezen tellen als bestaande actieve run.
Hervatten dispatcht nooit opnieuw zolang dezelfde run actief is.
Oude en nieuwe pogingen mogen nooit tot één snapshot worden samengevoegd.
De gewenste registrytag blijft de uiteindelijke naconditie voor het scans-image.
Netwerk/API-onbekend wordt als onbekend gemeld, niet als afwezig.

Regressietoets

Een gedragstest met eerst een queued run zonder runner-taak, daarna running en success plus een pullbare registrytag. Een tweede test combineert een oude complete run met een nieuwe gedeeltelijk zichtbare run en bewijst dat fase 3 niet start.

## Aanleiding De release van v0.6.14 stopte in fase 2 met de melding dat geen ci-image-scans-taak was aangemaakt. De dispatch had wel degelijk een workflowrun aangemaakt, maar die stond nog in de runnerwachtrij en had daarom nog geen entry in /actions/tasks. Dezelfde aanname zit in de scans-imagewacht, de drie PR-poorten en de release-CI. Bovendien groepeert release_ci_snapshot taken van verschillende pogingen alleen op jobnaam. Daardoor kan een oude downstream-success samen met nieuwe upstream-success ten onrechte een complete groene poging vormen. ## Bewijs - scripts/release_auto.sh:331-342 erkent zelf dat /actions/tasks alleen toegewezen runner-taken toont. - scripts/release_auto.sh:1400-1429 verklaart een niet-zichtbare queued run na circa 50 seconden ten onrechte niet aangemaakt. - scripts/release_auto.sh:1458-1528 gebruikt dezelfde taskwaarneming voor de PR-poorten. - scripts/release_auto.sh:325-358 en 1682-1684 mengen taken per naam zonder workflowrun-identiteit. - Bij v0.6.14 bestond workflowrun 9655 al; job 23603 had nog task_id 0 en werd later normaal uitgevoerd en groen. ## Gewenst gedrag - Volg de Forgejo workflowrun met een vaste run-id/attempt-id vanaf de dispatch. - queued, waiting en nog niet aan een runner toegewezen tellen als bestaande actieve run. - Hervatten dispatcht nooit opnieuw zolang dezelfde run actief is. - Oude en nieuwe pogingen mogen nooit tot één snapshot worden samengevoegd. - De gewenste registrytag blijft de uiteindelijke naconditie voor het scans-image. - Netwerk/API-onbekend wordt als onbekend gemeld, niet als afwezig. ## Regressietoets Een gedragstest met eerst een queued run zonder runner-taak, daarna running en success plus een pullbare registrytag. Een tweede test combineert een oude complete run met een nieuwe gedeeltelijk zichtbare run en bewijst dat fase 3 niet start.

brenno added the

release-blocker

bug

accepted

labels

2026-10-05 23:41:34 +00:00

brenno referenced this issue

2026-10-06 11:35:49 +00:00

Isoleer macOS-releasepoort van gewone main-concurrency #2321

brenno added the

in-progress

label