Browse code

Ensure quoting of selected API extensions

Ensure quoting of selected API extensions so the argument makes it to
iniset.

Change-Id: I7c80e3bfe4962c3e8c94736af21ff215cd98f7f3

Ian Wienand authored on 2015/03/03 09:16:05
Showing 1 changed files
... ...
@@ -321,7 +321,7 @@ function configure_tempest {
321 321
     DEFAULT_COMPUTE_EXT+="os-tenant-networks, os-used-limits, os-used-limits-for-admin, os-user-data, os-user-quotas, os-virtual-interfaces, "
322 322
     DEFAULT_COMPUTE_EXT+="os-volume-attachment-update, os-volumes"
323 323
 
324
-    iniset $TEMPEST_CONFIG compute-feature-enabled api_extensions ${COMPUTE_API_EXTENSIONS:-$DEFAULT_COMPUTE_EXT}
324
+    iniset $TEMPEST_CONFIG compute-feature-enabled api_extensions "${COMPUTE_API_EXTENSIONS:-$DEFAULT_COMPUTE_EXT}"
325 325
 
326 326
     # Compute admin
327 327
     iniset $TEMPEST_CONFIG "compute-admin" username $ADMIN_USERNAME
... ...
@@ -399,7 +399,7 @@ function configure_tempest {
399 399
     DEFAULT_VOLUME_EXT+="os-volume-encryption-metadata, os-volume-manage, os-volume-transfer, "
400 400
     DEFAULT_VOLUME_EXT+="os-volume-unmanage, qos-specs"
401 401
 
402
-    iniset $TEMPEST_CONFIG volume-feature-enabled api_extensions ${VOLUME_API_EXTENSIONS:-$DEFAULT_VOLUME_EXT}
402
+    iniset $TEMPEST_CONFIG volume-feature-enabled api_extensions "${VOLUME_API_EXTENSIONS:-$DEFAULT_VOLUME_EXT}"
403 403
 
404 404
     # Dashboard
405 405
     iniset $TEMPEST_CONFIG dashboard dashboard_url "http://$SERVICE_HOST/"
... ...
@@ -413,7 +413,7 @@ function configure_tempest {
413 413
     DEFAULT_NET_EXT+=", lbaas_agent_scheduler, metering, multi-provider, provider, quotas"
414 414
     DEFAULT_NET_EXT+=", router, security-group, service-type, vpnaas"
415 415
     # Networking
416
-    iniset $TEMPEST_CONFIG network-feature-enabled api_extensions ${NETWORK_API_EXTENSIONS:-$DEFAULT_NET_EXT}
416
+    iniset $TEMPEST_CONFIG network-feature-enabled api_extensions "${NETWORK_API_EXTENSIONS:-$DEFAULT_NET_EXT}"
417 417
 
418 418
     # Baremetal
419 419
     if [ "$VIRT_DRIVER" = "ironic" ] ; then
... ...
@@ -423,7 +423,7 @@ function configure_tempest {
423 423
     DEFAULT_SWIFT_EXT="account_quotas, bulk_delete, bulk_upload, container_quotas, container_sync, crossdomain"
424 424
     DEFAULT_SWIFT_EXT+=", formpost, keystoneauth, ratelimit, slo, staticweb, tempauth, tempurl"
425 425
     # Object Storage
426
-    iniset $TEMPEST_CONFIG object-storage-feature-enabled discoverable_apis ${OBJECT_STORAGE_API_EXTENSIONS:-$DEFAULT_SWIFT_EXT}
426
+    iniset $TEMPEST_CONFIG object-storage-feature-enabled discoverable_apis "${OBJECT_STORAGE_API_EXTENSIONS:-$DEFAULT_SWIFT_EXT}"
427 427
     # service_available
428 428
     for service in ${TEMPEST_SERVICES//,/ }; do
429 429
         if is_service_enabled $service ; then