Re: Review Request 14925: Added few misc changes to marvin

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

Re: Review Request 14925: Added few misc changes to marvin

2013-10-28 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14925/#review27601 --- Ship it! 03830c5 -> master Sorry to be an ass about this, but git

Re: Review Request 14925: Added few misc changes to marvin

2013-10-28 Thread Santhosh Edukulla
> On Oct. 28, 2013, 5:55 a.m., Prasanna Santhanam wrote: > > tools/marvin/marvin/marvinPlugin.py, line 148 > > > > > > Any reason to remove the debug logger? This will cause the write of all > > logs without timestamp

Re: Review Request 14925: Added few misc changes to marvin

2013-10-27 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14925/#review27589 --- tools/marvin/marvin/marvinPlugin.py

Review Request 14925: Added few misc changes to marvin

2013-10-25 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14925/ --- Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam. Reposit