)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0e794eee08751fd36726c16404d817e69dddb937","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Reintroduce upgrade tasks for stopping pacemaker cluster"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It is likely that change I50a1289a864f804d02a2e2bc0ca8738a186beff0"},{"line_number":10,"context_line":"broke upgrade CI, even though the job somehow passed on it. MariaDB"},{"line_number":11,"context_line":"upgrade tasks now cannot remove the container image, because it\u0027s"},{"line_number":12,"context_line":"being used by a running container. Let\u0027s keep all tasks for stopping"},{"line_number":13,"context_line":"Pacemaker cluster for now, both untagged ones and the ones tagged for"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bfb3d3c7_4cea1dd8","line":10,"range":{"start_line":10,"start_character":6,"end_line":10,"end_character":16},"updated":"2019-05-30 07:50:53.000000000","message":"ugh which job - you mean standalone-upgrade or something else","commit_id":"f8ca0d41add4254401654c89f146d8c38a3d9ac3"},{"author":{"_account_id":8042,"name":"Jiří Stránský","email":"jistr@redhat.com","username":"jistr"},"change_message_id":"17ed648b93d7a19c21895ad0de0ead80bbb32ad7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Reintroduce upgrade tasks for stopping pacemaker cluster"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"It is likely that change I50a1289a864f804d02a2e2bc0ca8738a186beff0"},{"line_number":10,"context_line":"broke upgrade CI, even though the job somehow passed on it. MariaDB"},{"line_number":11,"context_line":"upgrade tasks now cannot remove the container image, because it\u0027s"},{"line_number":12,"context_line":"being used by a running container. Let\u0027s keep all tasks for stopping"},{"line_number":13,"context_line":"Pacemaker cluster for now, both untagged ones and the ones tagged for"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bfb3d3c7_47d436e9","line":10,"range":{"start_line":10,"start_character":6,"end_line":10,"end_character":16},"in_reply_to":"bfb3d3c7_4cea1dd8","updated":"2019-05-30 09:06:24.000000000","message":"The other one, tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades","commit_id":"f8ca0d41add4254401654c89f146d8c38a3d9ac3"}],"deployment/pacemaker/pacemaker-baremetal-puppet.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0e794eee08751fd36726c16404d817e69dddb937","unresolved":false,"context_lines":[{"line_number":146,"context_line":"      upgrade_tasks:"},{"line_number":147,"context_line":"        - name: system_upgrade_prepare step 2"},{"line_number":148,"context_line":"          tags:"},{"line_number":149,"context_line":"            - never"},{"line_number":150,"context_line":"            - system_upgrade_prepare"},{"line_number":151,"context_line":"          when: step|int \u003d\u003d 2"},{"line_number":152,"context_line":"          block:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bfb3d3c7_ecd5111d","line":149,"updated":"2019-05-30 07:50:53.000000000","message":"what\u0027s the never tag?! is it the new --trash-my-machine?","commit_id":"f8ca0d41add4254401654c89f146d8c38a3d9ac3"},{"author":{"_account_id":8042,"name":"Jiří Stránský","email":"jistr@redhat.com","username":"jistr"},"change_message_id":"e73d00c211396226c8d633a5eb1b5490dfaa3005","unresolved":false,"context_lines":[{"line_number":146,"context_line":"      upgrade_tasks:"},{"line_number":147,"context_line":"        - name: system_upgrade_prepare step 2"},{"line_number":148,"context_line":"          tags:"},{"line_number":149,"context_line":"            - never"},{"line_number":150,"context_line":"            - system_upgrade_prepare"},{"line_number":151,"context_line":"          when: step|int \u003d\u003d 2"},{"line_number":152,"context_line":"          block:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bfb3d3c7_47eaf6d7","line":149,"in_reply_to":"bfb3d3c7_ecd5111d","updated":"2019-05-30 08:57:58.000000000","message":"Pretty much yes :D. It\u0027s \"don\u0027t run this unless explicitly told so by --tags system_upgrade_prepare\". It\u0027s a tag with special meaning in Ansible itself.","commit_id":"f8ca0d41add4254401654c89f146d8c38a3d9ac3"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"e31a8077b4d9bee0c0e9401013bd39f171ae19cf","unresolved":false,"context_lines":[{"line_number":151,"context_line":"          when: step|int \u003d\u003d 2"},{"line_number":152,"context_line":"          block:"},{"line_number":153,"context_line":"            - name: Stop pacemaker cluster"},{"line_number":154,"context_line":"              pacemaker_cluster: state\u003doffline"},{"line_number":155,"context_line":"        - name: Check pacemaker cluster running before upgrade"},{"line_number":156,"context_line":"          when: step|int \u003d\u003d 0"},{"line_number":157,"context_line":"          tags: validation"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bfb3d3c7_2c06c921","line":154,"updated":"2019-05-30 08:30:31.000000000","message":"I was wondering how could this patch https://review.opendev.org/#/c/651594/12 impact in the CI job when it had the never tag...but I dind\u0027t realize that we were skipping the pacemaker stop by doing it that way...thank\u0027s for the fix Jirka.","commit_id":"f8ca0d41add4254401654c89f146d8c38a3d9ac3"}]}