)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"cf74e0455ace61b5df40a4c05c003ba3392b4741","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9e189e4b_6d7f9aa2","updated":"2022-02-24 07:37:59.000000000","message":"Looks good","commit_id":"c773210b252cd317d1abf018f49656f4c623d16f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"25cec6c75efd7a7069ff3fa77a5534fe09cd516a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"288ea177_d322353b","updated":"2022-02-24 06:41:33.000000000","message":"Thanks","commit_id":"c773210b252cd317d1abf018f49656f4c623d16f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"91acd0fc24f89304e7607bf803d2f1659e21ca2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"0e98c82b_2edf23c8","updated":"2022-03-02 07:23:52.000000000","message":"recheck functional test failure test_good_address_allocation and kernel panic issue workaround https://review.opendev.org/c/openstack/neutron/+/830808 merged","commit_id":"a51aff743c4659347f39efa1fd36ed883a37766f"}],"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":"df461910f2c71456fe5a2404b44a037c04439fcd","unresolved":true,"context_lines":[{"line_number":1495,"context_line":"                grps \u003d self._get_router_ports(admin_context, port[\u0027device_id\u0027])"},{"line_number":1496,"context_line":"            except l3_exc.RouterNotFound:"},{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                pass"},{"line_number":1499,"context_line":"            else:"},{"line_number":1500,"context_line":"                network_ids \u003d set([port[\u0027network_id\u0027] for port in grps])"},{"line_number":1501,"context_line":"                for net in self._plugin.get_networks(admin_context,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9b351e12_eac76ea7","line":1498,"updated":"2022-02-23 17:01:43.000000000","message":"Yatin asked on the review to add a debug message here. I think it\u0027s a good idea.","commit_id":"7dcec70de9a02bec03d9d441851bade35d1733b9"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"353d4991be699e1409fdbcd3decc45cfafd0dc22","unresolved":false,"context_lines":[{"line_number":1495,"context_line":"                grps \u003d self._get_router_ports(admin_context, port[\u0027device_id\u0027])"},{"line_number":1496,"context_line":"            except l3_exc.RouterNotFound:"},{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                pass"},{"line_number":1499,"context_line":"            else:"},{"line_number":1500,"context_line":"                network_ids \u003d set([port[\u0027network_id\u0027] for port in grps])"},{"line_number":1501,"context_line":"                for net in self._plugin.get_networks(admin_context,"}],"source_content_type":"text/x-python","patch_set":1,"id":"4b1f2147_56f8ad76","line":1498,"in_reply_to":"9b351e12_eac76ea7","updated":"2022-02-23 17:22:20.000000000","message":"Done","commit_id":"7dcec70de9a02bec03d9d441851bade35d1733b9"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"df461910f2c71456fe5a2404b44a037c04439fcd","unresolved":true,"context_lines":[{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                pass"},{"line_number":1499,"context_line":"            else:"},{"line_number":1500,"context_line":"                network_ids \u003d set([port[\u0027network_id\u0027] for port in grps])"},{"line_number":1501,"context_line":"                for net in self._plugin.get_networks(admin_context,"},{"line_number":1502,"context_line":"                                            filters\u003d{\u0027id\u0027: network_ids}):"},{"line_number":1503,"context_line":"                    if net[\u0027mtu\u0027] \u003e network[\u0027mtu\u0027]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"0325bd68_59ac475c","line":1500,"range":{"start_line":1500,"start_character":30,"end_line":1500,"end_character":35},"updated":"2022-02-23 17:01:43.000000000","message":"I know it doesn\u0027t belong here but I thought since we\u0027re touching it perhaps you can change it to set comprehension instead of creating a list and typing it to set afterwards.","commit_id":"7dcec70de9a02bec03d9d441851bade35d1733b9"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"353d4991be699e1409fdbcd3decc45cfafd0dc22","unresolved":false,"context_lines":[{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                pass"},{"line_number":1499,"context_line":"            else:"},{"line_number":1500,"context_line":"                network_ids \u003d set([port[\u0027network_id\u0027] for port in grps])"},{"line_number":1501,"context_line":"                for net in self._plugin.get_networks(admin_context,"},{"line_number":1502,"context_line":"                                            filters\u003d{\u0027id\u0027: network_ids}):"},{"line_number":1503,"context_line":"                    if net[\u0027mtu\u0027] \u003e network[\u0027mtu\u0027]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"672dfb5f_26bb651c","line":1500,"range":{"start_line":1500,"start_character":30,"end_line":1500,"end_character":35},"in_reply_to":"0325bd68_59ac475c","updated":"2022-02-23 17:22:20.000000000","message":"Done!","commit_id":"7dcec70de9a02bec03d9d441851bade35d1733b9"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"9940345d6ce5158ff5872e2a96e28138b19bc78a","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":"                grps \u003d self._get_router_ports(admin_context, port[\u0027device_id\u0027])"},{"line_number":1496,"context_line":"            except l3_exc.RouterNotFound:"},{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                LOG.debug(\"Router %s not found\", port[\u0027device_id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"2e4127a3_acb825be","line":1495,"range":{"start_line":1495,"start_character":16,"end_line":1495,"end_character":20},"updated":"2022-02-24 12:38:22.000000000","message":"nit: instead of grps some more meaningful shortening would be better.\nr_ports, router_ports or similar","commit_id":"c773210b252cd317d1abf018f49656f4c623d16f"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"8d18ab699251c5c70b2a57e22a5458639b7f20cb","unresolved":false,"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":"                grps \u003d self._get_router_ports(admin_context, port[\u0027device_id\u0027])"},{"line_number":1496,"context_line":"            except l3_exc.RouterNotFound:"},{"line_number":1497,"context_line":"                # Don\u0027t add any mtu info if the router no longer exists"},{"line_number":1498,"context_line":"                LOG.debug(\"Router %s not found\", port[\u0027device_id\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"37f1d6e1_c1d235d1","line":1495,"range":{"start_line":1495,"start_character":16,"end_line":1495,"end_character":20},"in_reply_to":"2e4127a3_acb825be","updated":"2022-03-01 08:53:54.000000000","message":"Done","commit_id":"c773210b252cd317d1abf018f49656f4c623d16f"}]}