Re: Marvin refactoring

2013-12-13 Thread sebgoa
t's see if we need to revive this thread http://markmail.org/thread/a3rha4zwetfngqxb > > Please feel free to add your comments to the reviews, wherever you feel it > can be fixed better\improved and as well log bugs under Automation\Marvin and > assign them to me. > >

Re: Marvin refactoring

2013-12-12 Thread Prasanna Santhanam
On Wed, Dec 11, 2013 at 10:12:26AM +0100, sebgoa wrote: > > We should also have an open discussion about the import of > https://github.com/vogxn/cloud-autodeploy in > marvin/marvin/misc/build. That repo from prasanna while part of the > CI infra is really custom config for Citrix internal infra (

RE: Marvin refactoring

2013-12-12 Thread Santhosh Edukulla
bugs under Automation\Marvin and assign them to me. Regards, Santhosh From: sebgoa [run...@gmail.com] Sent: Wednesday, December 11, 2013 4:12 AM To: Santhosh Edukulla; gir...@clogeny.com; dev@cloudstack.apache.org Cc: Prasanna Santhanam Subject: Mar

RE: Marvin refactoring

2013-12-11 Thread Animesh Chaturvedi
+1 -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Wednesday, December 11, 2013 1:12 AM To: Santhosh Edukulla; gir...@clogeny.com; dev@cloudstack.apache.org Cc: Prasanna Santhanam Subject: Marvin refactoring Hi devs and Santosh and Girish, I am looking at Marvin lately

RE: Marvin refactoring

2013-12-11 Thread Rayees Namathponnan
: dev@cloudstack.apache.org; Santhosh Edukulla; gir...@clogeny.com Cc: Prasanna Santhanam Subject: RE: Marvin refactoring +1 for running on separate branch before check in -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Wednesday, December 11, 2013 1:12 AM To: Santhosh

RE: Marvin refactoring

2013-12-11 Thread Sudha Ponnaganti
+1 for running on separate branch before check in -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Wednesday, December 11, 2013 1:12 AM To: Santhosh Edukulla; gir...@clogeny.com; dev@cloudstack.apache.org Cc: Prasanna Santhanam Subject: Marvin refactoring Hi devs and

Marvin refactoring

2013-12-11 Thread sebgoa
going straight in 4.2, 4.3 and master. How about creating a marvin refactoring branch and working there ? I believe that what is being done to Marvin is significant enough (renaming of files, change of logger structures, change of function names, addition of constants etc..) that it shou