[GitHub] cloudstack pull request: CLOUDSTACK-7985: assignVM in Advanced zon...

2015-10-04 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/844#issuecomment-145445260 guys, anyone has update ? --- 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 no

[GitHub] cloudstack pull request: CLOUDSTACK-8923: Do not send zoneId with ...

2015-10-04 Thread borisroman
GitHub user borisroman opened a pull request: https://github.com/apache/cloudstack/pull/911 CLOUDSTACK-8923: Do not send zoneId with createStorageNetworkIpRange command. The zoneId was send with the createStorageNetworkIpRange from the wizard. This isn't necessary. It will only rem

[GitHub] cloudstack pull request: CLOUDSTACK-8915 - Cannot SSH into VMs dep...

2015-10-04 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/908#issuecomment-145441458 @remibergsma Hi, I see that currently we invoke the main function of configure.py every time we call the update_config when a new config to vr is pus

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/909 --- 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] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-10-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/885 --- 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

Re: Citrix References in Marvin

2015-10-04 Thread Rajani Karuturi
Dont know about [1] and [2]. [3]: they are simple data disks 1GB virtual size created from cloudstack and downloaded. they are used in tests for upload volume/template. We do not have a location to share them. Below are the links from my dropbox account. https://www.dropbox.com/s/euqaj7yrpjf6m3a/r

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145436050 I will merge this once travis is done. --- 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 proje

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145434199 commited, squashed and force-pushed the change requested by Rajani --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145423925 one minor comment. otherwise, LGTM (only did code review) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/909#discussion_r41109279 --- Diff: engine/orchestration/src/com/cloud/vm/VirtualMachinePowerStateSyncImpl.java --- @@ -111,7 +111,19 @@ private void processReport(long hostId, Ma

[GitHub] cloudstack pull request: Dockerfile part2

2015-10-04 Thread pdion891
GitHub user pdion891 opened a pull request: https://github.com/apache/cloudstack/pull/910 Dockerfile part2 This PR simplify download of systemvm templates, work with docker-compose and enable integration-api port and use of localstorage without reconfiguration of cloudstack.

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145382427 @karuturi @koushik-das You reviewed PR #885, @dahn has added one commit and made this PR ontop of the previous one. Would you be so kind to review it? Need one m

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145381656 Tested everything I tested in PR #885 again in this PR. @dahn already run the tests, I verified the functionality manually by destoying VMs out-of-band and see h

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145369924 ```bash nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1.cfg -s -a \ tags=advanced,required_hardware=false smoke/test_routers.p

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145324612 one test skipped, that requires two hosts. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] cloudstack pull request: CLOUDSTACK-8848 ensure power state is up ...

2015-10-04 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/909#issuecomment-145324409 Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS === ok Test restart network ... === TestName: test_03