Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-21 Thread Amruta Borkar
> On Feb. 21, 2017, 5:41 p.m., Sandor Magyari wrote: > > Ship It! Thank you Sandor, I don't have commit rights on trunk. Could you please help me push this to trunk? - Amruta --- This is an automatically generated e-mail. To reply,

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-21 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review166193 --- Ship it! Ship It! - Sandor Magyari On Feb. 20, 2017, 7:04

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-20 Thread Amruta Borkar
> On Feb. 16, 2017, 12:05 p.m., Sandor Magyari wrote: > > Yes Robert is right, so there's no such validation because in general is > > not necessary to have all hosts from all hostgroups. For ex. let's say you > > have a 'worker' hostgroup which contains only Slave or Client components > >

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-16 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review165833 --- Yes Robert is right, so there's no such validation because in

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-15 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review165711 --- Fix it, then Ship it! Overall, the patch looks fine, but I do

Re: Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-14 Thread Di Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/#review165566 --- Ship it! Ship It! - Di Li On Feb. 14, 2017, 12:37 a.m.,

Review Request 56627: Cluster deployment fails if cluster template has fewer host_groups than that in blueprint

2017-02-13 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/ --- Review request for Ambari, Di Li and Robert Nettleton. Bugs: AMBARI-19991