[ https://issues.apache.org/jira/browse/CLOUDSTACK-8889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14900502#comment-14900502 ]
ASF GitHub Bot commented on CLOUDSTACK-8889: -------------------------------------------- GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/860 CLOUDSTACK-8889: delete volume doesnt decrement primary store resource count 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 The resource counts are updated even before the disk is actually deleted resulting in the same value. Moved the resource counts updation to after the expunge operation as thats when the disk is actually deleted. Testing: Earlier, test_create_multiple_volumes in test/integration/component/test_ps_limits.py failed with error AssertionError: Resource count 37 should match with the expected resource count 32 Before 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 After the Fix Test create multiple volumes ... === TestName: test_create_multiple_volumes_1_root_domain_admin | Status : SUCCESS === ok Test create multiple volumes ... === TestName: test_create_multiple_volumes_2_child_domain_admin | Status : SUCCESS === ok ---------------------------------------------------------------------- Ran 2 tests in 334.823s OK You can merge this pull request into a Git repository by running: $ git pull https://github.com/karuturi/cloudstack CLOUDSTACK-8889 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/860.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 #860 ---- commit 4e56dfbfbfa82802f71717b333728d8b03d7892a Author: Rajani Karuturi <rajani.karut...@citrix.com> Date: 2015-09-21T10:41:15Z CLOUDSTACK-8889: delete volume doesnt decrement primary store resource count 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 The resource counts are updated even before the disk is actually deleted resulting in the same value. Moved the resource counts updation to after the expunge operation as thats when the disk is actually deleted. all the tests in test/integration/component/test_ps_limits.py now pass ---- > 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: Task > Security Level: Public(Anyone can view this level - this is the > default.) > Reporter: 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. > http://ccp-tests.xenrt.xs.citrite.net/jenkins/job/Guadalupe-Regression-Multi/CLOUDSTACK_CONFIG=AdvXS/23/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)