Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-10 Thread bharat kumar
> On Dec. 4, 2013, 2:06 p.m., daan Hoogland wrote: > > Bharat, neither version 4 nor version 3 of your patch apply to master. Also > > I noted the ticket number in the commit comment changed from 5160 to 5161. > > Can you explain. Hi Dann, This change is already shipped to master, closing thi

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-04 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/#review29732 --- Bharat, neither version 4 nor version 3 of your patch apply to maste

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-03 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/#review29707 --- Commit e5cfe948186b825d2b28c99ce2915a5ca8498aff in branch refs/heads

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-03 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/ --- (Updated Dec. 3, 2013, 10:53 a.m.) Review request for cloudstack, Jayapal Reddy

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-03 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/ --- (Updated Dec. 3, 2013, 8:52 a.m.) Review request for cloudstack, Jayapal Reddy

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-02 Thread bharat kumar
> On Nov. 27, 2013, 12:56 p.m., daan Hoogland wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 2724 > > > > > > Is there a guarantee that the parameter is castable to Long? sorry for the delay in reply

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-12-02 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/ --- (Updated Dec. 3, 2013, 6:20 a.m.) Review request for cloudstack, Jayapal Reddy

Re: Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-11-27 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/#review29485 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Review Request 15875: https://issues.apache.org/jira/browse/CLOUDSTACK-5160

2013-11-27 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15875/ --- Review request for cloudstack, Jayapal Reddy and Koushik Das. Bugs: CLOUDSTACK-