> On July 12, 2013, 5:24 a.m., Prasanna Santhanam wrote: > > Thanks for reviewing and reconsidering. it makes sense to avoid putting the > > hosts into the config file. can you perhaps share the sample vmware config? > > Would be illustrative to put it into the sandbox
We need to put host in config file, since we are reading config file to refresh the host In this case need to consider this patch again. Prasanna - any thoughts ? - Rayees ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12481/#review23059 ----------------------------------------------------------- On July 11, 2013, 11:57 p.m., Rayees Namathponnan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12481/ > ----------------------------------------------------------- > > (Updated July 11, 2013, 11:57 p.m.) > > > Review request for cloudstack, edison su and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > In Vmware zone, we no need add host; host will added as part of cluster > > > Diffs > ----- > > tools/marvin/marvin/deployDataCenter.py ac96d09 > > Diff: https://reviews.apache.org/r/12481/diff/ > > > Testing > ------- > > Tested > > > Thanks, > > Rayees Namathponnan > >
