cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mike-tutkowski <...@git.apache.org>
Subject [GitHub] cloudstack pull request: Marvin: Replace a timer.sleep(30) with pu...
Date Tue, 03 May 2016 05:50:39 GMT
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.tutkowski@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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message