Re: Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-25 Thread Daan Hoogland
September 25, 2014 1:05 AM > > To: Devdeep Singh > > Cc: dev@cloudstack.apache.org > > Subject: Re: Regarding commit > > 7f440854f7bcd41a1bd6581c0239cde2e98261b7 > > > > Devdeep, > > > > Guess I made a booboo. It should have read l

RE: Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-25 Thread Devdeep Singh
Yes, I have made a similar fix and committed the changes. Regards, Devdeep > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Thursday, September 25, 2014 1:05 AM > To: Devdeep Singh > Cc: dev@cloudstack.apache.org > Subject: Re

Re: Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-24 Thread Daan Hoogland
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);

Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-24 Thread Devdeep Singh
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 r