Re: Review Request 14595: Adding https support to marvin

2013-11-08 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14595/#review28521 --- 4.2: 6e1821f585df5c643873ab0ce1e7d91d1d94f00d - Girish Shilamkar

Re: Review Request 14595: Adding https support to marvin

2013-10-21 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14595/#review27278 --- Ship it! Thanks for the patch! dfa0678 on master. Please ensure yo

Re: Review Request 14595: Adding https support to marvin

2013-10-20 Thread Santhosh Edukulla
> On Oct. 18, 2013, 12:36 p.m., daan Hoogland wrote: > > The params are self explanatory indeed, i like them. The concern I have > > with them is regarding backward compatibility of the interface. I don't > > (yet) use the BVT suite due to lack of knowledge on my side. I use the > > connection

Re: Review Request 14595: Adding https support to marvin

2013-10-18 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14595/#review27185 --- The params are self explanatory indeed, i like them. The concern I h

Re: Review Request 14595: Adding https support to marvin

2013-10-11 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14595/ --- (Updated Oct. 11, 2013, 12:51 p.m.) Review request for cloudstack, daan Hooglan