From a0b536064620e4d18ab00c1154ec3b597ab16a67 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 11:44:14 -0500
Subject: updated nova-manage create network. better help, handling of required
 args, and exceptions. Also updated FLAG flat_network_bridge to default to
 None

---
 bin/nova-manage          | 89 ++++++++++++++++++++++++++----------------------
 nova/exception.py        |  4 +++
 nova/network/manager.py  |  8 ++---
 nova/tests/fake_flags.py |  1 +
 nova/virt/libvirt/vif.py |  2 +-
 5 files changed, 59 insertions(+), 45 deletions(-)

diff --git a/bin/nova-manage b/bin/nova-manage
index b63bd326f..c1a5d8c5f 100755
--- a/bin/nova-manage
+++ b/bin/nova-manage
@@ -631,21 +631,24 @@ class NetworkCommands(object):
     """Class for managing networks."""
 
     @args('--label', dest="label", metavar='<label>',
-            help='Label(ex: public)')
+            help='Label for network (ex: public)')
     @args('--network', dest="fixed_range", metavar='<x.x.x.x/yy>',
-            help='Network')
+            help='IPv4 subnet (ex: 10.0.0.0/8)')
     @args('--num_networks', dest="num_networks", metavar='<number>',
-            help='How many networks create')
+            help='Number of networks to create')
     @args('--network_size', dest="network_size", metavar='<number>',
-            help='How many hosts in network')
+            help='Number of IPs per network')
     @args('--vlan', dest="vlan_start", metavar='<vlan id>', help='vlan id')
     @args('--vpn', dest="vpn_start", help='vpn start')
-    @args('--fixed_range_v6', dest="fixed_range_v6", help='fixed ipv6 range')
+    @args('--fixed_range_v6', dest="fixed_range_v6",
+          help='IPv6 subnet (ex: fe80::/64')
     @args('--gateway_v6', dest="gateway_v6", help='ipv6 gateway')
-    @args('--flat_network_bridge', dest="flat_network_bridge",
-            metavar='<flat network bridge>', help='Flat_network_bridge')
+    @args('--bridge', dest="bridge",
+            metavar='<bridge>',
+            help='VIFs on this network are connected to this bridge')
     @args('--bridge_interface', dest="bridge_interface",
-            metavar='<bridge interface>', help='Bridge_interface')
+            metavar='<bridge interface>',
+            help='the bridge is connected to this interface')
     @args('--multi_host', dest="multi_host", metavar="<'T'|'F'>",
             help='Multi host')
     @args('--dns1', dest="dns1", metavar="<DNS Address>", help='First DNS')
@@ -653,18 +656,31 @@ class NetworkCommands(object):
     def create(self, label=None, fixed_range=None, num_networks=None,
                network_size=None, multi_host=None, vlan_start=None,
                vpn_start=None, fixed_range_v6=None, gateway_v6=None,
-               flat_network_bridge=None, bridge_interface=None,
-               dns1=None, dns2=None):
+               bridge=None, bridge_interface=None, dns1=None, dns2=None):
         """Creates fixed ips for host by range"""
+
+        # check for certain required inputs
         if not label:
-            msg = _('a label (ex: public) is required to create networks.')
-            print msg
-            raise TypeError(msg)
+            raise exception.NetworkNotCreated(req='label')
         if not fixed_range:
-            msg = _('Fixed range in the form of 10.0.0.0/8 is '
-                    'required to create networks.')
-            print msg
-            raise TypeError(msg)
+            raise exception.NetworkNotCreated(req='fixed range')
+
+        bridge = bridge or FLAGS.flat_network_bridge
+        if not bridge:
+            raise exception.NetworkNotCreated(req='bridge')
+            #TODO(tr3buchet) flat_network_bridge FLAG needs to go away
+            #                or be renamed to bridge
+
+        bridge_interface = bridge_interface or FLAGS.flat_interface or \
+                           FLAGS.vlan_interface
+        if not bridge_interface:
+            interface_required = ['nova.network.manager.FlatDHCPManager',
+                                  'nova.network.manager.VlanManager']
+            # ensure that bridge interface is set if certain managers are used
+            if FLAGS.network_manager in interface_required:
+                raise exception.NetworkNotCreated(req='bridge interface')
+
+        # sanitize other input using FLAGS if necessary
         if not num_networks:
             num_networks = FLAGS.num_networks
         if not network_size:
@@ -679,34 +695,27 @@ class NetworkCommands(object):
             vpn_start = FLAGS.vpn_start
         if not fixed_range_v6:
             fixed_range_v6 = FLAGS.fixed_range_v6
-        if not flat_network_bridge:
-            flat_network_bridge = FLAGS.flat_network_bridge
-        if not bridge_interface:
-            bridge_interface = FLAGS.flat_interface or FLAGS.vlan_interface
         if not gateway_v6:
             gateway_v6 = FLAGS.gateway_v6
         if not dns1 and FLAGS.flat_network_dns:
             dns1 = FLAGS.flat_network_dns
-        net_manager = utils.import_object(FLAGS.network_manager)
 
-        try:
-            net_manager.create_networks(context.get_admin_context(),
-                                        label=label,
-                                        cidr=fixed_range,
-                                        multi_host=multi_host,
-                                        num_networks=int(num_networks),
-                                        network_size=int(network_size),
-                                        vlan_start=int(vlan_start),
-                                        vpn_start=int(vpn_start),
-                                        cidr_v6=fixed_range_v6,
-                                        gateway_v6=gateway_v6,
-                                        bridge=flat_network_bridge,
-                                        bridge_interface=bridge_interface,
-                                        dns1=dns1,
-                                        dns2=dns2)
-        except ValueError, e:
-            print e
-            raise e
+        # create the network
+        net_manager = utils.import_object(FLAGS.network_manager)
+        net_manager.create_networks(context.get_admin_context(),
+                                    label=label,
+                                    cidr=fixed_range,
+                                    multi_host=multi_host,
+                                    num_networks=int(num_networks),
+                                    network_size=int(network_size),
+                                    vlan_start=int(vlan_start),
+                                    vpn_start=int(vpn_start),
+                                    cidr_v6=fixed_range_v6,
+                                    gateway_v6=gateway_v6,
+                                    bridge=bridge,
+                                    bridge_interface=bridge_interface,
+                                    dns1=dns1,
+                                    dns2=dns2)
 
     def list(self):
         """List all created networks"""
diff --git a/nova/exception.py b/nova/exception.py
index 38e705417..637cad7ee 100644
--- a/nova/exception.py
+++ b/nova/exception.py
@@ -372,6 +372,10 @@ class StorageRepositoryNotFound(NotFound):
     message = _("Cannot find SR to read/write VDI.")
 
 
+class NetworkNotCreated(NovaException):
+    message = _("A %(req)s is required to create a network.")
+
+
 class NetworkNotFound(NotFound):
     message = _("Network %(network_id)s could not be found.")
 
diff --git a/nova/network/manager.py b/nova/network/manager.py
index 6f7573f66..77572b73d 100644
--- a/nova/network/manager.py
+++ b/nova/network/manager.py
@@ -68,7 +68,7 @@ LOG = logging.getLogger("nova.network.manager")
 
 
 FLAGS = flags.FLAGS
-flags.DEFINE_string('flat_network_bridge', 'br100',
+flags.DEFINE_string('flat_network_bridge', None,
                     'Bridge for simple network instances')
 flags.DEFINE_string('flat_network_dns', '8.8.4.4',
                     'Dns for simple network')
@@ -720,9 +720,9 @@ class FlatManager(NetworkManager):
     """Basic network where no vlans are used.
 
     FlatManager does not do any bridge or vlan creation.  The user is
-    responsible for setting up whatever bridge is specified in
-    flat_network_bridge (br100 by default).  This bridge needs to be created
-    on all compute hosts.
+    responsible for setting up whatever bridges are specified when creating
+    networks through nova-manage. This bridge needs to be created on all
+    compute hosts.
 
     The idea is to create a single network for the host with a command like:
     nova-manage network create 192.168.0.0/24 1 256. Creating multiple
diff --git a/nova/tests/fake_flags.py b/nova/tests/fake_flags.py
index 2297d2f0e..073216495 100644
--- a/nova/tests/fake_flags.py
+++ b/nova/tests/fake_flags.py
@@ -42,3 +42,4 @@ FLAGS['iscsi_num_targets'].SetDefault(8)
 FLAGS['verbose'].SetDefault(True)
 FLAGS['sqlite_db'].SetDefault("tests.sqlite")
 FLAGS['use_ipv6'].SetDefault(True)
+FLAGS['flat_network_bridge'].SetDefault('br100')
diff --git a/nova/virt/libvirt/vif.py b/nova/virt/libvirt/vif.py
index 24d45d1a7..eef582fac 100644
--- a/nova/virt/libvirt/vif.py
+++ b/nova/virt/libvirt/vif.py
@@ -126,7 +126,7 @@ class LibvirtOpenVswitchDriver(VIFDriver):
         dev = "tap-%s" % vif_id
         try:
             utils.execute('sudo', 'ovs-vsctl', 'del-port',
-                          FLAGS.flat_network_bridge, dev)
+                          network['bridge'], dev)
             utils.execute('sudo', 'ip', 'link', 'delete', dev)
         except:
             LOG.warning(_("Failed while unplugging vif of instance '%s'"),
-- 
cgit 


From a300a964cedd44dec09dae3e631d0d11f5782213 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 13:34:45 -0500
Subject: updated the bridge arg requirements based on manager

---
 bin/nova-manage | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/bin/nova-manage b/bin/nova-manage
index c1a5d8c5f..09a65fc02 100755
--- a/bin/nova-manage
+++ b/bin/nova-manage
@@ -667,16 +667,16 @@ class NetworkCommands(object):
 
         bridge = bridge or FLAGS.flat_network_bridge
         if not bridge:
-            raise exception.NetworkNotCreated(req='bridge')
-            #TODO(tr3buchet) flat_network_bridge FLAG needs to go away
-            #                or be renamed to bridge
+            bridge_required = ['nova.network.manager.FlatManager',
+                               'nova.network.manager.FlatDHCPManager']
+            if FLAGS.network_manager in bridge_required:
+                raise exception.NetworkNotCreated(req='bridge')
 
         bridge_interface = bridge_interface or FLAGS.flat_interface or \
                            FLAGS.vlan_interface
         if not bridge_interface:
             interface_required = ['nova.network.manager.FlatDHCPManager',
                                   'nova.network.manager.VlanManager']
-            # ensure that bridge interface is set if certain managers are used
             if FLAGS.network_manager in interface_required:
                 raise exception.NetworkNotCreated(req='bridge interface')
 
-- 
cgit 


From 2b0134bdfa73e7eb616414b3f8cd1b5ed99b8283 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 13:49:36 -0500
Subject: added ipv6 requirements to nova-manage network create. changed
 --network to --fixed_range_v4

---
 bin/nova-manage   | 30 ++++++++++++++++++------------
 nova/exception.py |  2 +-
 2 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/bin/nova-manage b/bin/nova-manage
index 09a65fc02..76a3874cb 100755
--- a/bin/nova-manage
+++ b/bin/nova-manage
@@ -632,7 +632,7 @@ class NetworkCommands(object):
 
     @args('--label', dest="label", metavar='<label>',
             help='Label for network (ex: public)')
-    @args('--network', dest="fixed_range", metavar='<x.x.x.x/yy>',
+    @args('--fixed_range_v4', dest="fixed_range_v4", metavar='<x.x.x.x/yy>',
             help='IPv4 subnet (ex: 10.0.0.0/8)')
     @args('--num_networks', dest="num_networks", metavar='<number>',
             help='Number of networks to create')
@@ -653,7 +653,7 @@ class NetworkCommands(object):
             help='Multi host')
     @args('--dns1', dest="dns1", metavar="<DNS Address>", help='First DNS')
     @args('--dns2', dest="dns2", metavar="<DNS Address>", help='Second DNS')
-    def create(self, label=None, fixed_range=None, num_networks=None,
+    def create(self, label=None, fixed_range_v4=None, num_networks=None,
                network_size=None, multi_host=None, vlan_start=None,
                vpn_start=None, fixed_range_v6=None, gateway_v6=None,
                bridge=None, bridge_interface=None, dns1=None, dns2=None):
@@ -661,16 +661,16 @@ class NetworkCommands(object):
 
         # check for certain required inputs
         if not label:
-            raise exception.NetworkNotCreated(req='label')
-        if not fixed_range:
-            raise exception.NetworkNotCreated(req='fixed range')
+            raise exception.NetworkNotCreated(req='--label')
+        if not fixed_range_v4:
+            raise exception.NetworkNotCreated(req='--fixed_range_v4')
 
         bridge = bridge or FLAGS.flat_network_bridge
         if not bridge:
             bridge_required = ['nova.network.manager.FlatManager',
                                'nova.network.manager.FlatDHCPManager']
             if FLAGS.network_manager in bridge_required:
-                raise exception.NetworkNotCreated(req='bridge')
+                raise exception.NetworkNotCreated(req='--bridge')
 
         bridge_interface = bridge_interface or FLAGS.flat_interface or \
                            FLAGS.vlan_interface
@@ -678,7 +678,17 @@ class NetworkCommands(object):
             interface_required = ['nova.network.manager.FlatDHCPManager',
                                   'nova.network.manager.VlanManager']
             if FLAGS.network_manager in interface_required:
-                raise exception.NetworkNotCreated(req='bridge interface')
+                raise exception.NetworkNotCreated(req='--bridge_interface')
+
+        if FLAGS.use_ipv6:
+            fixed_range_v6 = fixed_range_v6 or FLAGS.fixed_range_v6
+            if not fixed_range_v6:
+                raise exception.NetworkNotCreated(req='with use_ipv6, '
+                                                      '--fixed_range_v6')
+            gateway_v6 = gateway_v6 or FLAGS.gateway_v6
+            if not gateway_v6:
+                raise exception.NetworkNotCreated(req='with use_ipv6, '
+                                                      '--gateway_v6')
 
         # sanitize other input using FLAGS if necessary
         if not num_networks:
@@ -693,10 +703,6 @@ class NetworkCommands(object):
             vlan_start = FLAGS.vlan_start
         if not vpn_start:
             vpn_start = FLAGS.vpn_start
-        if not fixed_range_v6:
-            fixed_range_v6 = FLAGS.fixed_range_v6
-        if not gateway_v6:
-            gateway_v6 = FLAGS.gateway_v6
         if not dns1 and FLAGS.flat_network_dns:
             dns1 = FLAGS.flat_network_dns
 
@@ -704,7 +710,7 @@ class NetworkCommands(object):
         net_manager = utils.import_object(FLAGS.network_manager)
         net_manager.create_networks(context.get_admin_context(),
                                     label=label,
-                                    cidr=fixed_range,
+                                    cidr=fixed_range_v4,
                                     multi_host=multi_host,
                                     num_networks=int(num_networks),
                                     network_size=int(network_size),
diff --git a/nova/exception.py b/nova/exception.py
index 637cad7ee..ca2ba3c96 100644
--- a/nova/exception.py
+++ b/nova/exception.py
@@ -373,7 +373,7 @@ class StorageRepositoryNotFound(NotFound):
 
 
 class NetworkNotCreated(NovaException):
-    message = _("A %(req)s is required to create a network.")
+    message = _("%(req)s is required to create a network.")
 
 
 class NetworkNotFound(NotFound):
-- 
cgit 


From 047c0e993e1dd63a2a6ad75ee40b892c0cd1aa25 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 14:07:31 -0500
Subject: updated handling of v6 in network manager create_networks to it can
 receive None for v6 args

---
 nova/network/manager.py | 39 +++++++++++++++++++--------------------
 1 file changed, 19 insertions(+), 20 deletions(-)

diff --git a/nova/network/manager.py b/nova/network/manager.py
index 77572b73d..dbcc76771 100644
--- a/nova/network/manager.py
+++ b/nova/network/manager.py
@@ -613,17 +613,31 @@ class NetworkManager(manager.SchedulerDependentManager):
                         network_size, cidr_v6, gateway_v6, bridge,
                         bridge_interface, dns1=None, dns2=None, **kwargs):
         """Create networks based on parameters."""
+        net_v6 = {}
+        if FLAGS.use_ipv6:
+            project_net_v6 = netaddr.IPNetwork(cidr_v6)
+            significant_bits_v6 = 64
+            network_size_v6 = 1 << 64
+            start_v6 = index * network_size_v6
+            cidr_v6 = '%s/%s' % (project_net_v6[start_v6],
+                                 significant_bits_v6)
+            net_v6['cidr_v6'] = cidr_v6
+
+            if gateway_v6:
+                # use a pre-defined gateway if one is provided
+                net_v6['gateway_v6'] = str(gateway_v6)
+            else:
+                net_v6['gateway_v6'] = str(project_net_v6[1])
+
+            net_v6['netmask_v6'] = str(project_net_v6._prefixlen)
+
         fixed_net = netaddr.IPNetwork(cidr)
-        fixed_net_v6 = netaddr.IPNetwork(cidr_v6)
-        significant_bits_v6 = 64
-        network_size_v6 = 1 << 64
         for index in range(num_networks):
             start = index * network_size
-            start_v6 = index * network_size_v6
             significant_bits = 32 - int(math.log(network_size, 2))
             cidr = '%s/%s' % (fixed_net[start], significant_bits)
             project_net = netaddr.IPNetwork(cidr)
-            net = {}
+            net = dict(net_v6)
             net['bridge'] = bridge
             net['bridge_interface'] = bridge_interface
             net['dns1'] = dns1
@@ -639,21 +653,6 @@ class NetworkManager(manager.SchedulerDependentManager):
             else:
                 net['label'] = label
 
-            if FLAGS.use_ipv6:
-                cidr_v6 = '%s/%s' % (fixed_net_v6[start_v6],
-                                     significant_bits_v6)
-                net['cidr_v6'] = cidr_v6
-
-                project_net_v6 = netaddr.IPNetwork(cidr_v6)
-
-                if gateway_v6:
-                    # use a pre-defined gateway if one is provided
-                    net['gateway_v6'] = str(gateway_v6)
-                else:
-                    net['gateway_v6'] = str(project_net_v6[1])
-
-                net['netmask_v6'] = str(project_net_v6._prefixlen)
-
             if kwargs.get('vpn', False):
                 # this bit here is for vlan-manager
                 del net['dns1']
-- 
cgit 


From 92f82f620182eab0d5e6dffc8629ec0e25c52303 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 14:35:39 -0500
Subject: oops! moved ipv6 block back into the for loop in network manager
 create_networks

---
 nova/network/manager.py | 37 ++++++++++++++++++-------------------
 1 file changed, 18 insertions(+), 19 deletions(-)

diff --git a/nova/network/manager.py b/nova/network/manager.py
index dbcc76771..34b26d728 100644
--- a/nova/network/manager.py
+++ b/nova/network/manager.py
@@ -613,31 +613,13 @@ class NetworkManager(manager.SchedulerDependentManager):
                         network_size, cidr_v6, gateway_v6, bridge,
                         bridge_interface, dns1=None, dns2=None, **kwargs):
         """Create networks based on parameters."""
-        net_v6 = {}
-        if FLAGS.use_ipv6:
-            project_net_v6 = netaddr.IPNetwork(cidr_v6)
-            significant_bits_v6 = 64
-            network_size_v6 = 1 << 64
-            start_v6 = index * network_size_v6
-            cidr_v6 = '%s/%s' % (project_net_v6[start_v6],
-                                 significant_bits_v6)
-            net_v6['cidr_v6'] = cidr_v6
-
-            if gateway_v6:
-                # use a pre-defined gateway if one is provided
-                net_v6['gateway_v6'] = str(gateway_v6)
-            else:
-                net_v6['gateway_v6'] = str(project_net_v6[1])
-
-            net_v6['netmask_v6'] = str(project_net_v6._prefixlen)
-
         fixed_net = netaddr.IPNetwork(cidr)
         for index in range(num_networks):
             start = index * network_size
             significant_bits = 32 - int(math.log(network_size, 2))
             cidr = '%s/%s' % (fixed_net[start], significant_bits)
             project_net = netaddr.IPNetwork(cidr)
-            net = dict(net_v6)
+            net = {}
             net['bridge'] = bridge
             net['bridge_interface'] = bridge_interface
             net['dns1'] = dns1
@@ -653,6 +635,23 @@ class NetworkManager(manager.SchedulerDependentManager):
             else:
                 net['label'] = label
 
+            if FLAGS.use_ipv6:
+                project_net_v6 = netaddr.IPNetwork(cidr_v6)
+                significant_bits_v6 = 64
+                network_size_v6 = 1 << 64
+                start_v6 = index * network_size_v6
+                cidr_v6 = '%s/%s' % (project_net_v6[start_v6],
+                                     significant_bits_v6)
+                net['cidr_v6'] = cidr_v6
+
+                if gateway_v6:
+                    # use a pre-defined gateway if one is provided
+                    net['gateway_v6'] = str(gateway_v6)
+                else:
+                    net['gateway_v6'] = str(project_net_v6[1])
+
+                net['netmask_v6'] = str(project_net_v6._prefixlen)
+
             if kwargs.get('vpn', False):
                 # this bit here is for vlan-manager
                 del net['dns1']
-- 
cgit 


From 7c9dc248af2e19abd04b342f53b1f93fe1547bba Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 17:19:19 -0500
Subject: --bridge defaults to br100 but with a deprecation warning and to be
 removed in d4

---
 bin/nova-manage | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/bin/nova-manage b/bin/nova-manage
index 76a3874cb..ef0ae0fbb 100755
--- a/bin/nova-manage
+++ b/bin/nova-manage
@@ -670,7 +670,16 @@ class NetworkCommands(object):
             bridge_required = ['nova.network.manager.FlatManager',
                                'nova.network.manager.FlatDHCPManager']
             if FLAGS.network_manager in bridge_required:
-                raise exception.NetworkNotCreated(req='--bridge')
+                # TODO(tr3buchet) - swap print statement and following line for
+                #                   raise statement in diablo 4
+                print _('--bridge parameter required or FLAG '
+                        'flat_network_bridge must be set to create networks'
+                        'WARNING! ACHTUNG! Setting the bridge to br100 '
+                        'automatically is deprecated and will be removed in '
+                        'Diablo milestone 4')
+                time.sleep(10)
+                bridge = 'br100'
+                #raise exception.NetworkNotCreated(req='--bridge')
 
         bridge_interface = bridge_interface or FLAGS.flat_interface or \
                            FLAGS.vlan_interface
-- 
cgit 


From f4c8c97cdd1ec9a9b6c7f271dcc92b1e716c3285 Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 17:24:39 -0500
Subject: typo

---
 bin/nova-manage | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bin/nova-manage b/bin/nova-manage
index ef0ae0fbb..75d74903c 100755
--- a/bin/nova-manage
+++ b/bin/nova-manage
@@ -673,10 +673,10 @@ class NetworkCommands(object):
                 # TODO(tr3buchet) - swap print statement and following line for
                 #                   raise statement in diablo 4
                 print _('--bridge parameter required or FLAG '
-                        'flat_network_bridge must be set to create networks'
+                        'flat_network_bridge must be set to create networks\n'
                         'WARNING! ACHTUNG! Setting the bridge to br100 '
                         'automatically is deprecated and will be removed in '
-                        'Diablo milestone 4')
+                        'Diablo milestone 4. Prepare yourself accordingly.')
                 time.sleep(10)
                 bridge = 'br100'
                 #raise exception.NetworkNotCreated(req='--bridge')
-- 
cgit 


From 45ff96e29669c5650873b0a7ddcf52160eddd27c Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 17:58:13 -0500
Subject: fixed create_networks ipv6 management

---
 nova/network/manager.py | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/nova/network/manager.py b/nova/network/manager.py
index 34b26d728..37f8ea25d 100644
--- a/nova/network/manager.py
+++ b/nova/network/manager.py
@@ -614,6 +614,11 @@ class NetworkManager(manager.SchedulerDependentManager):
                         bridge_interface, dns1=None, dns2=None, **kwargs):
         """Create networks based on parameters."""
         fixed_net = netaddr.IPNetwork(cidr)
+        if FLAGS.use_ipv6:
+            fixed_net_v6 = netaddr.IPNetwork(cidr_v6)
+            significant_bits_v6 = 64
+            network_size_v6 = 1 << 64
+
         for index in range(num_networks):
             start = index * network_size
             significant_bits = 32 - int(math.log(network_size, 2))
@@ -636,14 +641,12 @@ class NetworkManager(manager.SchedulerDependentManager):
                 net['label'] = label
 
             if FLAGS.use_ipv6:
-                project_net_v6 = netaddr.IPNetwork(cidr_v6)
-                significant_bits_v6 = 64
-                network_size_v6 = 1 << 64
-                start_v6 = index * network_size_v6
-                cidr_v6 = '%s/%s' % (project_net_v6[start_v6],
+                cidr_v6 = '%s/%s' % (fixed_net_v6[start_v6],
                                      significant_bits_v6)
                 net['cidr_v6'] = cidr_v6
 
+                project_net_v6 = netaddr.IPNetwork(cidr_v6)
+
                 if gateway_v6:
                     # use a pre-defined gateway if one is provided
                     net['gateway_v6'] = str(gateway_v6)
-- 
cgit 


From 7f5437a435cb7d98c65749488b9a45bfb8bd6fec Mon Sep 17 00:00:00 2001
From: Trey Morris <trey.morris@rackspace.com>
Date: Wed, 27 Jul 2011 18:00:45 -0500
Subject: forgot a line

---
 nova/network/manager.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/nova/network/manager.py b/nova/network/manager.py
index 37f8ea25d..4a3791d8a 100644
--- a/nova/network/manager.py
+++ b/nova/network/manager.py
@@ -641,6 +641,7 @@ class NetworkManager(manager.SchedulerDependentManager):
                 net['label'] = label
 
             if FLAGS.use_ipv6:
+                start_v6 = index * network_size_v6
                 cidr_v6 = '%s/%s' % (fixed_net_v6[start_v6],
                                      significant_bits_v6)
                 net['cidr_v6'] = cidr_v6
-- 
cgit