)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"06d62ba9ff27516a62e4593a54dc2dda06198741","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"27a921c5_4c984d0e","updated":"2023-06-09 07:02:45.000000000","message":"recheck","commit_id":"1af5b40c5e35a3150a69f09f2c23b69d2131a3a6"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"ad00201c881e3e64f3d779463eee84f4f7c880a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0bd13666_bac3cb4e","updated":"2023-06-13 07:42:15.000000000","message":"recheck","commit_id":"753d7287aee93a5cf610f1e70cd9090cf30e59ef"},{"author":{"_account_id":34598,"name":"Manojkatari","email":"mkatari@redhat.com","username":"mkatari"},"change_message_id":"aa718b54c270d75f4e0acee929575792b352726a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6c0f6fd9_be317b01","updated":"2023-06-15 10:16:24.000000000","message":"recheck","commit_id":"753d7287aee93a5cf610f1e70cd9090cf30e59ef"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"0a7e1e6a97d450adcd9fd44711a438a63c23f15c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c61bdca0_39795635","updated":"2023-06-15 18:43:13.000000000","message":"recheck","commit_id":"e4a8ff29d4ba12385883bbc82bdc71d872074e17"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"1e834c3b96ab48130dcc3b02af75d7fd53a27ff9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f5f8335c_907c8c01","updated":"2023-06-16 08:49:27.000000000","message":"recheck","commit_id":"e4a8ff29d4ba12385883bbc82bdc71d872074e17"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"14aee1f580339c87963217447fc6ef9c98b9b3b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ee5842af_914a34a2","updated":"2023-06-20 14:06:56.000000000","message":"Looks good to merge.","commit_id":"17b5db2275ce5c85cbc4acb9ee1cae52fe23f481"},{"author":{"_account_id":11166,"name":"Lukas Bezdicka","email":"social@v3.sk","username":"xbezdick"},"change_message_id":"4e74d696a368242b4e0edbd452acd1669fd996a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4b8d4717_c431d3b1","updated":"2023-06-20 13:29:47.000000000","message":"Tested and seems to work.","commit_id":"17b5db2275ce5c85cbc4acb9ee1cae52fe23f481"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"af8ef8f6361f816386542dba3a64fbe12ee37d51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c46dde41_ef4c69c8","updated":"2023-06-19 16:55:49.000000000","message":"recheck","commit_id":"17b5db2275ce5c85cbc4acb9ee1cae52fe23f481"}],"deployment/cephadm/ceph-osd.yaml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"1498d3d3a7aba146396876b3149c5dee89c637ec","unresolved":true,"context_lines":[{"line_number":106,"context_line":"      upgrade_tasks:"},{"line_number":107,"context_line":"        list_concat:"},{"line_number":108,"context_line":"        - {get_attr: [CephBase, role_data, upgrade_tasks]}"},{"line_number":109,"context_line":"        - - block:"},{"line_number":110,"context_line":"              - name: Set noout flag"},{"line_number":111,"context_line":"                shell: \"cephadm shell -- ceph osd set {{ item }}\""},{"line_number":112,"context_line":"                become: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"f72d93c7_04cd24cb","line":109,"range":{"start_line":109,"start_character":10,"end_line":109,"end_character":18},"updated":"2023-06-19 10:25:43.000000000","message":"This is a block, but only with one task. In post-upgrade tasks it looks much the same, but it not a block. Is the different correct?","commit_id":"753d7287aee93a5cf610f1e70cd9090cf30e59ef"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"42f055e3e19e00b2f1f5790fecd1ca4c26a849d6","unresolved":false,"context_lines":[{"line_number":106,"context_line":"      upgrade_tasks:"},{"line_number":107,"context_line":"        list_concat:"},{"line_number":108,"context_line":"        - {get_attr: [CephBase, role_data, upgrade_tasks]}"},{"line_number":109,"context_line":"        - - block:"},{"line_number":110,"context_line":"              - name: Set noout flag"},{"line_number":111,"context_line":"                shell: \"cephadm shell -- ceph osd set {{ item }}\""},{"line_number":112,"context_line":"                become: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"2041a9c3_e1be1066","line":109,"range":{"start_line":109,"start_character":10,"end_line":109,"end_character":18},"in_reply_to":"afac0567_f1475762","updated":"2023-06-19 10:41:04.000000000","message":"Done","commit_id":"753d7287aee93a5cf610f1e70cd9090cf30e59ef"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"9277464efe46d608e81e55985016b64f327c2983","unresolved":true,"context_lines":[{"line_number":106,"context_line":"      upgrade_tasks:"},{"line_number":107,"context_line":"        list_concat:"},{"line_number":108,"context_line":"        - {get_attr: [CephBase, role_data, upgrade_tasks]}"},{"line_number":109,"context_line":"        - - block:"},{"line_number":110,"context_line":"              - name: Set noout flag"},{"line_number":111,"context_line":"                shell: \"cephadm shell -- ceph osd set {{ item }}\""},{"line_number":112,"context_line":"                become: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"afac0567_f1475762","line":109,"range":{"start_line":109,"start_character":10,"end_line":109,"end_character":18},"in_reply_to":"f72d93c7_04cd24cb","updated":"2023-06-19 10:38:55.000000000","message":"Yes, and we actually don\u0027t really need a block here given we reduced the code to one task.\nUpdating the patch removing the block. Thanks, good catch.","commit_id":"753d7287aee93a5cf610f1e70cd9090cf30e59ef"},{"author":{"_account_id":25402,"name":"Francesco Pantano","email":"fpantano@redhat.com","username":"fmount"},"change_message_id":"42f055e3e19e00b2f1f5790fecd1ca4c26a849d6","unresolved":true,"context_lines":[{"line_number":106,"context_line":"      upgrade_tasks:"},{"line_number":107,"context_line":"        list_concat:"},{"line_number":108,"context_line":"        - {get_attr: [CephBase, role_data, upgrade_tasks]}"},{"line_number":109,"context_line":"        - - name: Set noout flag"},{"line_number":110,"context_line":"            shell: \"cephadm shell -- ceph osd set {{ item }}\""},{"line_number":111,"context_line":"            become: true"},{"line_number":112,"context_line":"            with_items:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6efc77b3_661129b7","line":109,"updated":"2023-06-19 10:41:04.000000000","message":"@Jesse I guess it looks better now. @Lukas, this code is equivalent to what we had in the previous PS: I don\u0027t expect changes in the behavior, but feel free to give it a try in your next run!","commit_id":"17b5db2275ce5c85cbc4acb9ee1cae52fe23f481"}]}