----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/#review47065 -----------------------------------------------------------
tools/marvin/marvin/configGenerator.py <https://reviews.apache.org/r/23169/#comment82669> Does cfg, files used has these option of domain? As well, please test it on both 4.4-forward and master, if it need to go to both? - Santhosh Edukulla On June 30, 2014, 2:30 p.m., John Dilley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23169/ > ----------------------------------------------------------- > > (Updated June 30, 2014, 2:30 p.m.) > > > Review request for cloudstack and Santhosh Edukulla. > > > Repository: cloudstack-git > > > Description > ------- > > Add domain to createZones in deployDataCenter > > > Diffs > ----- > > tools/marvin/marvin/configGenerator.py 8daaaa9 > tools/marvin/marvin/deployDataCenter.py c097238 > > Diff: https://reviews.apache.org/r/23169/diff/ > > > Testing > ------- > > Tested deployDataCenter, both with and without specifying a domain > > > Thanks, > > John Dilley > >
