)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"8a17c010c2b4152c67afe21868f59407fc95567f","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[OVN] Handle RouterNotFound exception in set_gateway_mtu"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When handling multiple router update/delete petitions, if they are"},{"line_number":10,"context_line":"executed at the same time, there is a chance that the set_gateweay_mtu"},{"line_number":11,"context_line":"tries to update a no-longer-existing router, triggering a"},{"line_number":12,"context_line":"RouterNotFound error from the _get_router_ports function. Now, if there"},{"line_number":13,"context_line":"is not a router, the _get_router_ports function will return an empty"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"46c127dc_835afc25","line":10,"range":{"start_line":10,"start_character":58,"end_line":10,"end_character":66},"updated":"2022-02-22 18:36:46.000000000","message":"gateway","commit_id":"7c805dbb2565b2c35fd7e6f504c297d9902eb167"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"98536aec5a41a38a189738bfa2dde75cfe333db0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"[OVN] Handle RouterNotFound exception in set_gateway_mtu"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When handling multiple router update/delete petitions, if they are"},{"line_number":10,"context_line":"executed at the same time, there is a chance that the set_gateweay_mtu"},{"line_number":11,"context_line":"tries to update a no-longer-existing router, triggering a"},{"line_number":12,"context_line":"RouterNotFound error from the _get_router_ports function. Now, if there"},{"line_number":13,"context_line":"is not a router, the _get_router_ports function will return an empty"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6b79226c_cef741dd","line":10,"range":{"start_line":10,"start_character":58,"end_line":10,"end_character":66},"in_reply_to":"46c127dc_835afc25","updated":"2022-02-23 09:50:38.000000000","message":"Done","commit_id":"7c805dbb2565b2c35fd7e6f504c297d9902eb167"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"06bb007eb94a2b2c7a3bafe3dfdf69f346b28778","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0a49c23e_1d5ecebb","updated":"2022-02-23 07:33:09.000000000","message":"looks good! Perhaps worth to add a text where l3_exc.RouterNotFound is raised","commit_id":"7c805dbb2565b2c35fd7e6f504c297d9902eb167"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"98536aec5a41a38a189738bfa2dde75cfe333db0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1509754c_29d39a53","in_reply_to":"0a49c23e_1d5ecebb","updated":"2022-02-23 09:50:38.000000000","message":"Done! I think this is a good idea to prevent future regressions","commit_id":"7c805dbb2565b2c35fd7e6f504c297d9902eb167"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"f6e48e4a78b8149f46c709d12dfeca09fff4dbfd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4ccb8d67_3b6bb0a1","updated":"2022-02-23 10:26:06.000000000","message":"Thanks Elvira, LGTM!","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"5690f55e92aad3d0e9822e7cdb8863092db9c04b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f65aef21_f2680763","updated":"2022-02-23 10:27:47.000000000","message":"Thanks!","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"aad9af57ebefdee99e64fd3ba0a1f5aec7f6aa57","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"12eb7f09_1d0ea23e","updated":"2022-02-23 13:41:42.000000000","message":"recheck","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"16d8b88ddb310f86b4bb1adcf409671163310336","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"01e47d5e_bd7b2f70","updated":"2022-02-23 13:53:52.000000000","message":"recheck - kernel panic in guest vm","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"80279c0cba9ebc692aa8a55e7b0a1456ddd6a581","unresolved":true,"context_lines":[{"line_number":1492,"context_line":"            \u0027device_owner\u0027)"},{"line_number":1493,"context_line":"        if is_gw_port and ovn_conf.is_ovn_emit_need_to_frag_enabled():"},{"line_number":1494,"context_line":"            try:"},{"line_number":1495,"context_line":"                network_ids \u003d set([port[\u0027network_id\u0027] for port in"},{"line_number":1496,"context_line":"                    self._get_router_ports(admin_context, port[\u0027device_id\u0027])])"},{"line_number":1497,"context_line":"                for net in self._plugin.get_networks(admin_context,"},{"line_number":1498,"context_line":"                                            filters\u003d{\u0027id\u0027: network_ids}):"},{"line_number":1499,"context_line":"                    if net[\u0027mtu\u0027] \u003e network[\u0027mtu\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"4a76aa9a_252919e6","line":1496,"range":{"start_line":1495,"start_character":0,"end_line":1496,"end_character":78},"updated":"2022-02-23 13:54:57.000000000","message":"Shall we limit the scope of try block only to this call?","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"c19513f55fcef4017b6423a2c27c2eab8867f256","unresolved":true,"context_lines":[{"line_number":1501,"context_line":"                                network[\u0027mtu\u0027])"},{"line_number":1502,"context_line":"                        break"},{"line_number":1503,"context_line":"            except l3_exc.RouterNotFound:"},{"line_number":1504,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1505,"context_line":"                pass"},{"line_number":1506,"context_line":"        return options"},{"line_number":1507,"context_line":""},{"line_number":1508,"context_line":"    def _create_lrouter_port(self, context, router, port, txn\u003dNone):"}],"source_content_type":"text/x-python","patch_set":3,"id":"411a6d32_1c531fac","line":1505,"range":{"start_line":1504,"start_character":11,"end_line":1505,"end_character":20},"updated":"2022-02-23 13:42:54.000000000","message":"May be better to add some debug logs instead of just pass?","commit_id":"b2421b01e5bc1269458b421643c13b620cdaccaf"}]}