)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"f61e70f8ce8cfd1b3966e8b82c4d508e849e6c30","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[OVN][Migration] More robust checks on stop-agents"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"On OSP17 there are some ml2 ovs services that are not"},{"line_number":10,"context_line":"present on some computes eventhought is defined on the"},{"line_number":11,"context_line":"ansible service facts."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This patch will ensure that only those services that are"},{"line_number":14,"context_line":"actually running will be stopped."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"76b2e62b_1731ad2a","line":11,"range":{"start_line":10,"start_character":16,"end_line":11,"end_character":22},"updated":"2022-04-28 05:56:17.000000000","message":"looks strange, is it known what causing this behavior?\n\nThe patch is good in itself.","commit_id":"2869d008e2e417e43f3fbf942225a9edc5868e37"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"5ee98b77a8c56bd8a7ab11a24206b19809397ef1","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[OVN][Migration] More robust checks on stop-agents"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"On OSP17 there are some ml2 ovs services that are not"},{"line_number":10,"context_line":"present on some computes eventhought is defined on the"},{"line_number":11,"context_line":"ansible service facts."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This patch will ensure that only those services that are"},{"line_number":14,"context_line":"actually running will be stopped."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"12d1044f_8705d7d8","line":11,"range":{"start_line":10,"start_character":16,"end_line":11,"end_character":22},"in_reply_to":"76b2e62b_1731ad2a","updated":"2022-04-28 08:27:32.000000000","message":"Thanks Yatin.\n\nNo, this worked fine in OSP 16, I will try to investigate the root cause","commit_id":"2869d008e2e417e43f3fbf942225a9edc5868e37"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"7c221b55f78f6d74ee65fc9671d01913b43113c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"126d3976_2795cf8c","updated":"2022-04-26 17:25:23.000000000","message":"Need to test it on a migration","commit_id":"04a9e7cb8ce4f7f628e2fa24a80b862b1ddeab58"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6b1470832d4a70d821444923188f3d26d1dd0a9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"87563d3a_73c67971","updated":"2022-04-27 08:38:49.000000000","message":"Makes sense","commit_id":"2869d008e2e417e43f3fbf942225a9edc5868e37"}],"tools/ovn_migration/tripleo_environment/playbooks/roles/stop-agents/tasks/cleanup.yml":[{"author":{"_account_id":29350,"name":"Roman Safronov","email":"rsafrono@redhat.com","username":"rsafrono"},"change_message_id":"f85b03592169787141e014124fa90207c5b53da6","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":"  when:"},{"line_number":9,"context_line":"    - ansible_facts.services[item] is defined"},{"line_number":10,"context_line":"    - ansible_facts.services[item][\"state\"] \u003d\u003d running"},{"line_number":11,"context_line":"  loop:"},{"line_number":12,"context_line":"    - \"{{ service.healthcheck_timer_file }}\""},{"line_number":13,"context_line":"    - \"{{ service.healthcheck_service_file }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"94a5d83d_246ad596","line":10,"range":{"start_line":10,"start_character":47,"end_line":10,"end_character":54},"updated":"2022-04-27 08:04:27.000000000","message":"I tried to test, it seems like quotes are missing here, i.e. it should be \"running\"","commit_id":"04a9e7cb8ce4f7f628e2fa24a80b862b1ddeab58"},{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"3640068c2120e1fcfcd4adcc83002e2e39bc01f2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":"  when:"},{"line_number":9,"context_line":"    - ansible_facts.services[item] is defined"},{"line_number":10,"context_line":"    - ansible_facts.services[item][\"state\"] \u003d\u003d running"},{"line_number":11,"context_line":"  loop:"},{"line_number":12,"context_line":"    - \"{{ service.healthcheck_timer_file }}\""},{"line_number":13,"context_line":"    - \"{{ service.healthcheck_service_file }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3cea1af1_1f9946ec","line":10,"range":{"start_line":10,"start_character":47,"end_line":10,"end_character":54},"in_reply_to":"94a5d83d_246ad596","updated":"2022-04-27 08:28:15.000000000","message":"Correct! Fixed in this patchet 2! Thanks Roman!","commit_id":"04a9e7cb8ce4f7f628e2fa24a80b862b1ddeab58"}]}