)]}'
{"playbooks/roles/mailman3/handlers/restart_mailman.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"954f542c57a3b8381f2d7f1368e32bc3efa8f796","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  block:"},{"line_number":9,"context_line":"    - name: restart containers"},{"line_number":10,"context_line":"      shell:"},{"line_number":11,"context_line":"        cmd: docker-compose -f /etc/mailman-compose/docker-compose.yaml restart"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"27d3cd1f_97f08af2","line":11,"updated":"2023-08-16 15:41:21.000000000","message":"`docker compose restart` does not reflect changes made to the compose.yaml file per https://docs.docker.com/engine/reference/commandline/compose_restart/. One of the tasks notifying this handler is the docker compose yaml file being written. I don\u0027t think it will do what we want in that case.\n\nI think this will also avoid updating containers if the container image updates. It might be better to do something like `docker-compose down \u0026\u0026 docker-compose up -d` instead? Probably as two separate ansible tasks within this block.","commit_id":"46e9c5a9bf87265b60b65dd99e4537445b9ba6b0"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a6fd671be2e445aa72984c88d07ad34bfa12d62b","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  block:"},{"line_number":9,"context_line":"    - name: restart containers"},{"line_number":10,"context_line":"      shell:"},{"line_number":11,"context_line":"        cmd: docker-compose -f /etc/mailman-compose/docker-compose.yaml restart"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d1b19262_9aa7df02","line":11,"in_reply_to":"27d3cd1f_97f08af2","updated":"2023-08-17 15:30:25.000000000","message":"Done","commit_id":"46e9c5a9bf87265b60b65dd99e4537445b9ba6b0"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"a6fd671be2e445aa72984c88d07ad34bfa12d62b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  register: uwsgi_pids"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- name: mailman restart containers if running"},{"line_number":7,"context_line":"  when: uwsgi_pids.rc \u003d\u003d 0 and \"is up-to-date\" in mailman_dcup.stdout"},{"line_number":8,"context_line":"  block:"},{"line_number":9,"context_line":"    - name: down containers"},{"line_number":10,"context_line":"      shell:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ebc04d1c_847e38ec","line":7,"updated":"2023-08-17 15:30:25.000000000","message":"The weird condition here might be worth a comment, but if you grep mailan_dcup it should eventually become clear.","commit_id":"5fc536b74802501ea00bf9e27146636280f7fb27"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0fe61d51dcc224323a3d7594f1a666cce4b9d218","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  register: uwsgi_pids"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- name: mailman restart containers if running"},{"line_number":7,"context_line":"  when: uwsgi_pids.rc \u003d\u003d 0 and \"is up-to-date\" in mailman_dcup.stdout"},{"line_number":8,"context_line":"  block:"},{"line_number":9,"context_line":"    - name: down containers"},{"line_number":10,"context_line":"      shell:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"3b7c4d5a_2533ab75","line":7,"in_reply_to":"ebc04d1c_847e38ec","updated":"2023-08-17 15:53:22.000000000","message":"Done","commit_id":"5fc536b74802501ea00bf9e27146636280f7fb27"}]}