get_subnet_for_dvr() returns wrong gateway mac
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| neutron |
Medium
|
Oleg Bondarev |
Bug Description
get_subnet_for_dvr should return proper gateway mac address in order for ovs agent to add proper flows for dvr interface on br-int.
commit e82b0e108332964
if fixed_ips:
else:
get_ports() here actually returns _all_ ports so mac address of a random port is returned as 'gateway_mac'. In most cases it doesn't lead to any noticeable side effects but in some cases it may cause very weird behavior.
The case that we faced was:
root@node-9:~# ovs-ofctl dump-flows br-int
...
cookie=
cookie=
...
fa:16:3e:2c:24:86 is mac address of a vm port and it was returned as gateway mac due to the bug. This vm was unreachable from other subnets connected to the same dvr router. However another vm on the same host and the same subnet was ok. It took a while to find out what was wrong :)
Changed in neutron: | |
status: | New → In Progress |
tags: | added: liberty-backport-potential |
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: master
commit 0a00205c2a209f2
Author: Oleg Bondarev <email address hidden>
Date: Wed Dec 30 20:24:55 2015 +0300
Fix get_subnet_
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: I8368255f00ab3e
Changed in neutron: | |
status: | In Progress → Fix Released |
This issue was fixed in the openstack/neutron 8.0.0.0b2 development milestone.
Fix proposed to branch: stable/liberty
Review: https:/
Reviewed: https:/
Committed: https:/
Submitter: Jenkins
Branch: stable/liberty
commit 1ab1e5873c4254d
Author: Oleg Bondarev <email address hidden>
Date: Wed Dec 30 20:24:55 2015 +0300
Fix get_subnet_
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: I8368255f00ab3e
(cherry picked from commit 0a00205c2a209f2
tags: | added: in-stable-liberty |
This issue was fixed in the openstack/neutron 7.0.3 release.
tags: | removed: liberty-backport-potential |
Fix proposed to branch: master /review. openstack. org/262574
Review: https:/