Mirantis OpenStack

OVS agent fails to setup distributed router port

Bug #1555359 reported by Eugene Nikanorov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Critical
Eugene Nikanorov
Mirantis OpenStack 7.0-mu-3

Bug Description

Corresponding upstream bug: https://bugs.launchpad.net/neutron/+bug/1404823

The following bug also needs to be taken into account: https://bugs.launchpad.net/neutron/+bug/1530179

See original description
Eugene Nikanorov (enikanorov)
tags: added: customer-found
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/neutron (openstack-ci/fuel-7.0/2015.1.0)

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Swaminathan Vasudevan <email address hidden>
Review: https://review.fuel-infra.org/17882

Eugene Nikanorov (enikanorov)
no longer affects: mos/8.0.x
Changed in mos:
status: Invalid → In Progress
importance: Undecided → Critical
milestone: none → 7.0-updates
assignee: nobody → Eugene Nikanorov (enikanorov)
Eugene Nikanorov (enikanorov)
description: updated
Vitaly Sedelnik (vsedelnik)
Changed in mos:
milestone: 7.0-updates → 7.0-mu-3
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote :

Fix proposed to branch: openstack-ci/fuel-7.0/2015.1.0
Change author: Swaminathan Vasudevan <email address hidden>
Review: https://review.fuel-infra.org/18007

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/neutron (openstack-ci/fuel-7.0/2015.1.0)

Change abandoned by Vitaly Sedelnik <email address hidden> on branch: openstack-ci/fuel-7.0/2015.1.0
Review: https://review.fuel-infra.org/18007
Reason: Abandoned per Eugene N comment

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/neutron (openstack-ci/fuel-7.0/2015.1.0)

Reviewed: https://review.fuel-infra.org/17882
Submitter: Vitaly Sedelnik <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: fbd93076d531982991a726b454ff87b3af1ef09f
Author: Swaminathan Vasudevan <email address hidden>
Date: Fri Mar 11 21:44:11 2016

1. Fix gateway port could not retrieve for subnet

In DVR routers when a port is added to a router, then
the command succeeds but the l2 ovs agent raises an
error stating that it could not retrieve the gateway
port for the subnet.

The reason for this is there is mismatch in the
ip_address returned from the subnet for the gateway
port and the actual ip_address of the port that we
added to the router.

Since the subnet info was passed to "get_subnet_for_dvr"
this mismatch was seen.

Instead of passing the subnet we will be passing the
actual fixed_ips with all the details and the subnet
gateway port will be filtered based on the ip_address
in the fixed_ips passed.

2. Fix get_subnet_for_dvr() to return correct gateway mac

Fix filters to get the right gateway port for a subnet.
Wrong filters led to random port's mac being returned which may cause
side effects. See bug for details.

Closes-Bug: #1530179
Change-Id: I8368255f00ab3e9586c8ff28dfe6739541000810
(cherry picked from commit 0a00205c2a209f20a9d900ddb30f25628a32ebd9)

Conflicts:
        neutron/api/rpc/handlers/dvr_rpc.py
        neutron/db/dvr_mac_db.py

Closes-Bug: #1404823
Closes-Bug: #1555359
Closes-Bug: #1530179
Change-Id: I87a3983951f814350e79f5e2274f4639bb6bc0f5

Vitaly Sedelnik (vsedelnik)
Changed in mos:
status: In Progress → Fix Committed
Ekaterina Shutova (eshutova)
tags: added: on-verification
Revision history for this message
Ekaterina Shutova (eshutova) wrote :

Verified on MOS 7.0 + mu3 updates.

Scenario:
1) Create network, subnet
2) Create router
3) Create port
4) Add an internal network interface to a router(port)
5) Check that no error appeared in the log

tags: removed: on-verification
Vadim Rovachev (vrovachev)
Changed in mos:
status: Fix Committed → Fix Released
no longer affects: mos/7.0.x
Roman Rufanov (rrufanov)
tags: added: support
Alexander Gromov (agromov)
tags: added: on-automated
Revision history for this message
Alexander Gromov (agromov) wrote :

Test case is automated: https://mirantis.testrail.com/index.php?/cases/view/843829

tags: removed: on-automated
Alexander Gromov (agromov)
tags: added: covered-automated-test
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.