Merge lp:~hopem/charms/trusty/neutron-gateway/lp1500386 into lp:~openstack-charmers-archive/charms/trusty/neutron-gateway/next

[Edward Hope-Morley]

[uosci-testing-bot (uosci-testing-bot)]

charm_lint_check #10897 neutron-gateway-next for hopem mp272571
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/10897/

[uosci-testing-bot (uosci-testing-bot)]

charm_unit_test #10114 neutron-gateway-next for hopem mp272571
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/10114/

[uosci-testing-bot (uosci-testing-bot)]

charm_amulet_test #6849 neutron-gateway-next for hopem mp272571
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/6849/

[lp:~hopem/charms/trusty/neutron-gateway/lp1500386]

149. By Edward Hope-Morley on 2015-09-29

sync ch

[uosci-testing-bot (uosci-testing-bot)]

charm_lint_check #11033 neutron-gateway-next for hopem mp272571
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/11033/

[uosci-testing-bot (uosci-testing-bot)]

charm_unit_test #10243 neutron-gateway-next for hopem mp272571
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/10243/

[uosci-testing-bot (uosci-testing-bot)]

charm_amulet_test #6887 neutron-gateway-next for hopem mp272571
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/6887/

[Liam Young (gnuoy)]

Approved

review: Approve

=== modified file 'hooks/charmhelpers/contrib/openstack/context.py'
--- hooks/charmhelpers/contrib/openstack/context.py	2015-09-25 16:13:05 +0000
+++ hooks/charmhelpers/contrib/openstack/context.py	2015-09-29 20:59:59 +0000
@@ -1364,7 +1364,7 @@
             normalized.update({port: port for port in resolved
                                if port in ports})
             if resolved:
-                return {bridge: normalized[port] for port, bridge in
+                return {normalized[port]: bridge for port, bridge in
                         six.iteritems(portmap) if port in normalized.keys()}
 
         return None
@@ -1375,8 +1375,8 @@
     def __call__(self):
         ctxt = {}
         mappings = super(PhyNICMTUContext, self).__call__()
-        if mappings and mappings.values():
-            ports = mappings.values()
+        if mappings and mappings.keys():
+            ports = sorted(mappings.keys())
             napi_settings = NeutronAPIContext()()
             mtu = napi_settings.get('network_device_mtu')
             all_ports = set()
 
=== modified file 'hooks/charmhelpers/contrib/openstack/neutron.py'
--- hooks/charmhelpers/contrib/openstack/neutron.py	2015-09-03 09:43:15 +0000
+++ hooks/charmhelpers/contrib/openstack/neutron.py	2015-09-29 20:59:59 +0000
@@ -310,10 +310,10 @@
 def parse_data_port_mappings(mappings, default_bridge='br-data'):
     """Parse data port mappings.
 
-    Mappings must be a space-delimited list of port:bridge mappings.
+    Mappings must be a space-delimited list of bridge:port.
 
-    Returns dict of the form {port:bridge} where port may be an mac address or
-    interface name.
+    Returns dict of the form {port:bridge} where ports may be mac addresses or
+    interface names.
     """
 
     # NOTE(dosaboy): we use rvalue for key to allow multiple values to be
 
=== modified file 'hooks/neutron_utils.py'
--- hooks/neutron_utils.py	2015-09-28 08:01:40 +0000
+++ hooks/neutron_utils.py	2015-09-29 20:59:59 +0000
@@ -751,11 +751,12 @@
         bridgemaps = parse_bridge_mappings(config('bridge-mappings'))
         for provider, br in bridgemaps.iteritems():
             add_bridge(br)
-
-            if not portmaps or br not in portmaps:
+            if not portmaps:
                 continue
 
-            add_bridge_port(br, portmaps[br], promisc=True)
+            for port, _br in portmaps.iteritems():
+                if _br == br:
+                    add_bridge_port(br, port, promisc=True)
 
         # Ensure this runs so that mtu is applied to data-port interfaces if
         # provided.
 
=== modified file 'tests/charmhelpers/contrib/openstack/amulet/utils.py'
--- tests/charmhelpers/contrib/openstack/amulet/utils.py	2015-09-28 10:06:18 +0000
+++ tests/charmhelpers/contrib/openstack/amulet/utils.py	2015-09-29 20:59:59 +0000
@@ -752,7 +752,7 @@
             self.log.debug('SSL is enabled  @{}:{} '
                            '({})'.format(host, port, unit_name))
             return True
-        elif not port and not conf_ssl:
+        elif not conf_ssl:
             self.log.debug('SSL not enabled @{}:{} '
                            '({})'.format(host, port, unit_name))
             return False
 
=== modified file 'unit_tests/test_neutron_utils.py'
--- unit_tests/test_neutron_utils.py	2015-09-28 08:01:40 +0000
+++ unit_tests/test_neutron_utils.py	2015-09-29 20:59:59 +0000
@@ -221,9 +221,10 @@
             call('br-ex'),
             call('br-data')
         ])
-        self.assertTrue(self.add_bridge_port.called)
+        calls = [call('br-data', 'eth0', promisc=True)]
+        self.add_bridge_port.assert_has_calls(calls)
 
-        # Now test with bridge:port format
+        # Now test with bridge:port format and bogus bridge
         self.test_config.set('data-port', 'br-foo:eth0')
         self.add_bridge.reset_mock()
         self.add_bridge_port.reset_mock()
@@ -236,6 +237,21 @@
         # Not called since we have a bogus bridge in data-ports
         self.assertFalse(self.add_bridge_port.called)
 
+        # Now test with bridge:port format
+        self.test_config.set('bridge-mappings', 'net1:br1')
+        self.test_config.set('data-port', 'br1:eth0.100 br1:eth0.200')
+        self.add_bridge.reset_mock()
+        self.add_bridge_port.reset_mock()
+        neutron_utils.configure_ovs()
+        self.add_bridge.assert_has_calls([
+            call('br-int'),
+            call('br-ex'),
+            call('br1')
+        ])
+        calls = [call('br1', 'eth0.100', promisc=True),
+                 call('br1', 'eth0.200', promisc=True)]
+        self.add_bridge_port.assert_has_calls(calls)
+
     @patch.object(neutron_utils, 'git_install_requested')
     def test_do_openstack_upgrade(self, git_requested):
         git_requested.return_value = False