)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"aaa89f1c1b754eda84800b4179099a5aee181239","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7abd4e8a_62f47534","updated":"2025-06-23 11:40:14.000000000","message":"looks ok","commit_id":"b0add6a34c1935a5db4386fdb5d727ead8e479d1"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"178b1c8e8491ea91181175c7bb64105582b73eb7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8356492b_e3cde680","updated":"2025-07-02 19:47:44.000000000","message":"Thanks for the update, just some nits on the strings","commit_id":"523aaea1fb01e0a09d157c58f929e5c2c6bc4e53"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b2d71b85f82afff24bd8d772a1512dfc3ee4ff19","unresolved":true,"context_lines":[{"line_number":1358,"context_line":"        from the network. Does not apply to \"network\" owned ports."},{"line_number":1359,"context_line":"        \"\"\""},{"line_number":1360,"context_line":"        if self.mode !\u003d ovn_const.OVN_DB_SYNC_MODE_REPAIR:"},{"line_number":1361,"context_line":"            return"},{"line_number":1362,"context_line":""},{"line_number":1363,"context_line":"        LOG.debug(\u0027OVN-NB Sync port QoS policies started @ %s\u0027,"},{"line_number":1364,"context_line":"                  str(datetime.now()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"6ec371a3_ac4bb352","line":1361,"updated":"2025-06-23 18:11:01.000000000","message":"Similar to my comment in your other change, I think we should log there is some inconsistency here that needs to be fixed and not just return early.","commit_id":"b0add6a34c1935a5db4386fdb5d727ead8e479d1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"190152f12306f9396644bf35a5b8e47f8b0f725a","unresolved":false,"context_lines":[{"line_number":1358,"context_line":"        from the network. Does not apply to \"network\" owned ports."},{"line_number":1359,"context_line":"        \"\"\""},{"line_number":1360,"context_line":"        if self.mode !\u003d ovn_const.OVN_DB_SYNC_MODE_REPAIR:"},{"line_number":1361,"context_line":"            return"},{"line_number":1362,"context_line":""},{"line_number":1363,"context_line":"        LOG.debug(\u0027OVN-NB Sync port QoS policies started @ %s\u0027,"},{"line_number":1364,"context_line":"                  str(datetime.now()))"}],"source_content_type":"text/x-python","patch_set":1,"id":"f80ff2b5_50068cff","line":1361,"in_reply_to":"6ec371a3_ac4bb352","updated":"2025-06-25 10:41:31.000000000","message":"Same as the comment in the previous patch, these two methods must be executed when called from the maintenance method, in order to sync the OVN database, adding the missing QoS to the ports.","commit_id":"b0add6a34c1935a5db4386fdb5d727ead8e479d1"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"178b1c8e8491ea91181175c7bb64105582b73eb7","unresolved":true,"context_lines":[{"line_number":1375,"context_line":"        elif not (self.mode \u003d\u003d ovn_const.OVN_DB_SYNC_MODE_REPAIR or"},{"line_number":1376,"context_line":"                  self.is_maintenance):"},{"line_number":1377,"context_line":"            for port in _ports:"},{"line_number":1378,"context_line":"                LOG.warning(\u0027The port %s QoS is missing in the OVN NB \u0027"},{"line_number":1379,"context_line":"                            \u0027database\u0027, port[\u0027id\u0027])"},{"line_number":1380,"context_line":"        else:"},{"line_number":1381,"context_line":"            with self.ovn_api.transaction(check_error\u003dTrue) as txn:"}],"source_content_type":"text/x-python","patch_set":4,"id":"710ddb22_bc2a7373","line":1378,"range":{"start_line":1378,"start_character":41,"end_line":1378,"end_character":44},"updated":"2025-07-02 19:47:44.000000000","message":"nit: \"policy\" is missing from string, and can probably shrink the text:\n\n\u0027Port QoS policy missing in OVN NB DB, port_id\u003d%s\u0027","commit_id":"523aaea1fb01e0a09d157c58f929e5c2c6bc4e53"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e43b4bb863b05f13fd8e0aef259df67b65e983db","unresolved":false,"context_lines":[{"line_number":1375,"context_line":"        elif not (self.mode \u003d\u003d ovn_const.OVN_DB_SYNC_MODE_REPAIR or"},{"line_number":1376,"context_line":"                  self.is_maintenance):"},{"line_number":1377,"context_line":"            for port in _ports:"},{"line_number":1378,"context_line":"                LOG.warning(\u0027The port %s QoS is missing in the OVN NB \u0027"},{"line_number":1379,"context_line":"                            \u0027database\u0027, port[\u0027id\u0027])"},{"line_number":1380,"context_line":"        else:"},{"line_number":1381,"context_line":"            with self.ovn_api.transaction(check_error\u003dTrue) as txn:"}],"source_content_type":"text/x-python","patch_set":4,"id":"6e4bfd93_82fc5754","line":1378,"range":{"start_line":1378,"start_character":41,"end_line":1378,"end_character":44},"in_reply_to":"710ddb22_bc2a7373","updated":"2025-07-07 09:41:37.000000000","message":"Done","commit_id":"523aaea1fb01e0a09d157c58f929e5c2c6bc4e53"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"178b1c8e8491ea91181175c7bb64105582b73eb7","unresolved":true,"context_lines":[{"line_number":1404,"context_line":"                  self.is_maintenance):"},{"line_number":1405,"context_line":"            for fip in _fips:"},{"line_number":1406,"context_line":"                LOG.warning(\u0027The floating IP %s QoS is missing in the OVN NB \u0027"},{"line_number":1407,"context_line":"                            \u0027database\u0027, fip[\u0027id\u0027])"},{"line_number":1408,"context_line":"        else:"},{"line_number":1409,"context_line":"            with self.ovn_api.transaction(check_error\u003dTrue) as txn:"},{"line_number":1410,"context_line":"                for fip in _fips:"}],"source_content_type":"text/x-python","patch_set":4,"id":"0ecec4e0_2f87cc63","line":1407,"updated":"2025-07-02 19:47:44.000000000","message":"nit: \"policy\" is missing from string, and can probably shrink the text:\n\n\u0027Floating IP QoS policy missing in OVN NB DB, fip_id\u003d%s\u0027","commit_id":"523aaea1fb01e0a09d157c58f929e5c2c6bc4e53"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"e43b4bb863b05f13fd8e0aef259df67b65e983db","unresolved":false,"context_lines":[{"line_number":1404,"context_line":"                  self.is_maintenance):"},{"line_number":1405,"context_line":"            for fip in _fips:"},{"line_number":1406,"context_line":"                LOG.warning(\u0027The floating IP %s QoS is missing in the OVN NB \u0027"},{"line_number":1407,"context_line":"                            \u0027database\u0027, fip[\u0027id\u0027])"},{"line_number":1408,"context_line":"        else:"},{"line_number":1409,"context_line":"            with self.ovn_api.transaction(check_error\u003dTrue) as txn:"},{"line_number":1410,"context_line":"                for fip in _fips:"}],"source_content_type":"text/x-python","patch_set":4,"id":"960806e4_d43b7cd8","line":1407,"in_reply_to":"0ecec4e0_2f87cc63","updated":"2025-07-07 09:41:37.000000000","message":"Done","commit_id":"523aaea1fb01e0a09d157c58f929e5c2c6bc4e53"}]}