diff options
| author |   Carlos Goncalves <carlos.goncalves@neclab.eu> | 2016-09-06 15:40:37 +0200 |
|---|---|---|
| committer |   wenjuan dong <dong.wenjuan@zte.com.cn> | 2016-09-08 02:03:17 +0000 |
| commit | c7dcfdf87eccce55647e4d8123374b63ca5cff97 (patch) | |
| tree | 413341f5c643e9bba048d028a88d5445b40fd13b | |
| parent | c225f703e77393f1c2776d537cb583dadea38821 (diff) | |
Unnecessary command and let VM creation stabilize
Method 'wait_for_status' in OpenStack CLI test_server.py suggests that
we might have to wait a little bit longer after server reaches expected
status. Better wait than break.
Change-Id: I4aaa3091ac2e458aa527b6bd0164d65d46c85017
Signed-off-by: Carlos Goncalves <carlos.goncalves@neclab.eu>
(cherry picked from commit 5d2b686e203842c57dae5bf593670dab637751f4)
| -rwxr-xr-x | tests/run.sh | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/tests/run.sh b/tests/run.sh index ee96da9d..2b251690 100755 --- a/tests/run.sh +++ b/tests/run.sh @@ -450,7 +450,12 @@ wait_for_vm_launch() { while [[ ${count} -lt 60 ]] do state=$(openstack $as_doctor_user server list | grep " $VM_NAME " | awk '{print $6}') - [[ "$state" == "ACTIVE" ]] && return 0 + if [[ "$state" == "ACTIVE" ]]; then + # NOTE(cgoncalves): sleeping for a bit to stabilize + # See python-openstackclient/functional/tests/compute/v2/test_server.py:wait_for_status + sleep 5 + return 0 + fi [[ "$state" == "ERROR" ]] && echo "vm state is ERROR" && exit 1 count=$(($count+1)) sleep 1 @@ -560,13 +565,12 @@ create_test_user echo "creating VM..." boot_vm wait_for_vm_launch -openstack $as_doctor_user server show $VM_NAME echo "get computer host info..." get_compute_host_info echo "creating alarm..." -#TODO: change back to use, network problems depends on infra and installers +#TODO: change back to use, network problems depends on infra and installers #get_consumer_ip create_alarm |