incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "deepti dohare" <deepti.doh...@citrix.com>
Subject Review Request: CS-15430 Create snapshot should fail if creating snapshot results in exceeding snapshot resource limit for domain-admin or user accounts.
Date Fri, 06 Jul 2012 12:43:11 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5806/
-----------------------------------------------------------

Review request for cloudstack.


Description
-------

Change:
1. Before creating the snapshot, we synchronized checkResourcelimit to allow the users to
create the snapshot and increment the resource count.
2. Depending on the failure of snapshot creation/ backup, we are decrementing the resource
count.


This addresses bug CS-15430.


Diffs
-----

  server/src/com/cloud/storage/snapshot/SnapshotManagerImpl.java b97a7d7 

Diff: https://reviews.apache.org/r/5806/diff/


Testing
-------

Steps to verify:
1.Login as admin, set snapshot limit '3' for a user account
2.login as user, create a VM1 with data volume
3.trigger 3 create snapshot command from the above data volume, succeeded
4.create one more snapshot, failed, "maximum limit exceeded for account user"


Thanks,

deepti dohare


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message