From 188ea926759c18731b5cbe5c072edf5cf51bb68e Mon Sep 17 00:00:00 2001 From: Jeremy Stanley Date: Thu, 17 Aug 2023 20:04:09 +0000 Subject: [PATCH] Mailman3: check docker-compose stderr not stdout For the conditional check as to whether docker-compose up was a no-op, we look for a recognizable string in the output it generates. This information is emitted on the stderr descriptor, not stdout, so correct the match condition accordingly. Change-Id: I82015e615071458c95342eaea2e6a17aeae44c07 --- playbooks/roles/mailman3/handlers/restart_mailman.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/playbooks/roles/mailman3/handlers/restart_mailman.yaml b/playbooks/roles/mailman3/handlers/restart_mailman.yaml index 9a8bc1b009..fc9f2dbc1b 100644 --- a/playbooks/roles/mailman3/handlers/restart_mailman.yaml +++ b/playbooks/roles/mailman3/handlers/restart_mailman.yaml @@ -5,7 +5,7 @@ - name: mailman restart containers if running # Also makes sure the containers weren't just restarted by an image update - when: uwsgi_pids.rc == 0 and "is up-to-date" in mailman_dcup.stdout + when: uwsgi_pids.rc == 0 and "is up-to-date" in mailman_dcup.stderr block: - name: down containers shell: