Merge "Get rid of puppet-openstack dependency"

xbezdick · Feb 7, 2015 · 43406a1 · 43406a1
2 parents 0a7304e + 4a4f0c1
commit 43406a1
Show file tree

Hide file tree

Showing 3 changed files with 39 additions and 26 deletions.
diff --git a/packstack/plugins/provision_700.py b/packstack/plugins/provision_700.py
@@ -71,7 +71,39 @@ def process_provision(param, process_args=None):
              "USE_DEFAULT": False,
              "NEED_CONFIRM": False,
              "CONDITION": False},
+        ],
+
+        "PROVISION_DEMO": [
+            {"CMD_OPTION": "provision-demo-floatrange",
+             "USAGE": "The CIDR network address for the floating IP subnet",
+             "PROMPT": "Enter the network address for the floating IP subnet",
+             "OPTION_LIST": False,
+             "VALIDATORS": False,
+             "DEFAULT_VALUE": "172.24.4.224/28",
+             "MASK_INPUT": False,
+             "LOOSE_VALIDATION": True,
+             "CONF_NAME": "CONFIG_PROVISION_DEMO_FLOATRANGE",
+             "USE_DEFAULT": False,
+             "NEED_CONFIRM": False,
+             "CONDITION": False},
+
+            {"CMD_OPTION": "provision-cirros-url",
+             "USAGE": "A URL or local file location for the Cirros demo image "
+                      "used for Glance",
+             "PROMPT": "Enter the URL or local file location for the Cirros "
+                       "image",
+             "OPTION_LIST": False,
+             "VALIDATORS": [validators.validate_not_empty],
+             "DEFAULT_VALUE": DEMO_CIRRUS_URL,
+             "MASK_INPUT": False,
+             "LOOSE_VALIDATION": True,
+             "CONF_NAME": "CONFIG_PROVISION_CIRROS_URL",
+             "USE_DEFAULT": False,
+             "NEED_CONFIRM": False,
+             "CONDITION": False},
+        ],
 
+        "PROVISION_TEMPEST": [
             {"CMD_OPTION": "provision-tempest-user",
              "USAGE": "The name of the Tempest Provisioning user. If you "
                       "don't provide a user name, Tempest will be configured "
@@ -103,39 +135,19 @@ def process_provision(param, process_args=None):
              "NEED_CONFIRM": True,
              "CONDITION": False},
 
-        ],
-
-        "PROVISION_DEMO": [
-            {"CMD_OPTION": "provision-demo-floatrange",
+            {"CMD_OPTION": "provision-tempest-floatrange",
              "USAGE": "The CIDR network address for the floating IP subnet",
              "PROMPT": "Enter the network address for the floating IP subnet",
              "OPTION_LIST": False,
              "VALIDATORS": False,
              "DEFAULT_VALUE": "172.24.4.224/28",
              "MASK_INPUT": False,
              "LOOSE_VALIDATION": True,
-             "CONF_NAME": "CONFIG_PROVISION_DEMO_FLOATRANGE",
-             "USE_DEFAULT": False,
-             "NEED_CONFIRM": False,
-             "CONDITION": False},
-
-            {"CMD_OPTION": "provision-cirros-url",
-             "USAGE": "A URL or local file location for the Cirros demo image "
-                      "used for Glance",
-             "PROMPT": "Enter the URL or local file location for the Cirros "
-                       "image",
-             "OPTION_LIST": False,
-             "VALIDATORS": [validators.validate_not_empty],
-             "DEFAULT_VALUE": DEMO_CIRRUS_URL,
-             "MASK_INPUT": False,
-             "LOOSE_VALIDATION": True,
-             "CONF_NAME": "CONFIG_PROVISION_CIRROS_URL",
+             "CONF_NAME": "CONFIG_PROVISION_TEMPEST_FLOATRANGE",
              "USE_DEFAULT": False,
              "NEED_CONFIRM": False,
              "CONDITION": False},
-        ],
 
-        "TEMPEST_GIT_REFS": [
             {"CMD_OPTION": "provision-tempest-repo-uri",
              "USAGE": "The uri of the tempest git repository to use",
              "PROMPT": "What is the uri of the Tempest git repository?",
@@ -200,12 +212,13 @@ def allow_all_in_one_ovs_bridge(config):
 
         {"GROUP_NAME": "PROVISION_DEMO",
          "DESCRIPTION": "Provisioning demo config",
+         "PRE_CONDITION": check_provisioning_demo,
          "PRE_CONDITION_MATCH": True,
          "POST_CONDITION": False,
          "POST_CONDITION_MATCH": True},
 
-        {"GROUP_NAME": "TEMPEST_GIT_REFS",
-         "DESCRIPTION": "Optional tempest git uri and branch",
+        {"GROUP_NAME": "PROVISION_TEMPEST",
+         "DESCRIPTION": "Provisioning tempest config",
          "PRE_CONDITION": check_provisioning_tempest,
          "PRE_CONDITION_MATCH": True,
          "POST_CONDITION": False,

diff --git a/packstack/puppet/templates/provision_demo.pp b/packstack/puppet/templates/provision_demo.pp
@@ -93,7 +93,7 @@
   firewall { '000 nat':
     chain    => 'POSTROUTING',
     jump     => 'MASQUERADE',
-    source   => $::openstack::provision::floating_range,
+    source   => hiera('CONFIG_PROVISION_DEMO_FLOATRANGE'),
     outiface => $::gateway_device,
     table    => 'nat',
     proto    => 'all',

diff --git a/packstack/puppet/templates/provision_tempest.pp b/packstack/puppet/templates/provision_tempest.pp
@@ -197,7 +197,7 @@
     firewall { '000 nat':
       chain    => 'POSTROUTING',
       jump     => 'MASQUERADE',
-      source   => $::openstack::provision::floating_range,
+      source   => hiera('CONFIG_PROVISION_TEMPEST_FLOATRANGE'),
       outiface => $::gateway_device,
       table    => 'nat',
       proto    => 'all',