Skip to content

Fix argocd app wait printing incorrect Sync output (resolves #542)#543

Merged
jessesuen merged 1 commit intoargoproj:masterfrom
jessesuen:sync-cli-cleanup
Aug 31, 2018
Merged

Fix argocd app wait printing incorrect Sync output (resolves #542)#543
jessesuen merged 1 commit intoargoproj:masterfrom
jessesuen:sync-cli-cleanup

Conversation

@jessesuen
Copy link
Member

@jessesuen jessesuen commented Aug 31, 2018

Fix argocd app wait printing incorrect Sync output
Timeout condition was not printing final status.

Example outputs:

$ argocd app wait guestbook-default

Application:        guestbook-default
Operation:          Sync
Phase:              Succeeded
Start:              2018-08-31 01:12:17 -0700 PDT
Finished:           2018-08-31 01:12:20 -0700 PDT
Duration:           3s
Message:            successfully synced

KIND        NAME          STATUS  HEALTH   HOOK  OPERATIONMSG
Deployment  guestbook-ui  Synced  Healthy        deployment.apps/guestbook-ui configured
Service     guestbook-ui  Synced  Healthy        service/guestbook-ui unchanged


$ argocd app sync guestbook-default
KIND        NAME          STATUS     HEALTH   HOOK  OPERATIONMSG
Deployment  guestbook-ui  OutOfSync  Healthy
Service     guestbook-ui  Synced     Healthy
Deployment  guestbook-ui  OutOfSync  Healthy       deployment.apps/guestbook-ui configured
Service     guestbook-ui  Synced     Healthy       service/guestbook-ui unchanged

Application:        guestbook-default
Operation:          Sync
Phase:              Succeeded
Start:              2018-08-31 01:12:17 -0700 PDT
Finished:           2018-08-31 01:12:20 -0700 PDT
Duration:           3s
Message:            successfully synced

KIND        NAME          STATUS  HEALTH   HOOK  OPERATIONMSG
Deployment  guestbook-ui  Synced  Healthy        deployment.apps/guestbook-ui configured
Service     guestbook-ui  Synced  Healthy        service/guestbook-ui unchanged

$ argocd app sync pre-post-sync
KIND        NAME           STATUS     HEALTH   HOOK  OPERATIONMSG
Service     pre-post-sync  OutOfSync  Missing
Deployment  pre-post-sync  OutOfSync  Missing
Job  before-a3ea618-presync-1535703633  Running       PreSync
Job  before-a3ea618-presync-1535703633  Succeeded       PreSync
Service     pre-post-sync  OutOfSync  Missing       service/pre-post-sync created
Deployment  pre-post-sync  OutOfSync  Missing       deployment.apps/pre-post-sync created
Deployment  pre-post-sync  Synced  Progressing       deployment.apps/pre-post-sync created
Service     pre-post-sync  Synced  Healthy           service/pre-post-sync created
Deployment  pre-post-sync  Synced  Healthy       deployment.apps/pre-post-sync created
Job  after-a3ea618-postsync-1535703633  Running       PostSync
Job  after-a3ea618-postsync-1535703633  Succeeded       PostSync

Application:        pre-post-sync
Operation:          Sync
Phase:              Succeeded
Start:              2018-08-31 01:20:33 -0700 PDT
Finished:           2018-08-31 01:21:48 -0700 PDT
Duration:           1m15s
Message:            successfully synced

KIND        NAME                               STATUS     HEALTH   HOOK      OPERATIONMSG
Job         before-a3ea618-presync-1535703633  Succeeded           PreSync
Service     pre-post-sync                      Synced     Healthy            service/pre-post-sync created
Deployment  pre-post-sync                      Synced     Healthy            deployment.apps/pre-post-sync created
Job         after-a3ea618-postsync-1535703633  Succeeded           PostSync

…oj#542)

Timeout condition was not printing final status.
@jessesuen jessesuen merged commit 1e8c570 into argoproj:master Aug 31, 2018
leoluz pushed a commit to leoluz/argo-cd that referenced this pull request Mar 13, 2025
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