[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-10-01 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144701793 Just merged this. --- 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 t

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-10-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/900 --- 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-8924: Removed duplicate test f...

2015-10-01 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144682751 @wilderrodrigues I can't merge... --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-10-01 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144682365 @remibergsma @borisroman also had a look at this one, which is marked as blocker - although is just about a test that wasn't properly executed.

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-10-01 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144680800 [root@cs1 smoke]# nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-xen1-basic.cfg -s -a tags=basic,required_hardware=false test_scale_vm

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144348817 LGTM. Since now test_01_scale_vm can be run on simulator, no need for the other one. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144346102 LGTM! Thanks Sanjeev! --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144344567 root@localhost:~/cloudstack# nosetests --with-marvin --marvin-config=/root/cloudstack/setup/dev/local1.cfg --zone=Sandbox-simulator --hypervisor=simulator -a tags=

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/900 CLOUDSTACK-8924: Removed duplicate test from test_scale_vm.py Please go through CS-8924 for more details. You can merge this pull request into a Git repository by running: $ git pull http