ML2 plugin does not set correct MTU for multiple provider networks

Bug #1617284 reported by bernd.herth on 2016-08-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Undecided
Hong Hui Xiao

Bug Description

When using multiple provider networks type vlan the setting :
"physical_network_mtus = net1:1500, net2:9000" is not taken only the "global_physnet_mtu=9000"
setting is taken.

It seems that in /neutron/plugins/ml2/plugin.py in function _get_network_mtu () the call to type.driver.get_mtu(provider.PHYSICAL_NETWORK) is wrong.

It should be :

mtu = type.driver.get_mtu(s[provider.PHYSICAL_NETWORK])

instead to read out the propper network name.

Thanks

Tags: mtu Edit Tag help

Fix proposed to branch: master
Review: https://review.openstack.org/361139

Changed in neutron:
assignee: nobody → Hong Hui Xiao (xiaohhui)
status: New → In Progress

Reviewed: https://review.openstack.org/361139
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=0d96b9b43cdfb6f354241f8147b0e1604aa04ec4
Submitter: Jenkins
Branch: master

commit 0d96b9b43cdfb6f354241f8147b0e1604aa04ec4
Author: Hong Hui Xiao <email address hidden>
Date: Fri Aug 26 08:20:58 2016 -0400

    Fix passing error physical network for get_mtu

    The network's physical network should be used to get the network's mtu.

    Change-Id: Ib68cf7a64332c6a4b3df7b5d0a41922421b58dba
    Closes-bug: #1617284

Changed in neutron:
status: In Progress → Fix Released

This issue was fixed in the openstack/neutron 9.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers