summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTim Rozet <trozet@redhat.com>2015-04-27 14:15:42 -0400
committerTim Rozet <trozet@redhat.com>2015-04-27 14:15:42 -0400
commita0aee1e0d6f256cafa745ff0fb1e073a55d12884 (patch)
treeb000b4d679bf9e2cb3895103be13b7875cee1cb8
parente2b2d46756213fde3bca42a49b04e6a1e8792498 (diff)
Removes interface names as parameters to the puppet modules
Patch changes parameters from being interface names of linux interfaces to being network subnets. This removes the need to actually specify the network interface to puppet module and is found out dynamically at puppet runtime. JIRA: BGS-42 Change-Id: Ibab114c46dd2ec9fde244b6687bf272849b15d6b Signed-off-by: Tim Rozet <trozet@redhat.com>
-rw-r--r--common/puppet-opnfv/manifests/compute.pp8
-rw-r--r--common/puppet-opnfv/manifests/controller_networker.pp12
2 files changed, 15 insertions, 5 deletions
diff --git a/common/puppet-opnfv/manifests/compute.pp b/common/puppet-opnfv/manifests/compute.pp
index 7bba609..0b81757 100644
--- a/common/puppet-opnfv/manifests/compute.pp
+++ b/common/puppet-opnfv/manifests/compute.pp
@@ -28,8 +28,6 @@ class opnfv::compute {
}
##Common Parameters
- if $ovs_tunnel_if == '' { fail('ovs_tunnel_if is empty') }
-
if !$rbd_secret_uuid { $rbd_secret_uuid = '3b519746-4021-4f72-957e-5b9d991723be' }
if !$private_subnet { fail('private_subnet is empty')}
if !$ceph_public_network { $ceph_public_network = $private_subnet }
@@ -54,6 +52,7 @@ class opnfv::compute {
##HA Global params
if $ha_flag {
+ if $private_network == '' { fail('private_network is empty') }
if !$keystone_private_vip { fail('keystone_private_vip is empty') }
if !$glance_private_vip { fail('glance_private_vip is empty') }
if !$nova_private_vip { fail('nova_private_vip is empty') }
@@ -73,8 +72,13 @@ class opnfv::compute {
if !$ceph_mon_initial_members { $ceph_mon_initial_members = $controllers_hostnames_array }
if !$ceph_mon_host { $ceph_mon_host = $controllers_ip_array }
if !$neutron_private_vip { fail('neutron_private_vip is empty') }
+
+ ##Find private interface
+ $ovs_tunnel_if = get_nic_from_network("$private_network")
+
} else {
##non HA params
+ if $ovs_tunnel_if == '' { fail('ovs_tunnel_if is empty') }
if !$private_ip { fail('private_ip is empty') }
$keystone_private_vip = $private_ip
$glance_private_vip = $private_ip
diff --git a/common/puppet-opnfv/manifests/controller_networker.pp b/common/puppet-opnfv/manifests/controller_networker.pp
index 6888850..cff258d 100644
--- a/common/puppet-opnfv/manifests/controller_networker.pp
+++ b/common/puppet-opnfv/manifests/controller_networker.pp
@@ -30,7 +30,6 @@ class opnfv::controller_networker {
##Mandatory Common variables
if $admin_email == '' { fail('admin_email is empty') }
- if $ovs_tunnel_if == '' { fail('ovs_tunnel_if is empty') }
##Most users will only care about a single user/password for all services
##so lets create one variable that can be used instead of separate usernames/passwords
@@ -73,7 +72,10 @@ class opnfv::controller_networker {
if !$nova_admin_vip { fail('nova_admin_vip is empty') }
if !$nova_private_vip { fail('nova_private_vip is empty') }
if !$nova_public_vip { fail('nova_public_vip is empty') }
+ if $private_network == '' { fail('private_network is empty') }
+ ##Find private interface
+ $ovs_tunnel_if = get_nic_from_network("$private_network")
##Optional HA variables
if !$amqp_username { $amqp_username = $single_username }
@@ -102,7 +104,11 @@ class opnfv::controller_networker {
if !$pcmk_server_addrs {$pcmk_server_addrs = $controllers_ip_array}
if !$pcmk_server_names {$pcmk_server_names = ["pcmk-${controllers_hostnames_array[0]}", "pcmk-${controllers_hostnames_array[1]}", "pcmk-${controllers_hostnames_array[2]}"] }
if !$rbd_secret_uuid { $rbd_secret_uuid = '3b519746-4021-4f72-957e-5b9d991723be' }
- if !$storage_iface { $storage_iface = $ovs_tunnel_if }
+ if !$storage_network {
+ $storage_iface = $ovs_tunnel_if
+ } else {
+ $storage_iface = get_nic_from_network("$storage_network")
+ }
##we assume here that if not provided, the first controller is where ODL will reside
##this is fine for now as we will replace ODL with ODL HA when it is ready
@@ -273,7 +279,7 @@ class opnfv::controller_networker {
}
} else {
-
+ if $ovs_tunnel_if == '' { fail('ovs_tunnel_if is empty') }
if $public_ip == '' { fail('public_ip is empty') }
if $private_ip == '' { fail('private_ip is empty') }