Browse code

Merge "Single call for 'nova flavor-list'"

Jenkins authored on 2015/12/01 03:08:26
Showing 1 changed files
... ...
@@ -196,8 +196,8 @@ function configure_tempest {
196 196
     if is_service_enabled nova; then
197 197
         # If ``DEFAULT_INSTANCE_TYPE`` is not declared, use the new behavior
198 198
         # Tempest creates its own instance types
199
+        available_flavors=$(nova flavor-list)
199 200
         if  [[ -z "$DEFAULT_INSTANCE_TYPE" ]]; then
200
-            available_flavors=$(nova flavor-list)
201 201
             if [[ ! ( $available_flavors =~ 'm1.nano' ) ]]; then
202 202
                 nova flavor-create m1.nano 42 64 0 1
203 203
             fi
... ...
@@ -210,15 +210,14 @@ function configure_tempest {
210 210
         else
211 211
             # Check Nova for existing flavors, if ``DEFAULT_INSTANCE_TYPE`` is set use it.
212 212
             boto_instance_type=$DEFAULT_INSTANCE_TYPE
213
-            flavor_lines=`nova flavor-list`
214 213
             IFS=$'\r\n'
215 214
             flavors=""
216
-            for line in $flavor_lines; do
215
+            for line in $available_flavors; do
217 216
                 f=$(echo $line | awk "/ $DEFAULT_INSTANCE_TYPE / { print \$2 }")
218 217
                 flavors="$flavors $f"
219 218
             done
220 219
 
221
-            for line in $flavor_lines; do
220
+            for line in $available_flavors; do
222 221
                 flavors="$flavors `echo $line | grep -v "^\(|\s*ID\|+--\)" | cut -d' ' -f2`"
223 222
             done
224 223