Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-08 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/#review47453 --- Commit 63ba1ddfb9ddfb47c7e0c564fea3ad8edf2bd19a in cloudstack's bran

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-08 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/ --- (Updated July 8, 2014, 9:48 a.m.) Review request for cloudstack and Santhosh Ed

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-08 Thread John Dilley
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 91 > > > > > > Does cfg, files used has these option of domain? > > > > As well, please test it on both 4.4-f

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread Santhosh Edukulla
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 91 > > > > > > Does cfg, files used has these option of domain? > > > > As well, please test it on both 4.4-f

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 91 > > > > > > Does cfg, files used has these option of domain? > > > > As well, please test it on both 4.4-f

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread Santhosh Edukulla
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 91 > > > > > > Does cfg, files used has these option of domain? > > > > As well, please test it on both 4.4-f

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > Tested on 4.4-forward and master - both work. - John --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/#review47065 ---

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
> On July 1, 2014, 3 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/configGenerator.py, line 91 > > > > > > Does cfg, files used has these option of domain? > > > > As well, please test it on both 4.4-f

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-01 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/#review47065 --- tools/marvin/marvin/configGenerator.py

Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-06-30 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/ --- Review request for cloudstack and Santhosh Edukulla. Repository: cloudstack-git