yahoo-eng-team team mailing list archive

[Hirofumi Ichihara <ichihara.hirofumi@xxxxxxxxxxxxx>]

This fix was provided in https://bugs.launchpad.net/neutron/+bug/1552089

** Changed in: neutron
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1582705

Title:
      Make agent interface plugging utilize network MTU

Status in neutron:
  Invalid

Bug description:
  https://review.openstack.org/305782
  Dear bug triager. This bug was created since a commit was marked with DOCIMPACT.
  Your project "openstack/neutron" is set up so that we directly report the documentation bugs against it. If this needs changing, the docimpact-group option needs to be added for the project. You can ask the OpenStack infra team (#openstack-infra on freenode) for help if you need to.

  commit c55aba1dba31c6730c80db0118286f1f9e84cd9b
  Author: Kevin Benton <kevin@xxxxxxxxxx>
  Date:   Mon Feb 22 16:41:45 2016 -0800

      Make agent interface plugging utilize network MTU
      
      This changes the 'plug' and 'plug_new' interfaces of the
      LinuxInterfaceDriver to accept an MTU argument. It then
      updates the dhcp agent and l3 agent to pass the MTU that
      is set on the network that the port belongs to. This allows
      it to take into account the overhead calculations that are
      done for encapsulation types.
      
      It's necessary for the L3 agent to have the MTU because it
      must recognize when fragmentation is needed so it can fragment
      or generate an ICMP error.
      
      It's necessary for the DHCP agent to have the MTU so it doesn't
      interfere when it plugs into a bridge with a larger than 1500
      MTU (the bridge would reduce its MTU to match the agent).
      
      If an operator sets 'network_device_mtu', the value of that
      will be used instead to preserve previous behavior.
      
      Conflicts:
      	neutron/agent/l3/dvr_edge_ha_router.py
      	neutron/agent/l3/dvr_edge_router.py
      	neutron/agent/l3/ha_router.py
      	neutron/agent/linux/interface.py
      	neutron/tests/functional/agent/l3/test_dvr_router.py
      	neutron/tests/functional/agent/test_dhcp_agent.py
      
      Additional modifications for Liberty:
      - test_dvr_router_lifecycle_ha_with_snat_with_fips_nmtu renamed into
        test_dvr_router_lifecycle_without_ha_with_snat_with_fips_nmtu,
      - the test validates DVR without HA.
      
      Reason for the change: Liberty does not support DVR + HA routers (the
      test raises DvrHaRouterNotSupported without those modifications).
      
      Closes-Bug: #1549470
      Closes-Bug: #1542108
      Closes-Bug: #1542475
      DocImpact: Neutron agents now support arbitrary MTU
                 configurations on each network (including
                 jumbo frames). This is accomplished by checking
                 the MTU value defined for each network on which
                 it is wiring VIFs.
      Co-Authored-By: Matt Kassawara <mkassawara@xxxxxxxxx>
      (cherry picked from commit 4df8d9a7016ab20fce235833d792b89309ec98a7)
      
      ===
      
      Also squashing in the following fix to pass unit tests for midonet
      interface driver:
      
      Support interface drivers that don't support mtu parameter for plug_new
      
      The method signature before Mitaka did not have the mtu= parameter. We
      should continue supporting the old signature, since it can be used in
      out of tree interface drivers. The class is part of public neutron API,
      so we should make an effort to not break out of tree code.
      
      Local modifications:
      - don't issue a deprecation warning in minor release update.
      
      Change-Id: I8e0c07c76fd0b4c55b66c20ebe29cdb7c07d6f27
      Closes-Bug: #1570392
      (cherry picked from commit 8a86ba1d014a5e758c0569aaf16cfe92492cc7f1)
      
      ===
      
      Change-Id: Ic091fa78dfd133179c71cbc847bf955a06cb248a

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1582705/+subscriptions