Comment 13 for bug 1512238

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/36212
Submitter: Denis V. Meltsaykin <email address hidden>
Branch: openstack-ci/fuel-7.0/2015.1.0

Commit: de3f5bb993549511b1397baddaba2b80de26b57a
Author: Max Yatsenko <email address hidden>
Date: Mon Aug 21 08:36:41 2017

Fix OVS flows creation for new networks

To solve this bug two patches were used:

* Fix _restore_local_vlan_map race [1]
  Local vlan mappings provisioned by _restore_local_vlan_map
  might not be cleaned up properly if the corresponding ports are
  removed before we enter the main loop.

  This commit fixes the problem by making _restore_local_vlan_map
  just record local vlan IDs and leave the rest of provisioning logic
  to the main loop.

* Fix usage of netaddr '.broadcast' [2]
  netaddr 0.7.16 changed the behavior of ipnetworks with /31 and /32
  prefixes to make their 'broadcast' attribute return none. this patch
  replaces the use of the attribute with a -1 index lookup to get the
  last address instead.

  For stable kilo we also fix neutron/db/db_base_plugin_v2.py which had
  the checks in neutron/ipam/utils.py for Liberty.

[1] https://review.openstack.org/#/c/238986/
[2] https://review.openstack.org/#/c/218852/

Closes-Bug: #1512238
Closes-Bug: #1507776

Change-Id: I5ec8fe74307a1dcb1ff7e9cc8c2a8a9acdbd1841