You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by sanju1010 <gi...@git.apache.org> on 2015/09/30 11:58:23 UTC

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

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 https://github.com/sanju1010/cloudstack scale_vm

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/900.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #900
    
----
commit 33bdfc773a39d7ff245fb4f50299a7b6cc0391ef
Author: sanjeev <sa...@apache.org>
Date:   2015-09-30T09:53:30Z

    CLOUDSTACK-8924: Removed duplicate test from test_scale_vm.py

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by sanju1010 <gi...@git.apache.org>.
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=basic,required_hardware=false /root/cloudstack/test/integration/smoke/test_scale_vm.py
    
    ==== Marvin Init Started ====
    
    === Marvin Parse Config Successful ===
    
    === Marvin Setting TestData Successful===
    
    ==== Log Folder Path: /tmp//MarvinLogs//Sep_30_2015_08_06_32_AWNF1O. All logs will be available here ====
    
    === Marvin Init Logging Successful===
    
    ==== Marvin Init Successful ====
    ===final results are now copied to: /tmp//MarvinLogs/test_scale_vm_OPS7AD===
    root@localhost:~/cloudstack# cd /tmp//MarvinLogs/test_scale_vm_OPS7AD
    root@localhost:/tmp/MarvinLogs/test_scale_vm_OPS7AD# ls
    failed_plus_exceptions.txt  results.txt  runinfo.txt
    root@localhost:/tmp/MarvinLogs/test_scale_vm_OPS7AD# vi  results.txt
    root@localhost:/tmp/MarvinLogs/test_scale_vm_OPS7AD# ls -al
    total 48
    drwxr-xr-x 2 root root  4096 Sep 30 08:07 .
    drwxr-xr-x 8 root root  4096 Sep 30 08:06 ..
    -rw-r--r-- 1 root root     0 Sep 30 08:06 failed_plus_exceptions.txt
    -rw-r--r-- 1 root root   186 Sep 30 08:06 results.txt
    -rw-r--r-- 1 root root 36164 Sep 30 08:06 runinfo.txt
    root@localhost:/tmp/MarvinLogs/test_scale_vm_OPS7AD#
    root@localhost:/tmp/MarvinLogs/test_scale_vm_OPS7AD# cat results.txt
    Test scale virtual machine ... === TestName: test_01_scale_vm | Status : SUCCESS ===
    ok
    
    ----------------------------------------------------------------------
    Ran 1 test in 23.455s
    
    OK


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by borisroman <gi...@git.apache.org>.
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
    
    ==== Marvin Init Started ====
    
    === Marvin Parse Config Successful ===
    
    === Marvin Setting TestData Successful===
    
    ==== Log Folder Path: /tmp//MarvinLogs//Oct_01_2015_11_34_15_WWBMZ3. All logs will be available here ====
    
    === Marvin Init Logging Successful===
    
    ==== Marvin Init Successful ====
    === TestName: test_01_scale_vm | Status : SUCCESS ===
    
    ===final results are now copied to: /tmp//MarvinLogs//NIMUQD===
    
    :+1: 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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by wilderrodrigues <gi...@git.apache.org>.
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.
    
    Could you please get this one merged?
    
    Cheers,
    Wilder


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by pvr9711 <gi...@git.apache.org>.
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by remibergsma <gi...@git.apache.org>.
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 this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by asfgit <gi...@git.apache.org>.
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by borisroman <gi...@git.apache.org>.
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 does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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

Posted by koushik-das <gi...@git.apache.org>.
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 your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---