diff --git a/packstack/plugins/provision_700.py b/packstack/plugins/provision_700.py index 0be3f0c4e..5e201cd38 100644 --- a/packstack/plugins/provision_700.py +++ b/packstack/plugins/provision_700.py @@ -60,38 +60,6 @@ def initConfig(controller): "USE_DEFAULT": False, "NEED_CONFIRM": False, "CONDITION": False}, - - {"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 " - "in a standalone mode", - "PROMPT": ("Enter the name of the Tempest Provisioning user " - "(if blank, Tempest will be configured in a " - "standalone mode) "), - "OPTION_LIST": False, - "VALIDATORS": False, - "DEFAULT_VALUE": "", - "MASK_INPUT": False, - "LOOSE_VALIDATION": True, - "CONF_NAME": "CONFIG_PROVISION_TEMPEST_USER", - "USE_DEFAULT": False, - "NEED_CONFIRM": False, - "CONDITION": False}, - - {"CMD_OPTION": "provision-tempest-user-passwd", - "USAGE": "The password to use for the Tempest Provisioning user", - "PROMPT": "Enter the password for the Tempest Provisioning user", - "OPTION_LIST": [], - "VALIDATORS": [validators.validate_not_empty], - "DEFAULT_VALUE": "PW_PLACEHOLDER", - "PROCESSORS": [processors.process_password], - "MASK_INPUT": True, - "LOOSE_VALIDATION": False, - "CONF_NAME": "CONFIG_PROVISION_TEMPEST_USER_PW", - "USE_DEFAULT": False, - "NEED_CONFIRM": True, - "CONDITION": False}, - ], "PROVISION_DEMO": [ @@ -124,7 +92,51 @@ def initConfig(controller): "CONDITION": False}, ], - "TEMPEST_GIT_REFS": [ + "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 " + "in a standalone mode", + "PROMPT": ("Enter the name of the Tempest Provisioning user " + "(if blank, Tempest will be configured in a " + "standalone mode) "), + "OPTION_LIST": False, + "VALIDATORS": False, + "DEFAULT_VALUE": "", + "MASK_INPUT": False, + "LOOSE_VALIDATION": True, + "CONF_NAME": "CONFIG_PROVISION_TEMPEST_USER", + "USE_DEFAULT": False, + "NEED_CONFIRM": False, + "CONDITION": False}, + + {"CMD_OPTION": "provision-tempest-user-passwd", + "USAGE": "The password to use for the Tempest Provisioning user", + "PROMPT": "Enter the password for the Tempest Provisioning user", + "OPTION_LIST": [], + "VALIDATORS": [validators.validate_not_empty], + "DEFAULT_VALUE": "PW_PLACEHOLDER", + "PROCESSORS": [processors.process_password], + "MASK_INPUT": True, + "LOOSE_VALIDATION": False, + "CONF_NAME": "CONFIG_PROVISION_TEMPEST_USER_PW", + "USE_DEFAULT": False, + "NEED_CONFIRM": True, + "CONDITION": False}, + + {"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_TEMPEST_FLOATRANGE", + "USE_DEFAULT": False, + "NEED_CONFIRM": False, + "CONDITION": False}, + {"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?", @@ -189,12 +201,13 @@ def initConfig(controller): {"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 index 0d4c92186..aa87b2826 100644 --- a/packstack/puppet/templates/provision_demo.pp +++ b/packstack/puppet/templates/provision_demo.pp @@ -93,7 +93,7 @@ if $setup_ovs_bridge { 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 index 8920484d7..8bcb3dafb 100644 --- a/packstack/puppet/templates/provision_tempest.pp +++ b/packstack/puppet/templates/provision_tempest.pp @@ -197,7 +197,7 @@ if $provision_tempest_user != '' { 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',