← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1540779] Re: DVR router should not allow manually removed from an agent in 'dvr' mode

 

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

commit af62088fb54c675917c07c2f94973075f24c440a
Author: lzklibj <lzklibj@xxxxxxxxxx>
Date:   Sat Jan 2 12:44:30 2016 +0800

    Fix remove_router_from_l3_agent for 'dvr' mode agent
    
    It's possible to run command remove_router_from_l3_agent to remove
    a DVR router from an agent in 'dvr' mode. But the implicit *binding*
    between DVR router and agent in 'dvr' mode should come and go as
    dvr serviced port on host come and go, not manually managed.
    
    Closes-Bug: #1540779
    Change-Id: Ied6c88c85ced7b956fad3473ede4688020a357a4


** Changed in: neutron
       Status: In Progress => Fix Released

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

Title:
  DVR router should not allow manually removed from an agent in 'dvr'
  mode

Status in neutron:
  Fix Released

Bug description:
  Per bp/improve-dvr-l3-agent-binding, command "neutron l3-agent-list-hosting-router ROUTER" couldn't show bindings for DVR routers on agents in 'dvr' mode now.
  It's good to hide the implicit *binding* between DVR router and agent in 'dvr' mode, for DVR routers should come and go as dvr serviced port on host come and go, not for manually managed.
  But it still be possible to run "neutron l3-agent-router-remove AGENT ROUTER" to remove a DVR router from an agent in 'dvr' mode. This will make DVR router namespace deleted, and l3 networking on that node crashed.
  We should add a checking for removing router from agent in 'dvr' mode, and forbidden processing going on.

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


References