Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1014#issuecomment-152964107
test passed, only one hypervisor used so migration was skipped:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/1014#issuecomment-152987503
@wilderrodrigues warrants a new marvin config, right? or just a test?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/1014
---
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
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/1014#issuecomment-152984837
Hi @DaanHoogland @remibergsma @ustcweizhou, the change LGTM! :+1:
The tests I have use an isolated network with default egress as "allow", so
that
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/1014
CLOUDSTACK-9013: Virtual router failed to start on KVM
This fix a typo of commit 4a177031b055f3649e3b4a00c80eddb5cafa1dd7
You can merge this pull request into a Git repository by running:
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/1014#issuecomment-152624797
Ping @wilderrodrigues to have a look
---
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