)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cd891052147533d331a98dcd8801dc4503dc579a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     ZhouHeng \u003czhouhenglc@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-06-28 03:05:44 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Set result when lswitch port exist"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #2025202"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a2c86a8b_64abdab4","line":7,"updated":"2023-06-28 03:24:25.000000000","message":"Thanks for opening the bug. Can you just put a small description here of the failure? Something about it was found running neutron-ovn-db-sync-util, etc. Thanks","commit_id":"3d9c57b59e595259c3edc2e6a0779a38713134ff"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"d82b76860e6907153a5e21cca2ec207ce09b9708","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ZhouHeng \u003czhouhenglc@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-06-28 03:05:44 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Set result when lswitch port exist"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #2025202"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9cb61cf3_765b7303","line":7,"in_reply_to":"a2c86a8b_64abdab4","updated":"2023-06-28 08:51:05.000000000","message":"hi Brian, description updated.","commit_id":"3d9c57b59e595259c3edc2e6a0779a38713134ff"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f5c791f005d4556c6512c7d4815732245db7f683","unresolved":true,"context_lines":[{"line_number":9,"context_line":"A TypeError was thrown during a synchronization"},{"line_number":10,"context_line":"command(neutron-ovn-db-sync-util) execution. From the code[1][2], it"},{"line_number":11,"context_line":"can be seen. The result of the AddLSwitchPortCommand command will be"},{"line_number":12,"context_line":"passed as a parameter to the UpdateLSwitchQosOptionsCommand. But if"},{"line_number":13,"context_line":"the logical switch port exists, the result will not be set. Therefore,"},{"line_number":14,"context_line":"when the UpdateLSwitchQosOptionsCommand is executed, the port_id will"},{"line_number":15,"context_line":"not be obtained, thereby throwing an exception TypeError."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"609d7b0a_a87de9fd","line":12,"range":{"start_line":12,"start_character":29,"end_line":12,"end_character":59},"updated":"2023-07-03 13:46:30.000000000","message":"This method doesn\u0027t exist: https://codesearch.openstack.org/?q\u003dUpdateLSwitchQosOptionsCommand\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d","commit_id":"56d644f5c6728dc2d68ee59a3fa23850e5b38900"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"8bab95754eb79a43a28b1a2d9afe392fc88a9c4c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"A TypeError was thrown during a synchronization"},{"line_number":10,"context_line":"command(neutron-ovn-db-sync-util) execution. From the code[1][2], it"},{"line_number":11,"context_line":"can be seen. The result of the AddLSwitchPortCommand command will be"},{"line_number":12,"context_line":"passed as a parameter to the UpdateLSwitchQosOptionsCommand. But if"},{"line_number":13,"context_line":"the logical switch port exists, the result will not be set. Therefore,"},{"line_number":14,"context_line":"when the UpdateLSwitchQosOptionsCommand is executed, the port_id will"},{"line_number":15,"context_line":"not be obtained, thereby throwing an exception TypeError."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"be0cac26_e02d5d77","line":12,"range":{"start_line":12,"start_character":29,"end_line":12,"end_character":59},"in_reply_to":"609d7b0a_a87de9fd","updated":"2023-07-04 11:26:17.000000000","message":"Done. update to UpdateLSwitchPortQosOptionsCommand.","commit_id":"56d644f5c6728dc2d68ee59a3fa23850e5b38900"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3e7c170fe03da6ace50ab127b2713d7b46442655","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b57e3ad2_a76543c4","updated":"2023-04-28 13:50:21.000000000","message":"Is there a bug filed for this?","commit_id":"80fd33ec917c4f0480cf1852ebe1147e80549988"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"739b398a31ee4b75b80ed5932a5539c142b5383c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f6901733_ff967fbd","in_reply_to":"b57e3ad2_a76543c4","updated":"2023-05-24 06:53:35.000000000","message":"I have encountered this issue in my environment. When executing neutron-ovn-db-sync-util to update the logical port QoS(UpdateLSwitchQosOptionsCommand) and lookup Logical_Switch_Port, an error was reported port_id is None. By updating the above code, it can be executed correctly. At that time, there was no careful observation of the OVN data situation, and this issue did not recur later.\nBut I think the same behavior should be the same. When a new logical port is created, result will be set, and it should also be set when it already exists.","commit_id":"80fd33ec917c4f0480cf1852ebe1147e80549988"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"7629cde845c83f89eb50f149bdc648c9def8c4c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"23c2a464_3b0327a8","updated":"2023-06-28 14:18:28.000000000","message":"Thanks ZhouHeng","commit_id":"56d644f5c6728dc2d68ee59a3fa23850e5b38900"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"f5c791f005d4556c6512c7d4815732245db7f683","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1184e4c7_90cee719","updated":"2023-07-03 13:46:30.000000000","message":"Very good catch.\n\nPlease, add a functional test in test_ovn_client testing this precisely, that when you call the OVNClient.create_port twice, the qos driver doesn\u0027t fail.","commit_id":"56d644f5c6728dc2d68ee59a3fa23850e5b38900"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"ff1c01f1859a4ecd180e7cddb9081259d40162b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b2438804_ac4ab3fd","updated":"2023-06-28 08:50:29.000000000","message":"recheck test_hotplug_nic failed","commit_id":"56d644f5c6728dc2d68ee59a3fa23850e5b38900"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"eed7ca0750ecfc74dfc90a8b4cf327444227c283","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"779ae913_633bdf9c","updated":"2023-07-04 07:29:00.000000000","message":"Ah, once merged, we must backport it.","commit_id":"c4f583048c61d16ebbefc3a2f8734c1247136381"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"37ef3cc267630d40fa0f78a7139a11132fa50fc9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"aed90918_8267279a","updated":"2023-07-04 07:28:34.000000000","message":"If CI passes, I\u0027m OK with this patch. Again, good catch!","commit_id":"c4f583048c61d16ebbefc3a2f8734c1247136381"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3c4040413dbe1293e4fc6e694f175b2704ecf601","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"386d7071_4227f63d","updated":"2023-07-04 09:04:03.000000000","message":"New FT test is failing","commit_id":"c4f583048c61d16ebbefc3a2f8734c1247136381"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/commands.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"a085115d4256b08ea89471be658413c17043777c","unresolved":true,"context_lines":[{"line_number":122,"context_line":"                                         \u0027Logical_Switch_Port\u0027, \u0027name\u0027,"},{"line_number":123,"context_line":"                                         self.lport, None)"},{"line_number":124,"context_line":"            if port:"},{"line_number":125,"context_line":"                self.result \u003d port.uuid"},{"line_number":126,"context_line":"                return"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        port \u003d txn.insert(self.api._tables[\u0027Logical_Switch_Port\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"7d29afd6_a1843211","line":125,"updated":"2023-06-24 23:18:47.000000000","message":"So looking at this again, it does seem like the correct thing to do.\n\nThe reason I asked for a bug is that we will need it if we want to backport this fix, which we probably do. Even if you can just give an example of the commands you ran and the failure you saw it would help.","commit_id":"5e142641ad83b7fc56b23ca7a4fae1d8f7a2c0c0"},{"author":{"_account_id":30380,"name":"ZhouHeng","email":"zhouhenglc@inspur.com","username":"zhouhenglc"},"change_message_id":"12790edd4ce3b8c724305444e7c7728e65009ea2","unresolved":false,"context_lines":[{"line_number":122,"context_line":"                                         \u0027Logical_Switch_Port\u0027, \u0027name\u0027,"},{"line_number":123,"context_line":"                                         self.lport, None)"},{"line_number":124,"context_line":"            if port:"},{"line_number":125,"context_line":"                self.result \u003d port.uuid"},{"line_number":126,"context_line":"                return"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        port \u003d txn.insert(self.api._tables[\u0027Logical_Switch_Port\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"0882d48f_3668c307","line":125,"in_reply_to":"7d29afd6_a1843211","updated":"2023-06-28 03:06:35.000000000","message":"hi Brian, a bug has been submitted.","commit_id":"5e142641ad83b7fc56b23ca7a4fae1d8f7a2c0c0"}]}