Probably not a bad time to ask if we have put in this check in all the operations - can you please enlist it ?
On 16/04/13 5:17 PM, "Sanjay Tripathi" <sanjay.tripa...@citrix.com> wrote: > >----------------------------------------------------------- >This is an automatically generated e-mail. To reply, visit: >https://reviews.apache.org/r/10541/ >----------------------------------------------------------- > >Review request for cloudstack, Devdeep Singh, Sateesh Chodapuneedi, and >Min Chen. > > >Description >------- > >CLOUDSTACK-1966 Primary Storage Limits are not validated while deploying >the VM. > > >This addresses bug CLOUDSTACK-1966. > > >Diffs >----- > > server/src/com/cloud/vm/UserVmManagerImpl.java d281e5b > >Diff: https://reviews.apache.org/r/10541/diff/ > > >Testing >------- > >Tests: >1. Create a user account. >2. Set the primary storage limit for this user account to 2GiB. >3. Log into the user account. >4. Deploy instance with template size > 2GiB >5. CloudStack displays the appropriate error message. > > >Thanks, > >Sanjay Tripathi >