)]}'
{"ovsdbapp/schema/ovn_northbound/commands.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"9c710d00a3466168740eee35068fa66c57cda3b1","unresolved":false,"context_lines":[{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def run_idl(self, txn):"},{"line_number":290,"context_line":"        try:"},{"line_number":291,"context_line":"            ls \u003d idlutils.row_by_value(self.api.idl, \u0027Logical_Switch\u0027, \u0027name\u0027,"},{"line_number":292,"context_line":"                                       self.switch)"},{"line_number":293,"context_line":"        except idlutils.RowNotFound:"},{"line_number":294,"context_line":"            if self.if_exists:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_ebd082cc","line":291,"updated":"2020-07-27 16:41:16.000000000","message":"switching from lookup() to row_by_value() means that anything that passes a uuid instead of a name will break. with indexing, it will fail because the type is different uuid vs string in the index. without, i think it\u0027d just be a RowNotFound.","commit_id":"4cb60b7a2fd8e04e165d7387586f09f83b2f4059"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"85eb34f0aaae62d0e721169d7a2241eb6aab5734","unresolved":false,"context_lines":[{"line_number":288,"context_line":""},{"line_number":289,"context_line":"    def run_idl(self, txn):"},{"line_number":290,"context_line":"        try:"},{"line_number":291,"context_line":"            ls \u003d idlutils.row_by_value(self.api.idl, \u0027Logical_Switch\u0027, \u0027name\u0027,"},{"line_number":292,"context_line":"                                       self.switch)"},{"line_number":293,"context_line":"        except idlutils.RowNotFound:"},{"line_number":294,"context_line":"            if self.if_exists:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_4be1ae9a","line":291,"in_reply_to":"9f560f44_ebd082cc","updated":"2020-07-27 16:45:42.000000000","message":"Now I understand it. Cool, I\u0027ll change it now.\n\nI think all Neutron commands pass the network name, but of course, this library should not only support Neutron.","commit_id":"4cb60b7a2fd8e04e165d7387586f09f83b2f4059"}],"ovsdbapp/tests/functional/schema/ovn_northbound/test_impl_idl.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ed8cad52b5ab70425b95186d3d35de606e654414","unresolved":false,"context_lines":[{"line_number":310,"context_line":"        self.assertRaises(TypeError, self.api.qos_del, self.switch.uuid,"},{"line_number":311,"context_line":"                          priority\u003d0)"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"    #def test_qos_del_ls_not_present_if_exists_true(self):"},{"line_number":314,"context_line":"    #    self.api.qos_del(\u0027some_other_ls\u0027).execute(check_error\u003dTrue)"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"    #def test_qos_del_ls_not_present_if_exists_false(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_eb836203","line":313,"updated":"2020-07-27 16:25:09.000000000","message":"pep8: E265 block comment should start with \u0027# \u0027","commit_id":"0b70cfbff8fe9e4468a82df10fefd5f7e80ffc45"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"ed8cad52b5ab70425b95186d3d35de606e654414","unresolved":false,"context_lines":[{"line_number":313,"context_line":"    #def test_qos_del_ls_not_present_if_exists_true(self):"},{"line_number":314,"context_line":"    #    self.api.qos_del(\u0027some_other_ls\u0027).execute(check_error\u003dTrue)"},{"line_number":315,"context_line":""},{"line_number":316,"context_line":"    #def test_qos_del_ls_not_present_if_exists_false(self):"},{"line_number":317,"context_line":"    #    cmd \u003d self.api.qos_del(\u0027some_other_ls\u0027, if_exists\u003dFalse)"},{"line_number":318,"context_line":"    #    self.assertRaises(RuntimeError, cmd.execute, check_error\u003dTrue)"},{"line_number":319,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_cb881e21","line":316,"updated":"2020-07-27 16:25:09.000000000","message":"pep8: E265 block comment should start with \u0027# \u0027","commit_id":"0b70cfbff8fe9e4468a82df10fefd5f7e80ffc45"}]}