)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"e2b654086824a53b94e4eccc52352031a0805961","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This change addresses an issue in the nova-libvirt-cleanup command,"},{"line_number":10,"context_line":"added in I46854ed7eaf1d5b5e3ccd8531c963427848bdc99."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Remove pipefail bash option from pgrep command, since a lack of matches"},{"line_number":13,"context_line":"is a pass."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, use bash for set -o pipefail."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Iffda0dfffce8768324ffec55e629134c70e2e996"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"e0149e0a_e2202efe","line":14,"range":{"start_line":12,"start_character":0,"end_line":14,"end_character":0},"updated":"2022-03-30 16:16:40.000000000","message":"nope","commit_id":"bb9986931456b749ba6703d98eb52a9beb0b64f0"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fc713482aaa45dcec70c411a64313ec32694690b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This change addresses an issue in the nova-libvirt-cleanup command,"},{"line_number":10,"context_line":"added in I46854ed7eaf1d5b5e3ccd8531c963427848bdc99."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Remove pipefail bash option from pgrep command, since a lack of matches"},{"line_number":13,"context_line":"is a pass."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Also, use bash for set -o pipefail."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Iffda0dfffce8768324ffec55e629134c70e2e996"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dbb12d76_52ab6ddf","line":14,"range":{"start_line":12,"start_character":0,"end_line":14,"end_character":0},"in_reply_to":"e0149e0a_e2202efe","updated":"2022-04-05 08:31:05.000000000","message":"Done","commit_id":"bb9986931456b749ba6703d98eb52a9beb0b64f0"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"c291bb52a0d89be96354889bc6fd59731fce7e56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9f80a0bb_14869b66","updated":"2022-03-30 12:30:43.000000000","message":"The suggestion is to catch other issues with pgrep (like pgrep missing for some reason or faulty) but pass if no matches (rc\u003d1).","commit_id":"8d36403a555c267c747d247aa370a9189d3bb24a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"b9c89b8a0755dfa8583b073fa3f06ffecec7eab5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f3e2a7d4_de425f30","updated":"2022-03-30 16:16:20.000000000","message":"commit message to amend","commit_id":"bb9986931456b749ba6703d98eb52a9beb0b64f0"}],"ansible/roles/nova-cell/tasks/libvirt-cleanup.yml":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"c291bb52a0d89be96354889bc6fd59731fce7e56","unresolved":true,"context_lines":[{"line_number":18,"context_line":"      become: true"},{"line_number":19,"context_line":"      shell:"},{"line_number":20,"context_line":"        cmd: \u003e"},{"line_number":21,"context_line":"          set -o pipefail \u0026\u0026"},{"line_number":22,"context_line":"          pgrep -l qemu | awk \u0027!/qemu-ga/  \u0026\u0026 !/qemu-img/ {print $1}\u0027"},{"line_number":23,"context_line":"      register: running_vms"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"45d3502d_138882b5","side":"PARENT","line":21,"updated":"2022-03-30 12:30:43.000000000","message":"keep it","commit_id":"f993047f4506080d076c175b0dfeef79b497f0e7"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fc713482aaa45dcec70c411a64313ec32694690b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      become: true"},{"line_number":19,"context_line":"      shell:"},{"line_number":20,"context_line":"        cmd: \u003e"},{"line_number":21,"context_line":"          set -o pipefail \u0026\u0026"},{"line_number":22,"context_line":"          pgrep -l qemu | awk \u0027!/qemu-ga/  \u0026\u0026 !/qemu-img/ {print $1}\u0027"},{"line_number":23,"context_line":"      register: running_vms"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"28ab4db0_906cb74e","side":"PARENT","line":21,"in_reply_to":"45d3502d_138882b5","updated":"2022-04-05 08:31:05.000000000","message":"Done","commit_id":"f993047f4506080d076c175b0dfeef79b497f0e7"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"c291bb52a0d89be96354889bc6fd59731fce7e56","unresolved":true,"context_lines":[{"line_number":14,"context_line":"  register: container_facts"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- block:"},{"line_number":17,"context_line":"    # NOTE(mgoddard): Don\u0027t set pipefail here, since a lack of matches is a pass."},{"line_number":18,"context_line":"    - name: Check if there are any running VMs"},{"line_number":19,"context_line":"      become: true"},{"line_number":20,"context_line":"      shell:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"98c2534f_1583a038","line":17,"updated":"2022-03-30 12:30:43.000000000","message":"amend the comment","commit_id":"8d36403a555c267c747d247aa370a9189d3bb24a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fc713482aaa45dcec70c411a64313ec32694690b","unresolved":false,"context_lines":[{"line_number":14,"context_line":"  register: container_facts"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- block:"},{"line_number":17,"context_line":"    # NOTE(mgoddard): Don\u0027t set pipefail here, since a lack of matches is a pass."},{"line_number":18,"context_line":"    - name: Check if there are any running VMs"},{"line_number":19,"context_line":"      become: true"},{"line_number":20,"context_line":"      shell:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"359ac275_a8c85d23","line":17,"in_reply_to":"98c2534f_1583a038","updated":"2022-04-05 08:31:05.000000000","message":"Done","commit_id":"8d36403a555c267c747d247aa370a9189d3bb24a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"c291bb52a0d89be96354889bc6fd59731fce7e56","unresolved":true,"context_lines":[{"line_number":19,"context_line":"      become: true"},{"line_number":20,"context_line":"      shell:"},{"line_number":21,"context_line":"        cmd: \u003e"},{"line_number":22,"context_line":"          pgrep -l qemu | awk \u0027!/qemu-ga/  \u0026\u0026 !/qemu-img/ {print $1}\u0027"},{"line_number":23,"context_line":"      register: running_vms"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    - name: Fail if there are any running VMs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2721249c_a812f51f","line":22,"range":{"start_line":22,"start_character":10,"end_line":22,"end_character":23},"updated":"2022-03-30 12:30:43.000000000","message":"{ pgrep -l qemu || test $? \u003d 1; }","commit_id":"8d36403a555c267c747d247aa370a9189d3bb24a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fc713482aaa45dcec70c411a64313ec32694690b","unresolved":false,"context_lines":[{"line_number":19,"context_line":"      become: true"},{"line_number":20,"context_line":"      shell:"},{"line_number":21,"context_line":"        cmd: \u003e"},{"line_number":22,"context_line":"          pgrep -l qemu | awk \u0027!/qemu-ga/  \u0026\u0026 !/qemu-img/ {print $1}\u0027"},{"line_number":23,"context_line":"      register: running_vms"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"    - name: Fail if there are any running VMs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"a33ea999_f6779a7a","line":22,"range":{"start_line":22,"start_character":10,"end_line":22,"end_character":23},"in_reply_to":"2721249c_a812f51f","updated":"2022-04-05 08:31:05.000000000","message":"Done","commit_id":"8d36403a555c267c747d247aa370a9189d3bb24a"}]}