argo-cd icon indicating copy to clipboard operation
argo-cd copied to clipboard
verified publisher icon argoproj

Incorrect health status for app of apps

Open LS80 opened this issue 4 years ago • 2 comments

Describe the bug

The health status of a parent app is briefly 💚 when a child app has status "missing". This becomes "progressing" as soon as the child application changes from "missing" to "progressing". This means that applications with a higher sync wave number start syncing before they should.

To Reproduce

  1. Follow the instructions at https://argoproj.github.io/argo-cd/operator-manual/health/#argocd-app to add the recommended custom health check.

  2. Add the repository https://github.com/LS80/argocd-app-of-apps

  3. Run this

argocd app create ls-sync-wave-issue \
  --repo https://github.com/LS80/argocd-app-of-apps.git \
  --path apps \
  --dest-namespace argocd \
  --dest-server https://kubernetes.default.svc \
  --helm-set namespace=ls-sync-wave-issue \
  --helm-set branch=ls-sync-wave-issue \
  --revision ls-sync-wave-issue \
  --sync-policy auto \
  --project default
  1. Observe the health status changes of ls-sync-wave-issue.

Expected behavior

The parent application should not have a 💚 status until all child applications are 💚 .

Screenshots Screen Shot 2021-07-13 at 22 26 54 ⬇️ Screen Shot 2021-07-13 at 22 27 08 ⬇️ Screen Shot 2021-07-13 at 22 10 14 ⬇️ Screen Shot 2021-07-13 at 22 10 27 ⬇️ Screen Shot 2021-07-13 at 22 33 33

Version

argocd: v2.0.1+33eaf11.dirty
  BuildDate: 2021-04-21T18:44:28Z
  GitCommit: 33eaf11e3abd8c761c726e815cbb4b6af7dcb030
  GitTreeState: dirty
  GoVersion: go1.16.3
  Compiler: gc
  Platform: darwin/amd64
argocd-server: v2.0.4+0842d44
  BuildDate: 2021-06-23T01:27:53Z
  GitCommit: 0842d448107eb1397b251e63ec4d4bc1b4efdd6e
  GitTreeState: clean
  GoVersion: go1.16
  Compiler: gc
  Platform: linux/amd64
  Ksonnet Version: v0.13.1
  Kustomize Version: v3.9.4 2021-02-09T19:22:10Z
  Helm Version: v3.5.1+g32c2223
  Kubectl Version: v0.20.4
  Jsonnet Version: v0.17.0

LS80 avatar Jul 13 '21 21:07 LS80

Hello! Has there been any resolution for this?

AnjaliSrivastava29 avatar Jan 17 '22 15:01 AnjaliSrivastava29

Hi! Any chance this might get looked into some time soon as the issue is persisting?

akolliakou avatar Apr 24 '24 08:04 akolliakou

I am running into this issue as well. Really hurts the ability to leverage app of apps pattern with sync-waves. Feels like this issue deserves more attention given its impact.

jackspirou avatar Jun 27 '24 15:06 jackspirou