Browse code

check all possible services when configuring tempest

The previous approach assumed that devstack in tree service support
would always be a super set of tempest. That's not necessarily
true. Instead when configuring tempest we should look at all the
possible services that tempest could know about, which will let us
disable services we don't have support for.

Change-Id: I9c24705e494689f09a885eb0a640efd50db33fcf

Sean Dague authored on 2015/08/26 22:38:37
Showing 12 changed files
... ...
@@ -102,9 +102,6 @@ CEILOMETER_EVENTS=${CEILOMETER_EVENTS:-True}
102 102
 CEILOMETER_COORDINATION_URL=${CEILOMETER_COORDINATION_URL:-}
103 103
 CEILOMETER_PIPELINE_INTERVAL=${CEILOMETER_PIPELINE_INTERVAL:-}
104 104
 
105
-# Tell Tempest this project is present
106
-TEMPEST_SERVICES+=,ceilometer
107
-
108 105
 
109 106
 # Functions
110 107
 # ---------
... ...
@@ -108,9 +108,6 @@ CINDER_PERIODIC_INTERVAL=${CINDER_PERIODIC_INTERVAL:-60}
108 108
 
109 109
 CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-tgtadm}
110 110
 
111
-# Tell Tempest this project is present
112
-TEMPEST_SERVICES+=,cinder
113
-
114 111
 
115 112
 # Source the enabled backends
116 113
 if is_service_enabled c-vol && [[ -n "$CINDER_ENABLED_BACKENDS" ]]; then
... ...
@@ -75,9 +75,6 @@ GLANCE_SEARCH_PORT=${GLANCE_SEARCH_PORT:-9393}
75 75
 GLANCE_SEARCH_PORT_INT=${GLANCE_SEARCH_PORT_INT:-19393}
76 76
 GLANCE_SEARCH_HOSTPORT=${GLANCE_SEARCH_HOSTPORT:-$GLANCE_SERVICE_HOST:$GLANCE_SEARCH_PORT}
77 77
 
78
-# Tell Tempest this project is present
79
-TEMPEST_SERVICES+=,glance
80
-
81 78
 # Functions
82 79
 # ---------
83 80
 
... ...
@@ -64,10 +64,6 @@ else
64 64
     HEAT_DEFERRED_AUTH=${HEAT_DEFERRED_AUTH:-trusts}
65 65
 fi
66 66
 
67
-# Tell Tempest this project is present
68
-TEMPEST_SERVICES+=,heat
69
-
70
-
71 67
 # Functions
72 68
 # ---------
73 69
 
... ...
@@ -35,10 +35,6 @@ HORIZON_DIR=$DEST/horizon
35 35
 # The example file in Horizon repo is used by default.
36 36
 HORIZON_SETTINGS=${HORIZON_SETTINGS:-$HORIZON_DIR/openstack_dashboard/local/local_settings.py.example}
37 37
 
38
-# Tell Tempest this project is present
39
-TEMPEST_SERVICES+=,horizon
40
-
41
-
42 38
 # Functions
43 39
 # ---------
44 40
 
... ...
@@ -114,9 +114,6 @@ IRONIC_SERVICE_PROTOCOL=http
114 114
 IRONIC_SERVICE_PORT=${IRONIC_SERVICE_PORT:-6385}
115 115
 IRONIC_HOSTPORT=${IRONIC_HOSTPORT:-$SERVICE_HOST:$IRONIC_SERVICE_PORT}
116 116
 
117
-# Tell Tempest this project is present
118
-TEMPEST_SERVICES+=,ironic
119
-
120 117
 # Enable iPXE
121 118
 IRONIC_IPXE_ENABLED=$(trueorfalse False IRONIC_IPXE_ENABLED)
122 119
 IRONIC_HTTP_DIR=${IRONIC_HTTP_DIR:-$IRONIC_DATA_DIR/httpboot}
... ...
@@ -358,10 +358,6 @@ else
358 358
     Q_USE_SECGROUP=False
359 359
 fi
360 360
 
361
-# Tell Tempest this project is present
362
-TEMPEST_SERVICES+=,neutron
363
-
364
-
365 361
 # Save trace setting
366 362
 XTRACE=$(set +o | grep xtrace)
367 363
 set +o xtrace
... ...
@@ -167,10 +167,6 @@ NOVA_ALLOW_MOVE_TO_SAME_HOST=$(trueorfalse True NOVA_ALLOW_MOVE_TO_SAME_HOST)
167 167
 TEST_FLOATING_POOL=${TEST_FLOATING_POOL:-test}
168 168
 TEST_FLOATING_RANGE=${TEST_FLOATING_RANGE:-192.168.253.0/29}
169 169
 
170
-# Tell Tempest this project is present
171
-TEMPEST_SERVICES+=,nova
172
-
173
-
174 170
 # Functions
175 171
 # ---------
176 172
 
... ...
@@ -141,10 +141,6 @@ SWIFT_TEMPURL_KEY=${SWIFT_TEMPURL_KEY:-}
141 141
 # Toggle for deploying Swift under HTTPD + mod_wsgi
142 142
 SWIFT_USE_MOD_WSGI=${SWIFT_USE_MOD_WSGI:-False}
143 143
 
144
-# Tell Tempest this project is present
145
-TEMPEST_SERVICES+=,swift
146
-
147
-
148 144
 # Functions
149 145
 # ---------
150 146
 
... ...
@@ -536,6 +536,11 @@ function configure_tempest {
536 536
     fi
537 537
 
538 538
     # ``service_available``
539
+    #
540
+    # this tempest service list needs to be all the services that
541
+    # tempest supports, otherwise we can have an erroneous set of
542
+    # defaults (something defaulting true in Tempest, but not listed here).
543
+    TEMPEST_SERVICES="key,glance,nova,neutron,cinder,swift,heat,ceilometer,horizon,sahara,ironic,trove,zaqar"
539 544
     for service in ${TEMPEST_SERVICES//,/ }; do
540 545
         if is_service_enabled $service ; then
541 546
             iniset $TEMPEST_CONFIG service_available $service "True"
... ...
@@ -59,10 +59,6 @@ ZAQAR_SERVICE_HOST=${ZAQAR_SERVICE_HOST:-$SERVICE_HOST}
59 59
 ZAQAR_SERVICE_PORT=${ZAQAR_SERVICE_PORT:-8888}
60 60
 ZAQAR_SERVICE_PROTOCOL=${ZAQAR_SERVICE_PROTOCOL:-$SERVICE_PROTOCOL}
61 61
 
62
-# Tell Tempest this project is present
63
-TEMPEST_SERVICES+=,zaqar
64
-
65
-
66 62
 # Functions
67 63
 # ---------
68 64
 
... ...
@@ -78,12 +78,6 @@ fi
78 78
 # services will rely on the local toggle variable (e.g. ``KEYSTONE_USE_MOD_WSGI``)
79 79
 ENABLE_HTTPD_MOD_WSGI_SERVICES=True
80 80
 
81
-# Tell Tempest which services are available.  The default is set here as
82
-# Tempest falls late in the configuration sequence.  This differs from
83
-# ``ENABLED_SERVICES`` in that the project names are used here rather than
84
-# the service names, i.e.: ``TEMPEST_SERVICES="key,glance,nova"``
85
-TEMPEST_SERVICES=""
86
-
87 81
 # Set the default Nova APIs to enable
88 82
 NOVA_ENABLED_APIS=ec2,osapi_compute,metadata
89 83