Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/936
---
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 borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/936#issuecomment-150578738
@remibergsma Did the same :) KVM Basic setup though. Integeration test:
smoke/test_vm_life_cycle.py returned all success!
LGTM :+1:
---
If your project
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/936#issuecomment-150572549
LGTM, built infra and run this test:
```
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a
tags=advanced,required_hardware=false smoke/test_
Github user wido commented on the pull request:
https://github.com/apache/cloudstack/pull/936#issuecomment-149136968
With @DaanHoogland code-wise is LGTM
---
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 doe
Github user DaanHoogland commented on the pull request:
https://github.com/apache/cloudstack/pull/936#issuecomment-148669695
Code LGTM, as testing is a drag at this moment I think we should put this
in a batch and test a combination of small PRs to report on. This one is
candidate!
GitHub user ustcweizhou opened a pull request:
https://github.com/apache/cloudstack/pull/936
Add agent name in received response
Before change:
2015-10-15 12:55:34,268 DEBUG [c.c.a.t.Request]
(RouterStatusMonitor-1:ctx-d2b917a4) Seq 20-2693152577167557361: Sending { Cmd
,
