Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

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

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/#review27368 --- Ship it! d4072c6 - master - Prasanna Santhanam On Oct. 23, 2013,

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/ --- (Updated Oct. 23, 2013, 8:19 a.m.) Review request for cloudstack and Prasanna S

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Santhosh Edukulla
> On Oct. 23, 2013, 7:25 a.m., Prasanna Santhanam wrote: > > tools/marvin/marvin/cloudstackConnection.py, line 169 > > > > > > can we please use # for inline comments in code and """ quoted > > doc-string comments und

Re: Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-23 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/#review27366 --- tools/marvin/marvin/cloudstackConnection.py

Review Request 14869: Added changes related to bug 4920 and few code clean up

2013-10-22 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14869/ --- Review request for cloudstack and Prasanna Santhanam. Repository: cloudstack-gi