)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"7abcbd13ad7b8efad58489c22ce92945da900fc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f267b3cc_39f087ab","updated":"2022-03-30 13:45:44.000000000","message":"Hi @Terry, can you take a look on this, please? Thanks.","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"6cbae00cadb60151a17bc9da89431984ec67a81c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e97750c1_0c130899","updated":"2022-07-29 18:18:15.000000000","message":"Went ahead and fixed doc nits, and merging since Rodolfo had a previous +2 and no code changes. Thanks!","commit_id":"f3c5da5402ea2d1c129684a84c91dbc72556f14f"}],"ovsdbapp/schema/ovn_northbound/api.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"6cbae00cadb60151a17bc9da89431984ec67a81c","unresolved":false,"context_lines":[{"line_number":911,"context_line":""},{"line_number":912,"context_line":"        :param lb:        The name or uuid of a load balancer"},{"line_number":913,"context_line":"        :type lb:         string or uuid.UUID"},{"line_number":914,"context_line":"        :param hc:        uuid of a load balancer"},{"line_number":915,"context_line":"        :type hc:         uuid.UUID"},{"line_number":916,"context_line":"        :param if_exists: If True, don\u0027t fail if the hc doesn\u0027t exist"},{"line_number":917,"context_line":"        :type if_exists:  boolean"},{"line_number":918,"context_line":"        :returns:         :class:`Command` with no result"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f0c0cf7_8e6d5447","line":915,"range":{"start_line":914,"start_character":0,"end_line":915,"end_character":35},"updated":"2022-07-29 18:18:15.000000000","message":"the parameter is hc_uuid, and is presumably the uuid of a health check","commit_id":"25c24976edb0c404408d9eccd14ba3d551a44e00"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"6cbae00cadb60151a17bc9da89431984ec67a81c","unresolved":false,"context_lines":[{"line_number":951,"context_line":"    def health_check_set_options(self, hc_uuid, **options):"},{"line_number":952,"context_line":"        \"\"\"Set options to the \u0027health_check\u0027"},{"line_number":953,"context_line":""},{"line_number":954,"context_line":"        :param hc:      uuid of the health check"},{"line_number":955,"context_line":"        :type hc:       uuid.UUID"},{"line_number":956,"context_line":"        :param options: keys and values for the port \u0027options\u0027 dict"},{"line_number":957,"context_line":"        :type options:  key: string, value: string"}],"source_content_type":"text/x-python","patch_set":2,"id":"48b86b1d_a66f28cc","line":954,"range":{"start_line":954,"start_character":15,"end_line":954,"end_character":17},"updated":"2022-07-29 18:18:15.000000000","message":"same","commit_id":"25c24976edb0c404408d9eccd14ba3d551a44e00"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"6cbae00cadb60151a17bc9da89431984ec67a81c","unresolved":false,"context_lines":[{"line_number":962,"context_line":"    def health_check_get_options(self, hc_uuid):"},{"line_number":963,"context_line":"        \"\"\"Get the options for \u0027health_check\u0027"},{"line_number":964,"context_line":""},{"line_number":965,"context_line":"        :param hc: uuid of the health check"},{"line_number":966,"context_line":"        :type hc:  uuid.UUID"},{"line_number":967,"context_line":"        :returns:  :class:`Command` with dict result"},{"line_number":968,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"067a8350_9c0d6a6a","line":965,"range":{"start_line":965,"start_character":15,"end_line":965,"end_character":17},"updated":"2022-07-29 18:18:15.000000000","message":"same","commit_id":"25c24976edb0c404408d9eccd14ba3d551a44e00"}],"ovsdbapp/schema/ovn_northbound/commands.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"79b24384548b0e0efaee1b13823fdd393d80d71e","unresolved":true,"context_lines":[{"line_number":760,"context_line":"class DhcpOptionsGetCommand(cmd.BaseGetRowCommand):"},{"line_number":761,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":""},{"line_number":764,"context_line":"class DhcpOptionsSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":765,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":766,"context_line":""},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"class DhcpOptionsGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":769,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":770,"context_line":""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"class LrAddCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9c448745_f12c62fb","line":769,"range":{"start_line":763,"start_character":0,"end_line":769,"end_character":26},"updated":"2022-07-12 11:46:40.000000000","message":"Nice but unrelated","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"4a54608360434e2a0d99395a69ca2d84eb3f7dd1","unresolved":true,"context_lines":[{"line_number":760,"context_line":"class DhcpOptionsGetCommand(cmd.BaseGetRowCommand):"},{"line_number":761,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":""},{"line_number":764,"context_line":"class DhcpOptionsSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":765,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":766,"context_line":""},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"class DhcpOptionsGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":769,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":770,"context_line":""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"class LrAddCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fbd302d7_b4ad0285","line":769,"range":{"start_line":763,"start_character":0,"end_line":769,"end_character":26},"in_reply_to":"9c448745_f12c62fb","updated":"2022-07-12 14:23:18.000000000","message":"Should I do this in the separate PR?","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"42923be1959034e151e74b345659a1864f902972","unresolved":false,"context_lines":[{"line_number":760,"context_line":"class DhcpOptionsGetCommand(cmd.BaseGetRowCommand):"},{"line_number":761,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":""},{"line_number":764,"context_line":"class DhcpOptionsSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":765,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":766,"context_line":""},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"class DhcpOptionsGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":769,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":770,"context_line":""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"class LrAddCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"8cfe7881_f101087e","line":769,"range":{"start_line":763,"start_character":0,"end_line":769,"end_character":26},"in_reply_to":"d738ed78_17887634","updated":"2022-07-12 18:31:00.000000000","message":"Done","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ae16896ec1c6621742c0c5ca7dc5bebea4382fe0","unresolved":true,"context_lines":[{"line_number":760,"context_line":"class DhcpOptionsGetCommand(cmd.BaseGetRowCommand):"},{"line_number":761,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":""},{"line_number":764,"context_line":"class DhcpOptionsSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":765,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":766,"context_line":""},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"class DhcpOptionsGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":769,"context_line":"    table \u003d \u0027DHCP_Options\u0027"},{"line_number":770,"context_line":""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"class LrAddCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"d738ed78_17887634","line":769,"range":{"start_line":763,"start_character":0,"end_line":769,"end_character":26},"in_reply_to":"fbd302d7_b4ad0285","updated":"2022-07-12 15:31:13.000000000","message":"yes, it should, in a \"Trivial-Fix\" patch","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"79b24384548b0e0efaee1b13823fdd393d80d71e","unresolved":true,"context_lines":[{"line_number":927,"context_line":"        self.result \u003d next(iter(lrp.enabled), True)"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":""},{"line_number":930,"context_line":"class LrpSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":931,"context_line":"    table \u003d \u0027Logical_Router_Port\u0027"},{"line_number":932,"context_line":""},{"line_number":933,"context_line":""},{"line_number":934,"context_line":"class LrpGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":935,"context_line":"    table \u003d \u0027Logical_Router_Port\u0027"},{"line_number":936,"context_line":""},{"line_number":937,"context_line":""},{"line_number":938,"context_line":"class LrpSetGatewayChassisCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"f8e81f09_b7c19cd6","line":935,"range":{"start_line":930,"start_character":0,"end_line":935,"end_character":33},"updated":"2022-07-12 11:46:40.000000000","message":"ditto, unrelated","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"42923be1959034e151e74b345659a1864f902972","unresolved":false,"context_lines":[{"line_number":927,"context_line":"        self.result \u003d next(iter(lrp.enabled), True)"},{"line_number":928,"context_line":""},{"line_number":929,"context_line":""},{"line_number":930,"context_line":"class LrpSetOptionsCommand(cmd.BaseSetOptionsCommand):"},{"line_number":931,"context_line":"    table \u003d \u0027Logical_Router_Port\u0027"},{"line_number":932,"context_line":""},{"line_number":933,"context_line":""},{"line_number":934,"context_line":"class LrpGetOptionsCommand(cmd.BaseGetOptionsCommand):"},{"line_number":935,"context_line":"    table \u003d \u0027Logical_Router_Port\u0027"},{"line_number":936,"context_line":""},{"line_number":937,"context_line":""},{"line_number":938,"context_line":"class LrpSetGatewayChassisCommand(cmd.BaseCommand):"}],"source_content_type":"text/x-python","patch_set":1,"id":"14174a26_f411321c","line":935,"range":{"start_line":930,"start_character":0,"end_line":935,"end_character":33},"in_reply_to":"f8e81f09_b7c19cd6","updated":"2022-07-12 18:31:00.000000000","message":"Done","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"}],"ovsdbapp/tests/functional/schema/ovn_northbound/test_impl_idl.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"79b24384548b0e0efaee1b13823fdd393d80d71e","unresolved":true,"context_lines":[{"line_number":1706,"context_line":""},{"line_number":1707,"context_line":"        self.api.lb_del_health_check(val, hc.uuid).execute(check_error\u003dTrue)"},{"line_number":1708,"context_line":"        self.assertEqual(len(lb.health_check), 0)"},{"line_number":1709,"context_line":"        self.assertNotIn(hc.uuid,"},{"line_number":1710,"context_line":"                         self.api.tables[\u0027Load_Balancer_Health_Check\u0027].rows)"},{"line_number":1711,"context_line":""},{"line_number":1712,"context_line":"    def test_lb_add_del_health_check_uuid(self):"},{"line_number":1713,"context_line":"        self._test_lb_add_del_health_check(\u0027uuid\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf6f76e5_4bd73572","line":1710,"range":{"start_line":1709,"start_character":8,"end_line":1710,"end_character":76},"updated":"2022-07-12 11:46:40.000000000","message":"Nice! Check the LBHC has been deleted too.","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"cead018fb838a2a915adf18bdeb20c1e4503314e","unresolved":false,"context_lines":[{"line_number":1706,"context_line":""},{"line_number":1707,"context_line":"        self.api.lb_del_health_check(val, hc.uuid).execute(check_error\u003dTrue)"},{"line_number":1708,"context_line":"        self.assertEqual(len(lb.health_check), 0)"},{"line_number":1709,"context_line":"        self.assertNotIn(hc.uuid,"},{"line_number":1710,"context_line":"                         self.api.tables[\u0027Load_Balancer_Health_Check\u0027].rows)"},{"line_number":1711,"context_line":""},{"line_number":1712,"context_line":"    def test_lb_add_del_health_check_uuid(self):"},{"line_number":1713,"context_line":"        self._test_lb_add_del_health_check(\u0027uuid\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"19b21c23_703f85a5","line":1710,"range":{"start_line":1709,"start_character":8,"end_line":1710,"end_character":76},"in_reply_to":"75291c1d_5b46c657","updated":"2022-07-13 09:54:10.000000000","message":"Done","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"},{"author":{"_account_id":33871,"name":"Anton Vazhnetsov","display_name":"Anton Vazhnetsov","email":"dragen15051@gmail.com","username":"0x5b"},"change_message_id":"f38e4a962a2f8c3abe55a9a110945c88f7fad53d","unresolved":true,"context_lines":[{"line_number":1706,"context_line":""},{"line_number":1707,"context_line":"        self.api.lb_del_health_check(val, hc.uuid).execute(check_error\u003dTrue)"},{"line_number":1708,"context_line":"        self.assertEqual(len(lb.health_check), 0)"},{"line_number":1709,"context_line":"        self.assertNotIn(hc.uuid,"},{"line_number":1710,"context_line":"                         self.api.tables[\u0027Load_Balancer_Health_Check\u0027].rows)"},{"line_number":1711,"context_line":""},{"line_number":1712,"context_line":"    def test_lb_add_del_health_check_uuid(self):"},{"line_number":1713,"context_line":"        self._test_lb_add_del_health_check(\u0027uuid\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"75291c1d_5b46c657","line":1710,"range":{"start_line":1709,"start_character":8,"end_line":1710,"end_character":76},"in_reply_to":"bf6f76e5_4bd73572","updated":"2022-07-12 16:30:42.000000000","message":"If you mean \u0027health_check\u0027 column from \u0027lb\u0027, it\u0027s checked on the previous line. If you mean record from \u0027Load_Balancer_Health_Check` table, I don\u0027t know how to check it in another way.","commit_id":"bf19e8e66fab3b4d2cfd77765f05d4752932e2c9"}]}