Behandel een onzekere remote tagpush als point of no return #2297

New issue

Closed

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

brenno commented

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

Owner

Aanleiding

TAG_PUSHED wordt pas gezet nadat git push origin TAG en een aparte teruglezing met ls-remote beide slagen. Als de server de push accepteert maar het antwoord of de teruglezing wegvalt, bestaat de publieke tag al terwijl release_auto denkt dat nog niets onherroepelijks is gebeurd. De lokale releasebranch wordt dan opgeruimd en het advies is opnieuw beginnen in plaats van dezelfde tag hervatten.

Bewijs

scripts/release_auto.sh:138 laat die() altijd cleanup_branch aanroepen.
scripts/release_auto.sh:1643-1652 zet TAG_PUSHED pas na push plus teruglezing.
scripts/release_auto.sh:199-220 kiest hersteladvies en cleanup op basis van TAG_PUSHED.

Gewenst gedrag

Vanaf de eerste remote tagpushpoging is de toestand remote-onzeker en nooit meer veilig terugdraaibaar.
Bij ontbrekende bevestiging: geen branchcleanup en geen advies voor een verse release.
Herstel bestaat uit read-only statuscontrole en daarna uitsluitend --resume van dezelfde tag.
Een aantoonbaar afgewezen push vóór serveracceptatie mag apart worden behandeld.

Regressietoets

Simuleer een server die de tag opslaat waarna de clientverbinding faalt. De run moet de branch bewaren, dezelfde tag als herstelroute noemen en nooit een volgende release adviseren.

## Aanleiding TAG_PUSHED wordt pas gezet nadat git push origin TAG en een aparte teruglezing met ls-remote beide slagen. Als de server de push accepteert maar het antwoord of de teruglezing wegvalt, bestaat de publieke tag al terwijl release_auto denkt dat nog niets onherroepelijks is gebeurd. De lokale releasebranch wordt dan opgeruimd en het advies is opnieuw beginnen in plaats van dezelfde tag hervatten. ## Bewijs - scripts/release_auto.sh:138 laat die() altijd cleanup_branch aanroepen. - scripts/release_auto.sh:1643-1652 zet TAG_PUSHED pas na push plus teruglezing. - scripts/release_auto.sh:199-220 kiest hersteladvies en cleanup op basis van TAG_PUSHED. ## Gewenst gedrag - Vanaf de eerste remote tagpushpoging is de toestand remote-onzeker en nooit meer veilig terugdraaibaar. - Bij ontbrekende bevestiging: geen branchcleanup en geen advies voor een verse release. - Herstel bestaat uit read-only statuscontrole en daarna uitsluitend --resume van dezelfde tag. - Een aantoonbaar afgewezen push vóór serveracceptatie mag apart worden behandeld. ## Regressietoets Simuleer een server die de tag opslaat waarna de clientverbinding faalt. De run moet de branch bewaren, dezelfde tag als herstelroute noemen en nooit een volgende release adviseren.

brenno added the

release-blocker

bug

accepted

labels

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

brenno added the

in-progress

label

2026-10-07 04:22:58 +00:00

brenno referenced this issue from a commit

2026-10-07 04:53:10 +00:00