> On April 17, 2013, 6:21 a.m., Devdeep Singh wrote: > > Ship It! > > Devdeep Singh wrote: > Changes look fine. Someone with the right karma needs to merge the change.
merged - Abhinandan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10541/#review19304 ----------------------------------------------------------- On April 16, 2013, 11:47 a.m., Sanjay Tripathi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10541/ > ----------------------------------------------------------- > > (Updated April 16, 2013, 11:47 a.m.) > > > 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 > >