[
https://issues.apache.org/jira/browse/CLOUDSTACK-8889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14973977#comment-14973977
]
ASF GitHub Bot commented on CLOUDSTACK-8889:
--------------------------------------------
Github user borisroman commented on the pull request:
https://github.com/apache/cloudstack/pull/860#issuecomment-151089307
@remibergsma @karuturi I tested this PR with the changes in #979. Results:
```
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===
```
Based on the test: LGTM :+1:
> Primary Storage count for an account does not decrease when a Data Disk is
> deleted belonging to the account unless the VM to which volume belonged is
> destroyed
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: CLOUDSTACK-8889
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8889
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Affects Versions: 4.5.2, 4.6.0
> Reporter: Rajani Karuturi
> Assignee: Rajani Karuturi
>
> Steps to reproduce:
> 1. Create an account, deploy a VM in it
> 2. Create a data disk and attach to VM
> 3. Check the primary storage count for the account with listAccounts API
> 4. Delete the Data disk
> 5. Check the primary storage count for the account with listAccounts API - It
> is the same as before deleting the data disk
> It should decrease when data disk is deleted.
> Now delete the VM. You can see the primary storage count is decreased by
> (root volume size + data disk size)
> Primary Storage Limits test cases are failing due to this.
> testReport/junit/integration.component.test_ps_limits/TestVolumeLimits/test_create_multiple_volumes_1_root_domain_admin/
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
