)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"3e0883d5cc4323895780bab25820d8220e052d45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25c30d2b_04158c41","updated":"2023-01-27 17:25:23.000000000","message":"Apologies, didn\u0027t get to this today. Will take a look on Monday morning.","commit_id":"e3c2627c431771905f5b36a4b7ed1ea7196790af"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"377937fa9a589f75740f7713fb349d7fa945fcbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"34d85746_6b759e91","updated":"2023-02-06 10:01:07.000000000","message":"Thanks Bartosz, this looks good, just one question.","commit_id":"6c4b70e857e8ecbeff00d810b8a22fb9a41b0d38"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"7e2db81d321177730f223ae9530729a5de64995a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2efb387e_279e2886","updated":"2023-01-31 13:13:30.000000000","message":"recheck - arm debian upgrade rabbit started late","commit_id":"6c4b70e857e8ecbeff00d810b8a22fb9a41b0d38"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"9275090cd879a82aae42643400c3a8753cc18c50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ee028d9c_ebb093aa","updated":"2023-02-07 14:13:13.000000000","message":"Thanks","commit_id":"ee658f454941b67d1ae077fcf7cf6094b7b0bb20"}],"ansible/roles/opensearch/tasks/upgrade.yml":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"377937fa9a589f75740f7713fb349d7fa945fcbd","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  register: result"},{"line_number":35,"context_line":"  until: (\u0027status\u0027 in result) and result.status \u003d\u003d 200"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Stop all opensearch containers before applying configuration to ensure"},{"line_number":38,"context_line":"# handlers are triggered to restart them."},{"line_number":39,"context_line":"- name: Stopping all opensearch containers"},{"line_number":40,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7c74bce8_17b053c0","line":37,"updated":"2023-02-06 10:01:07.000000000","message":"Hmm, should we split this out to a separate patch? I\u0027m not sure I follow why we need it. Shouldn\u0027t check-containers.yml / config.yml trigger the handler when the container / config is updated?","commit_id":"6c4b70e857e8ecbeff00d810b8a22fb9a41b0d38"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"f5922f2330e4be2069d4570c6647b79540db2b89","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  register: result"},{"line_number":35,"context_line":"  until: (\u0027status\u0027 in result) and result.status \u003d\u003d 200"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Stop all opensearch containers before applying configuration to ensure"},{"line_number":38,"context_line":"# handlers are triggered to restart them."},{"line_number":39,"context_line":"- name: Stopping all opensearch containers"},{"line_number":40,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b5606f67_7b37d16f","line":37,"in_reply_to":"7c74bce8_17b053c0","updated":"2023-02-07 10:22:55.000000000","message":"I re-added it as it was in pre-opensearch era - https://github.com/openstack/kolla-ansible/blob/stable/yoga/ansible/roles/elasticsearch/tasks/upgrade.yml#L39\n\nI will test upgrade jobs with out","commit_id":"6c4b70e857e8ecbeff00d810b8a22fb9a41b0d38"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"538cfa972bed38ffd16103f7d78c31944fd19dd0","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  register: result"},{"line_number":35,"context_line":"  until: (\u0027status\u0027 in result) and result.status \u003d\u003d 200"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Stop all opensearch containers before applying configuration to ensure"},{"line_number":38,"context_line":"# handlers are triggered to restart them."},{"line_number":39,"context_line":"- name: Stopping all opensearch containers"},{"line_number":40,"context_line":"  vars:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a8d45b91_da0b8de8","line":37,"in_reply_to":"b5606f67_7b37d16f","updated":"2023-02-07 13:19:10.000000000","message":"Done","commit_id":"6c4b70e857e8ecbeff00d810b8a22fb9a41b0d38"}]}