Re: Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-20 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23550/ --- (Updated July 21, 2014, 4:58 a.m.) Review request for cloudstack, daan Hoogland

Re: Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-20 Thread Santhosh Edukulla
> On July 16, 2014, 3:35 p.m., Santhosh Edukulla wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 856 > > > > > > I believe we are verifying the max size of an integer? If yes, use > > limits, somethin

Re: Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-20 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23550/#review48198 --- Ship it! Ship It! - Santhosh Edukulla On July 16, 2014, 12:37 p.

Re: Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-16 Thread Rajani Karuturi
> On July 16, 2014, 3:35 p.m., Santhosh Edukulla wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 856 > > > > > > I believe we are verifying the max size of an integer? If yes, use > > limits, somethin

Re: Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-16 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23550/#review47889 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Review Request 23550: Fixed coverity reported issues in UserVmManagerImpl

2014-07-16 Thread Rajani Karuturi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23550/ --- Review request for cloudstack, daan Hoogland and Santhosh Edukulla. Repository: