Devdeep,

Guess I made a booboo. It should have read like this , agree?

                    cmd = resource.getCurrentStatus(_id);
                    while ( cmd == null && ++retried <
_HostPingRetryCount.value())
                    {
                        cmd = resource.getCurrentStatus(_id);
                        Thread.sleep(1000*_HostPingRetryTimer.value());
                    }


On Wed, Sep 24, 2014 at 4:22 PM, Devdeep Singh <devdeep.si...@citrix.com>
wrote:

> Hi Daan,
>
> I am looking into bug [1] which reports that the vmsync functionality is
> broken on master. If a vm deployed by cloudstack is stopped directly on the
> hypervisor, its state is not updated in cloudstack. I see that in commit
> 7f440854f7bcd41a1bd6581c0239cde2e98261b7, a change was made to retry
> retrieving the resource status (PingCommand) to deal with network glitches.
> However, there is an issue with the commit and it has caused a regression
> with the vmsync functionality. Now, by default, the PingTask is not
> checking for the status of the resource. This breaks vmsync for all direct
> connected agents, which includes xenserver, hyper-v etc.
>
> I'll be submitting a fix for this issue. Do let me know if you have any
> concerns with it.
>
> [1] https://issues.apache.org/jira/browse/CLOUDSTACK-7598
>
> Regards,
> Devdeep
>



-- 
Daan

Reply via email to