[ https://issues.apache.org/jira/browse/CLOUDSTACK-9374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15268154#comment-15268154 ]
ASF GitHub Bot commented on CLOUDSTACK-9374: -------------------------------------------- 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 ---- > Replace timer.sleep in host-delete method to correct timing problem > ------------------------------------------------------------------- > > Key: CLOUDSTACK-9374 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9374 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the > default.) > Components: marvin > Affects Versions: 4.9.0 > Environment: N/A > Reporter: Mike Tutkowski > Assignee: Mike Tutkowski > Fix For: 4.9.0 > > > 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). -- This message was sent by Atlassian JIRA (v6.3.4#6332)