Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/834
---
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
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/834#issuecomment-160537225
@maneesha-p Probably not cleanly anymore. I'll give it a shot.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user maneesha-p commented on the pull request:
https://github.com/apache/cloudstack/pull/834#issuecomment-160512875
@remibergsma 2 LGTM s . Can this be merged?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/834#issuecomment-152452258
LGTM, based on a set of tests that I run on this branch (which I rebased
myself first):
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -
Github user sureshanaparti commented on the pull request:
https://github.com/apache/cloudstack/pull/834#issuecomment-140641999
Reschedule time is always based on the work type. Code changes are OK. LGTM.
---
If your project is set up for it, you can reply to this email and have your
r
GitHub user maneesha-p opened a pull request:
https://github.com/apache/cloudstack/pull/834
CLOUDSTACK-8866 : restart.retry.interval is being used instead of
migrate.retry.interval during host maintenance
The frequency at which Cloudstack tries to migrate the VMs is currently
contr
