)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"7ebffbb0d6c9619a2bcf0bf1d7f5992e9775786e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2374b6ee_28e292d9","updated":"2022-07-14 06:21:57.000000000","message":"besides zull complaining, just one nit.\n\nAlso, this seems like something unittest should have tested/detected it","commit_id":"6c46c79ab13d38379409f7bc85bdbc4c5e232d93"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"6fe1fa887ad844872c5099d2c1029125754aa28c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"37916663_26b83c39","in_reply_to":"2374b6ee_28e292d9","updated":"2022-07-14 07:12:18.000000000","message":"TODO: add IPv6 tests in a future patch, it is currently tested by the Depends on patch related on octavia-tempest-plugin","commit_id":"6c46c79ab13d38379409f7bc85bdbc4c5e232d93"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"0d2df334a8890eff27289a376d2f7c2e634fb2fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"39c6d9b6_6f896ef9","updated":"2022-07-14 07:16:07.000000000","message":"ack, so, lets wait for the gates, once they are green we remove the depends-on and merge it","commit_id":"4aca8293bccfea770375b777420a252ed1ec295d"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"0264d970afd9474d72ae74ca18ea02da870e5866","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a8d02c9d_c822a8a0","updated":"2022-07-14 15:28:29.000000000","message":"error now seems unrelated, so we can remove the depends on and merge. I\u0027ll give +2 after removing it","commit_id":"4aca8293bccfea770375b777420a252ed1ec295d"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"d1fd3b7e73fe9b176b380f30ca0e7bed8e630610","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"deeb0fdd_2da68f72","updated":"2022-07-14 13:57:29.000000000","message":"recheck ovn-octavia-provider-tempest-master unrelated","commit_id":"4aca8293bccfea770375b777420a252ed1ec295d"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"b3a6fa71ddbd8a011dcb03c7583a4a5c16fa3664","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b966671e_29efa1d7","updated":"2022-07-18 13:13:39.000000000","message":"recheck  ovn-octavia-provider-functional-master cursor reset commit/rollback","commit_id":"6711a3e32dd1b5e5d8bbcc401a57ec229b690546"}],"ovn_octavia_provider/driver.py":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"7ebffbb0d6c9619a2bcf0bf1d7f5992e9775786e","unresolved":true,"context_lines":[{"line_number":391,"context_line":""},{"line_number":392,"context_line":"        for member in members_to_delete:"},{"line_number":393,"context_line":"            member_info \u003d member.split(\u0027_\u0027)"},{"line_number":394,"context_line":"            ("},{"line_number":395,"context_line":"                member_ip,"},{"line_number":396,"context_line":"                member_port,"},{"line_number":397,"context_line":"                subnet_id,"},{"line_number":398,"context_line":"            ) \u003d self._ovn_helper._extract_member_info(member)[0]"},{"line_number":399,"context_line":"            request_info \u003d {\u0027id\u0027: member_info[1],"},{"line_number":400,"context_line":"                            \u0027address\u0027: member_ip,"},{"line_number":401,"context_line":"                            \u0027protocol_port\u0027: member_port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"feae229d_b84ad1db","line":398,"range":{"start_line":394,"start_character":0,"end_line":398,"end_character":64},"updated":"2022-07-14 06:21:57.000000000","message":"this is a bit ugly split, perhaps doing it the other way around:\nmember_ip, member_pot, subnet_id \u003d (\n    self._ovn_helper._extract_member_info(member)","commit_id":"6c46c79ab13d38379409f7bc85bdbc4c5e232d93"},{"author":{"_account_id":34451,"name":"Fernando Royo","email":"froyo@redhat.com","username":"froyo"},"change_message_id":"6fe1fa887ad844872c5099d2c1029125754aa28c","unresolved":false,"context_lines":[{"line_number":391,"context_line":""},{"line_number":392,"context_line":"        for member in members_to_delete:"},{"line_number":393,"context_line":"            member_info \u003d member.split(\u0027_\u0027)"},{"line_number":394,"context_line":"            ("},{"line_number":395,"context_line":"                member_ip,"},{"line_number":396,"context_line":"                member_port,"},{"line_number":397,"context_line":"                subnet_id,"},{"line_number":398,"context_line":"            ) \u003d self._ovn_helper._extract_member_info(member)[0]"},{"line_number":399,"context_line":"            request_info \u003d {\u0027id\u0027: member_info[1],"},{"line_number":400,"context_line":"                            \u0027address\u0027: member_ip,"},{"line_number":401,"context_line":"                            \u0027protocol_port\u0027: member_port,"}],"source_content_type":"text/x-python","patch_set":1,"id":"af53738e_869a9793","line":398,"range":{"start_line":394,"start_character":0,"end_line":398,"end_character":64},"in_reply_to":"feae229d_b84ad1db","updated":"2022-07-14 07:12:18.000000000","message":"Ack","commit_id":"6c46c79ab13d38379409f7bc85bdbc4c5e232d93"}]}