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

(Updated July 6, 2012, 8:40 p.m.)


Review request for cloudstack.


Description (updated)
-------

When an instance is created with data volume, account limit for this data 
volume was not checked against. Adding a check for that


This addresses bug CS-15429.


Diffs
-----

  server/src/com/cloud/vm/UserVmManagerImpl.java d81ed7d 

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


Testing
-------

Steps: 
1. create domains /ROOT/d1 + domain-admin account d1domain /ROOT/d2 + user 
account d2user 
2. login admin, set all resource limits for d1domain to be 2, set all resource 
limits for d2user to be 3 
3. login d1domain, create VM 1 with no data volume: succeeded, create VM 2 with 
data volume, Failed. Error message: Maximum number of resources of type 
'volume' for account name=d1domain in domain id=2 has been exceeded.
4. login d2user, create VM 1 and VM2 both with no data volume: succeeded, 
create VM 3 with data volume:failed, 
   Error message: "Maximum number of resources of type 'volume' for account 
name=d2user in domain id=3 has been exceeded"


Thanks,

deepti dohare

Reply via email to