You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by borisroman <gi...@git.apache.org> on 2015/10/26 11:06:42 UTC
[GitHub] cloudstack pull request: CLOUDSTACK-8989 test_ps_limits.py tests c...
GitHub user borisroman opened a pull request:
https://github.com/apache/cloudstack/pull/979
CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic" setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup. This PR adds that functionality.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/borisroman/cloudstack CLOUDSTACK-8989
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/979.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 #979
----
commit 1834ea673a77354a820838d2d6d2de995f87a1f1
Author: Boris Schrijver <bo...@pcextreme.nl>
Date: 2015-10-26T10:05:31Z
CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic" setup.
----
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151460148
Hi @borisroman ,
I just finish the tests: All looking fine!
```
[root@cs3 integration]# nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone3-kvm3-basic.cfg -s -a tags=basic,required_hardware=false component/test_ps_limits.py
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_27_2015_10_22_08_R41BV1. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_assign_vm_different_account_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_assign_vm_different_account_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_2_child_domain_admin | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_ps_limits_TTWIAZ===
```
I will also run "test_vm_life_cycle.py". Will let you know once done.
Ping @remibergsma @DaanHoogland @karuturi - are anyone of you capable of running/LGTMing this PR? :)
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-8989 test_ps_limits.py tests c...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151145630
Hi @borisroman
Although my first round of tests failed, after looking into the test code it seems to be environment related:
```
Test assign Vm to different account ... === TestName: test_assign_vm_different_account_1_root_domain_admin | Status : SUCCESS ===
ok
Test assign Vm to different account ... === TestName: test_assign_vm_different_account_2_child_domain_admin | Status : SUCCESS ===
ok
Stop attach and detach volume from VM ... === TestName: test_attach_detach_volume_1_root_domain_admin | Status : SUCCESS ===
ok
Stop attach and detach volume from VM ... === TestName: test_attach_detach_volume_2_child_domain_admin | Status : SUCCESS ===
ok
Test create multiple volumes ... === TestName: test_create_multiple_volumes_1_root_domain_admin | Status : FAILED ===
FAIL
Test create multiple volumes ... === TestName: test_create_multiple_volumes_2_child_domain_admin | Status : FAILED ===
FAIL
Test Deploy multiple VMs with & verify the usage ... === TestName: test_deploy_multiple_vm_1_root_domain_admin | Status : SUCCESS ===
ok
Test Deploy multiple VMs with & verify the usage ... === TestName: test_deploy_multiple_vm_2_child_domain_admin | Status : SUCCESS ===
ok
Test delete and recover instance ... SKIP: skip
Test delete and recover instance ... SKIP: skip
Test Deploy VM with 5 GB volume & verify the usage ... === TestName: test_stop_start_vm_1_root_domain_admin | Status : SUCCESS ===
ok
Test Deploy VM with 5 GB volume & verify the usage ... === TestName: test_stop_start_vm_2_child_domain_admin | Status : SUCCESS ===
ok
======================================================================
FAIL: Test create multiple volumes
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/ddt.py", line 146, in wrapper
return func(self, *args, **kwargs)
File "/data/git/cs3/cloudstack/test/integration/component/test_ps_limits.py", line 381, in test_create_multiple_volumes
self.assertEqual(response[0], PASS, response[1])
AssertionError: Resource count 17 should match with the expected resource count 12
```
I will try cleaning the my primary/secondary storage and running the tests again.
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151556191
@remibergsma Jenkins happy :)
https://builds.apache.org/job/cloudstack-pull-analysis/1008/
---
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-8989 test_ps_limits.py tests c...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151104355
thanks for the tests, @karuturi! I will test this PR.
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151798750
LGTM, test runs fine:
```
=== TestName: test_deploy_vm | Status : SUCCESS ===
=== TestName: test_deploy_vm_multiple | Status : SUCCESS ===
=== TestName: test_01_stop_vm | Status : SUCCESS ===
=== TestName: test_02_start_vm | Status : SUCCESS ===
=== TestName: test_03_reboot_vm | Status : SUCCESS ===
=== TestName: test_06_destroy_vm | Status : SUCCESS ===
=== TestName: test_07_restore_vm | Status : SUCCESS ===
=== TestName: test_09_expunge_vm | Status : SUCCESS ===
```
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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-8989 test_ps_limits.py tests c...
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151116939
Jenkins failed due to the following tests:
```
org.apache.cloudstack.utils.hypervisor.HypervisorUtilsTest.checkVolumeFileForActivityTest
```
https://builds.apache.org/job/cloudstack-pull-analysis/980/org.apache.cloudstack$cloud-utils/
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151467948
Ping @borisroman @remibergsma @karuturi @DaanHoogland
This PR LGTM :+1:
```
[root@cs3 integration]# nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone3-kvm3-basic.cfg -s -a tags=basic,required_hardware=false smoke/test_vm_life_cycle.py
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_27_2015_11_21_04_UVR6BU. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_deploy_vm | Status : SUCCESS ===
=== TestName: test_deploy_vm_multiple | Status : SUCCESS ===
=== TestName: test_01_stop_vm | Status : SUCCESS ===
=== TestName: test_02_start_vm | Status : SUCCESS ===
=== TestName: test_03_reboot_vm | Status : SUCCESS ===
=== TestName: test_06_destroy_vm | Status : SUCCESS ===
=== TestName: test_07_restore_vm | Status : SUCCESS ===
=== TestName: test_09_expunge_vm | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_vm_life_cycle_CPQFDY===
```
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151755463
Ping @remibergsma @DaanHoogland
Any time to test this one today?
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by borisroman <gi...@git.apache.org>.
GitHub user borisroman reopened a pull request:
https://github.com/apache/cloudstack/pull/979
[4.6] CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic" setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup. This PR adds that functionality.
```
nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-basic.cfg -s -a tags=basic,required_hardware=false smoke/test_vm_life_cycle.py
```
```
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_26_2015_09_55_20_4UDZRZ. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_deploy_vm | Status : SUCCESS ===
=== TestName: test_deploy_vm_multiple | Status : SUCCESS ===
=== TestName: test_01_stop_vm | Status : SUCCESS ===
=== TestName: test_02_start_vm | Status : SUCCESS ===
=== TestName: test_03_reboot_vm | Status : SUCCESS ===
=== TestName: test_06_destroy_vm | Status : SUCCESS ===
=== TestName: test_07_restore_vm | Status : SUCCESS ===
=== TestName: test_09_expunge_vm | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_vm_life_cycle_2GU6K4===
```
```
nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-basic.cfg -s -a tags=basic,required_hardware=false component/test_ps_limits.py
```
```
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_26_2015_10_19_51_ZY3KD5. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_assign_vm_different_account_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_assign_vm_different_account_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_2_child_domain_admin | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_ps_limits_0E1HFP===
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/borisroman/cloudstack CLOUDSTACK-8989
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/979.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 #979
----
commit 730173041e4c391b78ba3be2dd5d016fff644cae
Author: Boris Schrijver <bo...@pcextreme.nl>
Date: 2015-10-27T13:27:18Z
CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a basic setup.
----
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by wilderrodrigues <gi...@git.apache.org>.
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151210234
I have talked to @borisroman and he explained me that this PR depends on the fixes on PR #860 from @karuturi . I will merge both and run the tests again.
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by borisroman <gi...@git.apache.org>.
Github user borisroman closed the pull request at:
https://github.com/apache/cloudstack/pull/979
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by borisroman <gi...@git.apache.org>.
GitHub user borisroman reopened a pull request:
https://github.com/apache/cloudstack/pull/979
[4.6] CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a "basic" setup.
The tests in the test_ps_limits.py file can all be run on a "basic" setup. This PR adds that functionality.
```
nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-basic.cfg -s -a tags=basic,required_hardware=false smoke/test_vm_life_cycle.py
```
```
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_26_2015_09_55_20_4UDZRZ. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_deploy_vm | Status : SUCCESS ===
=== TestName: test_deploy_vm_multiple | Status : SUCCESS ===
=== TestName: test_01_stop_vm | Status : SUCCESS ===
=== TestName: test_02_start_vm | Status : SUCCESS ===
=== TestName: test_03_reboot_vm | Status : SUCCESS ===
=== TestName: test_06_destroy_vm | Status : SUCCESS ===
=== TestName: test_07_restore_vm | Status : SUCCESS ===
=== TestName: test_09_expunge_vm | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_vm_life_cycle_2GU6K4===
```
```
nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-basic.cfg -s -a tags=basic,required_hardware=false component/test_ps_limits.py
```
```
==== Marvin Init Started ====
=== Marvin Parse Config Successful ===
=== Marvin Setting TestData Successful===
==== Log Folder Path: /tmp//MarvinLogs//Oct_26_2015_10_19_51_ZY3KD5. All logs will be available here ====
=== Marvin Init Logging Successful===
==== Marvin Init Successful ====
=== TestName: test_assign_vm_different_account_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_assign_vm_different_account_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_attach_detach_volume_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_create_multiple_volumes_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_deploy_multiple_vm_2_child_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_1_root_domain_admin | Status : SUCCESS ===
=== TestName: test_stop_start_vm_2_child_domain_admin | Status : SUCCESS ===
===final results are now copied to: /tmp//MarvinLogs/test_ps_limits_0E1HFP===
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/borisroman/cloudstack CLOUDSTACK-8989
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/979.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 #979
----
commit a24cbafb15da0159664be344d2981b807c37d6bf
Author: Boris Schrijver <bo...@pcextreme.nl>
Date: 2015-10-27T13:30:58Z
CLOUDSTACK-8989 test_ps_limits.py tests can also be run on a basic setup.
----
---
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: [4.6] CLOUDSTACK-8989 test_ps_limits.py t...
Posted by remibergsma <gi...@git.apache.org>.
Github user remibergsma commented on the pull request:
https://github.com/apache/cloudstack/pull/979#issuecomment-151476883
@borisroman If you make Jenkins happy, I'll try to run this test as well ;-)
---
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.
---