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

2017-02-22 Thread Sandor Magyari
> On Feb. 21, 2017, 5:41 p.m., Sandor Magyari wrote: > > Ship It! > > Amruta Borkar wrote: > Thank you Sandor, I don't have commit rights on trunk. Could you please > help me push this to trunk? Committed to trunk, you can close the request. - Sandor ---

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, vis

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 p.

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

2017-02-20 Thread Amruta Borkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56627/ --- (Updated Feb. 20, 2017, 7:04 p.m.) Review request for Ambari, Di Li, Robert Net

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 > > and

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 gene

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 h

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., Amru