Skip to content

Adopt verified remote routes without redeploying - #45

Merged
squarepots merged 5 commits into
mainfrom
fix/adopt-verified-deployment
Sep 17, 2026
Merged

squarepots merged 5 commits into
mainfrom
fix/adopt-verified-deployment

Conversation

@squarepots

Copy link
Copy Markdown
Owner

Summary

  • treat a verified in-sync preflight audit as completed remote deployment and adopt that state locally instead of mutating the host again
  • keep remote deployment limited to a verified service-missing state while the Route is still pending/deploying
  • preserve the existing fail-closed behavior for drifted or undetermined remote state
  • add regression coverage for the recovery decision and local adoption path

Why

PR #41 made every deployment retry audit first, but still allowed in-sync to continue into performRouteOperation, which could redeploy a Route whose remote mutation had already succeeded. That violates #7 acceptance criterion 3: a local persistence failure after successful remote deployment must recover without redeploying blindly.

Version impact

patch — 2.2.2 → 2.2.3

Closes #7

Signed-off-by: squarepots <46488165+squarepots@users.noreply.github.com>
Signed-off-by: squarepots <46488165+squarepots@users.noreply.github.com>
Signed-off-by: squarepots <46488165+squarepots@users.noreply.github.com>
Signed-off-by: squarepots <46488165+squarepots@users.noreply.github.com>
Signed-off-by: squarepots <46488165+squarepots@users.noreply.github.com>
@squarepots
squarepots merged commit 6ae89d7 into main Sep 17, 2026
9 checks passed
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.

Make migration and partial deployment recovery reliable

1 participant