From ba6cd618ba4ec2a5aa3a9b8f279b6ae62b29cd2d Mon Sep 17 00:00:00 2001 From: Diego Braga Date: Fri, 25 Sep 2026 20:14:16 +0200 Subject: [PATCH] =?UTF-8?q?chore(deps):=20provider-runtime=20v1.4.0=20?= =?UTF-8?q?=E2=80=94=20recover=20interrupted=20creates=20instead=20of=20re?= =?UTF-8?q?fusing=20forever=20(#22)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit v1.4.0 gives an unresolved create handshake a bounded path back. Previously, an interrupted create whose external resource Observe could not find was terminal: the reconciler refused with Requeue:false and nothing ever resolved the ambiguity, so the only documented remedy was a human deleting krateo.io/external-create-pending by hand. Six Repo CRs have been in exactly that state on krateo-057 since 2026-09-21, wedged by the go-git transport race that 0.15.4 fixed. That fix stopped NEW wedges; it could not recover the existing ones. This bump is what recovers them. The runtime now confirms the negative rather than holding on the ambiguity: within a 2-minute grace it re-observes without recreating, so a create that did land is adopted rather than duplicated; once the grace has fully elapsed with the resource still absent, the create did not land and the pending marker is cleared so the normal create path runs again. All six markers are days old, so they resolve on the first reconcile after rollout. No git-provider code change is needed. An earlier reading of #22 proposed making Observe consult the forge when the external name is empty; that was wrong. Returning ResourceExists:false there is CORRECT — it is how a new Repo gets created — and changing it could stop Create ever running. The recovery is time-based precisely because the local signal cannot be made meaningful. Verified: unit suite green (9 packages, -race). Integration tests need a Docker daemon and did not run locally; CI covers them. Co-Authored-By: Claude Opus 5 (1M context) --- go.mod | 2 +- go.sum | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/go.mod b/go.mod index 2b9fb84..37f2b89 100644 --- a/go.mod +++ b/go.mod @@ -11,7 +11,7 @@ require ( github.com/go-git/go-git/v5 v5.19.2 github.com/go-logr/logr v1.4.3 github.com/krateo-platformops/plumbing v1.14.3 - github.com/krateo-platformops/provider-runtime v1.3.1 + github.com/krateo-platformops/provider-runtime v1.4.0 github.com/moby/moby/api v1.52.0 github.com/moby/moby/client v0.1.0 github.com/pkg/errors v0.9.1 diff --git a/go.sum b/go.sum index fd66876..1cf859d 100644 --- a/go.sum +++ b/go.sum @@ -149,8 +149,8 @@ github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY= github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE= github.com/krateo-platformops/plumbing v1.14.3 h1:g60QTgbhIsZVTFq9JZveNx2QZauVyFeWHUAVxahxKRU= github.com/krateo-platformops/plumbing v1.14.3/go.mod h1:JMHwVbS2vdA/OSm3Gi3QCDTntFjPmrGtqZ6LQuhciZQ= -github.com/krateo-platformops/provider-runtime v1.3.1 h1:X9kbeePcZEpMXQks69VOn+i3ZMWPT380sRF9Nf1MAAU= -github.com/krateo-platformops/provider-runtime v1.3.1/go.mod h1:m/nv2S97NIkQNZ/SOWRn8/wStU4vlU4L8OwZC7xBtoQ= +github.com/krateo-platformops/provider-runtime v1.4.0 h1:V6RxporZtVdjix/ICN/9g1znSbf5FWb9vgFBL4LofII= +github.com/krateo-platformops/provider-runtime v1.4.0/go.mod h1:m/nv2S97NIkQNZ/SOWRn8/wStU4vlU4L8OwZC7xBtoQ= github.com/kylelemons/godebug v1.1.0 h1:RPNrshWIDI6G2gRW9EHilWtl7Z6Sb1BR0xunSBf0SNc= github.com/kylelemons/godebug v1.1.0/go.mod h1:9/0rRGxNHcop5bhtWyNeEfOS8JIWk580+fNqagV/RAw= github.com/mailru/easyjson v0.9.0 h1:PrnmzHw7262yW8sTBwxi1PdJA3Iw/EKBa8psRf7d9a4=