]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Round-robin SVI switch selection fails on Cisco Nexus plugin
authorDane LeBlanc <leblancd@cisco.com>
Thu, 14 Nov 2013 04:10:21 +0000 (23:10 -0500)
committerDane LeBlanc <leblancd@cisco.com>
Thu, 14 Nov 2013 17:19:54 +0000 (12:19 -0500)
commit6b21cd29bec6b4c069eeda5bfd4b40e8d1f2a89e
tree44c7b1781b13f26463098abaa3fa42ae10845ff3
parent0e2198ef75c377c538f735b207e38a79d05a2f85
Round-robin SVI switch selection fails on Cisco Nexus plugin

Fixes bug 1250969

This fix addresses improper behavior with the Cisco Nexus plugin's
selection of Nexus switch on which to place a Switch Virtual Interface
(SVI) when round-robin switch placement is enabled.

The expected behavior when round-robin SVI switch selection is configured
via the cisco_plugins.ini file, i.e.:
    [cisco]
    nexus_l3_enable = True
    svi_round_robin = True
is that when a virtual router interface is created, the Nexus plugin should
select the Nexus switch with the least number of SVI interfaces configured
for creating the new SVI. The current selection is based on the first
entry in a dictionary, and is therefore indeterminate.

Similarly, this fix also addresses incorrect behavior when round-robin
selection is disabled. In this case, the desired behavior is that
the plugin should select the first switch which appears in the Nexus
switch configuration. Instead, the current selection is also based on the
first entry in a dictionary, and is likewise indeterminate.

Change-Id: I548c1efaa8b54695246251b6823ab59e077836fe
neutron/plugins/cisco/common/config.py
neutron/plugins/cisco/nexus/cisco_nexus_plugin_v2.py
neutron/tests/unit/cisco/test_network_plugin.py
neutron/tests/unit/cisco/test_nexus_db.py
neutron/tests/unit/cisco/test_nexus_plugin.py