exercise/euca volume timeout is wrong

Bug #1087656 reported by Mate Lakat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
devstack
Fix Released
Undecided
Mate Lakat

Bug Description

Running euca fails with this message:

+ VOLUME=vol-00000001
+ die_if_not_set VOLUME 'Failure to get volume'
+ local exitcode=0
+ set +o xtrace
+ timeout 15 sh -c 'while ! euca-describe-volumes vol-00000001 | grep -q available; do sleep 1; done'
+ echo 'volume didnt become available within 530 seconds'
volume didnt become available within 530 seconds
+ exit 1

It was waiting for 15 seconds, however, the log message was saying 530 seconds.

I guess the comment is right, so we should wait 530 seconds.

Code:
   # Test volume has become available
   if ! timeout $ASSOCIATE_TIMEOUT sh -c "while ! euca-describe-volumes $VOLUME | grep -q available; do sleep 1; done"; then
       echo "volume didnt become available within $RUNNING_TIMEOUT seconds"
       exit 1
   fi

Mate Lakat (mate-lakat)
Changed in devstack:
status: New → In Progress
assignee: nobody → Mate Lakat (mate-lakat)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to devstack (master)

Fix proposed to branch: master
Review: https://review.openstack.org/17674

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to devstack (master)

Reviewed: https://review.openstack.org/17674
Committed: http://github.com/openstack-dev/devstack/commit/1e32d0ab191bfe8a8c89580b9f84fe38ded7af0a
Submitter: Jenkins
Branch: master

commit 1e32d0ab191bfe8a8c89580b9f84fe38ded7af0a
Author: Mate Lakat <email address hidden>
Date: Fri Dec 7 12:46:15 2012 +0000

    exercises/euca: Fix volume timeout

    Fixes bug 1087656

    In euca exercise, the timeout for one of the volume operations was
    specified as ASSOCIATE_TIMEOUT, whereas the timeout error message was
    mentioning RUNNING_TIMEOUT. This fix changes the timeout to
    RUNNING_TIMEOUT so that it is consistent with the error message.

    As RUNNING is usually larger than ASSOCIATE, it leaves more time for the
    volume operation.

    Change-Id: Ic016c7920ae6e4ec9a476bb5612b7df9eed01c75

Changed in devstack:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.