GitHub user mike-tutkowski opened a pull request:
https://github.com/apache/cloudstack/pull/1529
Marvin: Replace a timer.sleep(30) with pulling logic
https://issues.apache.org/jira/browse/CLOUDSTACK-9374
From the ticket:
In the base.py file, there is a Host class with a delete instance method.
This method first attempts to transition the host into the maintenance
resource state.
The first step in this process is to transition the host into the
prepare-for-maintenance resource state.
A while later, the host can be transitioned completely into the maintenance
resource state.
In an attempt to wait for this transition to occur, the delete method has a
timer.sleep(30) call.
The hope is that the host will have transitioned from the
prepare-for-maintenance resource state to the maintenance resource state within
30 seconds, but this does not always happen.
We should correct this problem by putting in logic to query the management
server for the resource state of the host. If it's in the expected state, move
on; else, sleep for a bit and try again (up to a certain limit).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mike-tutkowski/cloudstack marvin_replace_sleep
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1529.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1529
----
commit 6945d3780b2a01c28ab08d66e0a320082ca4709e
Author: Mike Tutkowski <mike.tutkow...@solidfire.com>
Date: 2016-05-03T04:38:55Z
Replace a timer.sleep(30) with pulling logic
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
