Skip to content

feat(argocd-wait-sync): add opt-in resource-level health tracking - #334

Merged
joscdk merged 1 commit into
mainfrom
feat/argocd-wait-sync-resource-tracking
Aug 28, 2026
Merged

feat(argocd-wait-sync): add opt-in resource-level health tracking#334
joscdk merged 1 commit into
mainfrom
feat/argocd-wait-sync-resource-tracking

Conversation

@joscdk

@joscdk joscdk commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds opt-in resource-name (+ resource-kind, default Deployment, and resource-namespace) inputs to argocd-wait-sync.
  • When resource-name is set, completion and fail-fast (Degraded/Missing) are judged on that specific resource in .status.resources[] instead of the app's aggregate .status.health — so an app that manages multiple independently-owned resources (e.g. a shared/monorepo-style app with several Deployments) won't falsely pass before your resource rolls out, or block forever because an unrelated resource in the same app is Degraded/Progressing.
  • Empty resource-name (default) is fully backwards compatible — identical behavior to before.
  • test-local.sh gains matching --resource-name/--resource-kind/--resource-namespace flags. README and CLAUDE.md updated.

Test plan

  • bash -n on wait-sync.sh and test-local.sh
  • YAML-parsed action.yml
  • Manual jq check confirming the resource filter picks the tracked resource's health (Healthy) independently of app aggregate health (Degraded from an unrelated resource)
  • Run test-local.sh against a real shared ArgoCD app with --resource-name before merging

🤖 Generated with Claude Code

https://claude.ai/code/session_016sb2uMVCHMsoWxYaPLAA7A

Adds resource-name/resource-kind/resource-namespace inputs so the action
can wait on a specific resource in the app's tree instead of the app's
aggregate health, for apps that manage multiple independently-owned
resources (e.g. a shared/monorepo-style app with several Deployments).
Defaults preserve existing behavior exactly.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016sb2uMVCHMsoWxYaPLAA7A
@joscdk
joscdk requested a review from a team as a code owner August 28, 2026 07:57
@joscdk
joscdk requested review from maoanran and removed request for a team August 28, 2026 07:57
@joscdk
joscdk merged commit b271ef2 into main Aug 28, 2026
1 check passed
@joscdk
joscdk deleted the feat/argocd-wait-sync-resource-tracking branch August 28, 2026 08:09
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