Skip to content

feat(acceptance): version-aware pull-wait for edge ArgoCD (no argocd-sync)#4

Merged
winggundamth merged 1 commit into
mainfrom
feat/v2.0-pull-wait-acceptance
Jun 21, 2026
Merged

feat(acceptance): version-aware pull-wait for edge ArgoCD (no argocd-sync)#4
winggundamth merged 1 commit into
mainfrom
feat/v2.0-pull-wait-acceptance

Conversation

@winggundamth

Copy link
Copy Markdown
Contributor

Extends the acceptance-test-curl reusable into a pull-wait gate for the v2.0 edge-ArgoCD model: set expected_version and the poll also waits until the app is serving that version before passing — so it waits for the edge cluster's own ArgoCD to pull + roll out the promoted image, with no argocd-sync and no ArgoCD API token.

  • Without it, the curl poll can false-green against the old pod (same body); the version gate fixes that.
  • Probes the body, or a response header (version_header, e.g. X-App-Version); separate version_url optional.
  • Version match is grep -F (tags contain dots). Back-compat: expected_version empty → unchanged.

Completes opsta-ai-pe v2.0 S4 (with #2 cross-repo gitops-deploy + #3 crane cross-registry bridge).

🤖 Generated with Claude Code

…sync)

Add expected_version (+ version_url / version_header) so the curl poll also gates on
the promoted version being LIVE before passing — the check waits for the edge cluster's
own ArgoCD to pull + roll out the new image, with no argocd-sync / no ArgoCD API token.
Without it the poll false-greens against the old pod (identical body). Version match is
grep -F (tags contain dots); reads body or a response header. Back-compat: expected_version
empty → acceptance_text-only behavior unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

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.

2 participants