)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"e8d9eb4751cc206a44b7210aa13f56dd8b441f24","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The smartnic port\u0027s MTU should be set according to the network\u0027s MTU"},{"line_number":10,"context_line":"which the port belongs to."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ibcc29c998065da521b35e5845727794a68782db0"},{"line_number":13,"context_line":"Signed-off-by: Xiaoyu Min \u003cjackmin@nvidia.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f6b1bfe_62ef7b87","line":11,"updated":"2020-10-14 07:37:40.000000000","message":"Is there any bug related to that? If yes, please link to it, if not, please open one and link to it here too. It may help us track backports of that fix to stable branches as I think it may be needed","commit_id":"4b8bd49b5e5dd3896e386b73adf225a30da382e5"},{"author":{"_account_id":32461,"name":"Xiaoyu MIN","email":"jackmin@nvidia.com","username":"jackmin"},"change_message_id":"86ee88ac2b7e8d28c5c15483c23fbfcda4e8dc44","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The smartnic port\u0027s MTU should be set according to the network\u0027s MTU"},{"line_number":10,"context_line":"which the port belongs to."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ibcc29c998065da521b35e5845727794a68782db0"},{"line_number":13,"context_line":"Signed-off-by: Xiaoyu Min \u003cjackmin@nvidia.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7f6b1bfe_7d60afcf","line":11,"in_reply_to":"7f6b1bfe_62ef7b87","updated":"2020-10-15 02:43:31.000000000","message":"Sure, I will open a bug on it.","commit_id":"4b8bd49b5e5dd3896e386b73adf225a30da382e5"}],"neutron/plugins/ml2/drivers/openvswitch/agent/ovs_neutron_agent.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00f9975fe44576a8a76b1d709bc167790259b115","unresolved":false,"context_lines":[{"line_number":546,"context_line":"        for port_binding in port_data.get(\u0027bindings\u0027, []):"},{"line_number":547,"context_line":"            if port_binding[\u0027vnic_type\u0027] \u003d\u003d portbindings.VNIC_SMARTNIC:"},{"line_number":548,"context_line":"                if port_binding[\u0027host\u0027] \u003d\u003d self.conf.host:"},{"line_number":549,"context_line":"                    mtu \u003d self._get_network_mtu(port_data[\u0027network_id\u0027])"},{"line_number":550,"context_line":"                    local_link \u003d (port_binding[\u0027profile\u0027]"},{"line_number":551,"context_line":"                                  [\u0027local_link_information\u0027])"},{"line_number":552,"context_line":"                    if local_link:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f681702_58a50bb5","line":549,"updated":"2020-10-19 11:52:50.000000000","message":"why do we need a new variable here? you can use it directly in L559","commit_id":"d76b22cf048086653af61afabdf691262db4b317"},{"author":{"_account_id":32461,"name":"Xiaoyu MIN","email":"jackmin@nvidia.com","username":"jackmin"},"change_message_id":"afa8ae3cf211d40a44b733b9855fcd5da07ef11c","unresolved":false,"context_lines":[{"line_number":546,"context_line":"        for port_binding in port_data.get(\u0027bindings\u0027, []):"},{"line_number":547,"context_line":"            if port_binding[\u0027vnic_type\u0027] \u003d\u003d portbindings.VNIC_SMARTNIC:"},{"line_number":548,"context_line":"                if port_binding[\u0027host\u0027] \u003d\u003d self.conf.host:"},{"line_number":549,"context_line":"                    mtu \u003d self._get_network_mtu(port_data[\u0027network_id\u0027])"},{"line_number":550,"context_line":"                    local_link \u003d (port_binding[\u0027profile\u0027]"},{"line_number":551,"context_line":"                                  [\u0027local_link_information\u0027])"},{"line_number":552,"context_line":"                    if local_link:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f65232a_54884c1e","line":549,"in_reply_to":"5f681702_58a50bb5","updated":"2020-10-21 02:03:42.000000000","message":"Done","commit_id":"d76b22cf048086653af61afabdf691262db4b317"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00f9975fe44576a8a76b1d709bc167790259b115","unresolved":false,"context_lines":[{"line_number":619,"context_line":"            network\u003dnetwork, address\u003dstr(mac))"},{"line_number":620,"context_line":""},{"line_number":621,"context_line":"    def _add_port_to_updated_smartnic_ports(self, mac, vif_name, iface_id,"},{"line_number":622,"context_line":"                                            vif_type, vm_uuid\u003d\u0027\u0027, mtu\u003d1500):"},{"line_number":623,"context_line":"        self.updated_smartnic_ports.append({"},{"line_number":624,"context_line":"            \u0027mac\u0027: mac,"},{"line_number":625,"context_line":"            \u0027vm_uuid\u0027: vm_uuid,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f681702_580a2b8a","line":622,"range":{"start_line":622,"start_character":70,"end_line":622,"end_character":74},"updated":"2020-10-19 11:52:50.000000000","message":"This should be \"global_physnet_mtu\", not a value defined here.","commit_id":"d76b22cf048086653af61afabdf691262db4b317"},{"author":{"_account_id":32461,"name":"Xiaoyu MIN","email":"jackmin@nvidia.com","username":"jackmin"},"change_message_id":"afa8ae3cf211d40a44b733b9855fcd5da07ef11c","unresolved":false,"context_lines":[{"line_number":619,"context_line":"            network\u003dnetwork, address\u003dstr(mac))"},{"line_number":620,"context_line":""},{"line_number":621,"context_line":"    def _add_port_to_updated_smartnic_ports(self, mac, vif_name, iface_id,"},{"line_number":622,"context_line":"                                            vif_type, vm_uuid\u003d\u0027\u0027, mtu\u003d1500):"},{"line_number":623,"context_line":"        self.updated_smartnic_ports.append({"},{"line_number":624,"context_line":"            \u0027mac\u0027: mac,"},{"line_number":625,"context_line":"            \u0027vm_uuid\u0027: vm_uuid,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f65232a_b47c4021","line":622,"range":{"start_line":622,"start_character":70,"end_line":622,"end_character":74},"in_reply_to":"5f681702_580a2b8a","updated":"2020-10-21 02:03:42.000000000","message":"Done","commit_id":"d76b22cf048086653af61afabdf691262db4b317"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"00f9975fe44576a8a76b1d709bc167790259b115","unresolved":false,"context_lines":[{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        def _process_added_ports(smartnic_added_ports):"},{"line_number":748,"context_line":"            for smartnic_port in smartnic_added_ports:"},{"line_number":749,"context_line":"                mtu \u003d self._get_network_mtu(smartnic_port[\u0027network_id\u0027])"},{"line_number":750,"context_line":"                local_link \u003d (smartnic_port[\u0027binding:profile\u0027]"},{"line_number":751,"context_line":"                              [\u0027local_link_information\u0027])"},{"line_number":752,"context_line":"                if local_link:"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f681702_f83df72c","line":749,"updated":"2020-10-19 11:52:50.000000000","message":"nit: ditto, we don\u0027t need to create a new variable","commit_id":"d76b22cf048086653af61afabdf691262db4b317"},{"author":{"_account_id":32461,"name":"Xiaoyu MIN","email":"jackmin@nvidia.com","username":"jackmin"},"change_message_id":"afa8ae3cf211d40a44b733b9855fcd5da07ef11c","unresolved":false,"context_lines":[{"line_number":746,"context_line":""},{"line_number":747,"context_line":"        def _process_added_ports(smartnic_added_ports):"},{"line_number":748,"context_line":"            for smartnic_port in smartnic_added_ports:"},{"line_number":749,"context_line":"                mtu \u003d self._get_network_mtu(smartnic_port[\u0027network_id\u0027])"},{"line_number":750,"context_line":"                local_link \u003d (smartnic_port[\u0027binding:profile\u0027]"},{"line_number":751,"context_line":"                              [\u0027local_link_information\u0027])"},{"line_number":752,"context_line":"                if local_link:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f65232a_94794431","line":749,"in_reply_to":"5f681702_f83df72c","updated":"2020-10-21 02:03:42.000000000","message":"Done","commit_id":"d76b22cf048086653af61afabdf691262db4b317"}]}
