)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"73a55949d21db860b58533b222f02ca1d67f6ad5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5c6cdb53_9899ec6e","updated":"2023-02-22 07:19:03.000000000","message":"patch looks good, just one nit to consider","commit_id":"1a8c63e6d2b293e2e7a7fa9c44c982934114f05f"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"3dd50ee3f76f0a96350bd0a521b9a93db01b2445","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1b6a83f4_26a0cfe9","updated":"2023-02-21 18:10:58.000000000","message":"recheck ovn-octavia-provider-tempest-release timeout","commit_id":"1a8c63e6d2b293e2e7a7fa9c44c982934114f05f"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"1a2f6243a94982d6cf2c541dd18b9549e32b11d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"21738c43_0a96a8b4","updated":"2023-03-01 07:11:40.000000000","message":"recheck","commit_id":"569c9c011a2cc9e5f041633dde4a47257e4d8565"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"17faa3a77b25a5b26b25270b0ee22fa9b376651f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e6fc9839_500e7c0e","updated":"2023-03-01 07:10:36.000000000","message":"trying to trigger the gates","commit_id":"569c9c011a2cc9e5f041633dde4a47257e4d8565"}],"ovn_octavia_provider/helper.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"73a55949d21db860b58533b222f02ca1d67f6ad5","unresolved":true,"context_lines":[{"line_number":2664,"context_line":"        hms_key \u003d ovn_lb.external_ids.get(ovn_const.LB_EXT_IDS_HMS_KEY, [])"},{"line_number":2665,"context_line":""},{"line_number":2666,"context_line":"        # Update status for members in the pool related to HM"},{"line_number":2667,"context_line":"        member_status \u003d []"},{"line_number":2668,"context_line":"        existing_members \u003d ovn_lb.external_ids.get("},{"line_number":2669,"context_line":"            ovn_const.LB_EXT_IDS_POOL_PREFIX + str(pool_id_related))"},{"line_number":2670,"context_line":"        if len(existing_members) \u003e 0:"},{"line_number":2671,"context_line":"            for mem_info in existing_members.split(\u0027,\u0027):"},{"line_number":2672,"context_line":"                member_status.append({"},{"line_number":2673,"context_line":"                    constants.ID: mem_info.split(\u0027_\u0027)[1],"},{"line_number":2674,"context_line":"                    constants.PROVISIONING_STATUS: constants.ACTIVE,"},{"line_number":2675,"context_line":"                    constants.OPERATING_STATUS: constants.NO_MONITOR})"},{"line_number":2676,"context_line":"                self._update_member_status("},{"line_number":2677,"context_line":"                    ovn_lb,"},{"line_number":2678,"context_line":"                    mem_info.split(\u0027_\u0027)[1],"},{"line_number":2679,"context_line":"                    constants.NO_MONITOR)"},{"line_number":2680,"context_line":""},{"line_number":2681,"context_line":"        if hms_key:"},{"line_number":2682,"context_line":"            hms_key \u003d jsonutils.loads(hms_key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"62568fdc_ea0b8837","line":2679,"range":{"start_line":2667,"start_character":0,"end_line":2679,"end_character":41},"updated":"2023-02-22 07:19:03.000000000","message":"this is pretty similar to the code in hm_create, perhaps it can be split out to a function and used it in both places","commit_id":"1a8c63e6d2b293e2e7a7fa9c44c982934114f05f"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"2a9a18586ba51449aff37fe3531bd005beadffa2","unresolved":false,"context_lines":[{"line_number":2664,"context_line":"        hms_key \u003d ovn_lb.external_ids.get(ovn_const.LB_EXT_IDS_HMS_KEY, [])"},{"line_number":2665,"context_line":""},{"line_number":2666,"context_line":"        # Update status for members in the pool related to HM"},{"line_number":2667,"context_line":"        member_status \u003d []"},{"line_number":2668,"context_line":"        existing_members \u003d ovn_lb.external_ids.get("},{"line_number":2669,"context_line":"            ovn_const.LB_EXT_IDS_POOL_PREFIX + str(pool_id_related))"},{"line_number":2670,"context_line":"        if len(existing_members) \u003e 0:"},{"line_number":2671,"context_line":"            for mem_info in existing_members.split(\u0027,\u0027):"},{"line_number":2672,"context_line":"                member_status.append({"},{"line_number":2673,"context_line":"                    constants.ID: mem_info.split(\u0027_\u0027)[1],"},{"line_number":2674,"context_line":"                    constants.PROVISIONING_STATUS: constants.ACTIVE,"},{"line_number":2675,"context_line":"                    constants.OPERATING_STATUS: constants.NO_MONITOR})"},{"line_number":2676,"context_line":"                self._update_member_status("},{"line_number":2677,"context_line":"                    ovn_lb,"},{"line_number":2678,"context_line":"                    mem_info.split(\u0027_\u0027)[1],"},{"line_number":2679,"context_line":"                    constants.NO_MONITOR)"},{"line_number":2680,"context_line":""},{"line_number":2681,"context_line":"        if hms_key:"},{"line_number":2682,"context_line":"            hms_key \u003d jsonutils.loads(hms_key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"e4c04e4c_6e3c9450","line":2679,"range":{"start_line":2667,"start_character":0,"end_line":2679,"end_character":41},"in_reply_to":"62568fdc_ea0b8837","updated":"2023-02-22 11:31:49.000000000","message":"Done","commit_id":"1a8c63e6d2b293e2e7a7fa9c44c982934114f05f"}]}