Re: Review Request 23735: Fix deployment of data center with marvin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/#review79037 --- Thank you for submitting your CloudStack contribution through review

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/marvinLog.py, line 168 > > > > > > Make it more abstract and see if is not aware of cfg(log_cfg), i mean > > pass the logfile path, as si

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/marvinLog.py, line 168 > > > > > > Make it more abstract and see if is not aware of cfg(log_cfg), i mean > > pass the logfile path, as si

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/marvinLog.py, line 168 > > > > > > Make it more abstract and see if is not aware of cfg(log_cfg), i mean > > pass the logfile path, as si

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/marvin/marvin/deployDataCenter.py, line 1099 > > > > > > Instead of this, can we make "logger" node under devcloud.cfg as > > similar to setup/dev/adv

Re: Review Request 23735: Fix deployment of data center with marvin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/ --- (Updated July 22, 2014, 7:24 a.m.) Review request for cloudstack, daan Hoogland

Re: Review Request 23735: Fix deployment of data center with marvin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/ --- (Updated July 22, 2014, 7:24 a.m.) Review request for cloudstack, daan Hoogland

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:56 p.m., Santhosh Edukulla wrote: > > tools/devcloud/devcloud-advanced.cfg, line 133 > > > > > > Is this change required, compared to earlier devcloud cfg, it was a > > working cfg for devcloud.

Re: Review Request 23735: Fix deployment of data center with marvin

> On July 21, 2014, 1:58 p.m., Hugo Trippaers wrote: > > tools/devcloud/devcloud-advanced.cfg, line 133 > > > > > > For devcloud deployments the mgtSvrIp should be .10 as the management > > server is running on the de

Re: Review Request 23735: Fix deployment of data center with marvin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/#review48217 --- tools/devcloud/devcloud-advanced.cfg

Re: Review Request 23735: Fix deployment of data center with marvin

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23735/#review48216 --- tools/devcloud/devcloud-advanced.cfg