Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/#review184091 --- Ship it! Ship It! - Jayush Luniya On Aug. 29, 2017, 7:37

Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/ --- (Updated Aug. 29, 2017, 7:37 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/ --- (Updated Aug. 29, 2017, 7:31 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/ --- (Updated Aug. 29, 2017, 6:28 p.m.) Review request for Ambari, Jayush Luniya

Re: Review Request 61971: Log Search UI GA - merge into trunk

2017-08-29 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61971/#review184073 --- Ship it! Ship It! - Oliver Szabo On Aug. 29, 2017, 4:29

Re: Review Request 61971: Log Search UI GA - merge into trunk

2017-08-29 Thread Andrii Tkach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61971/#review184068 --- Ship it! Ship It! - Andrii Tkach On Aug. 29, 2017, 4:29

Review Request 61971: Log Search UI GA - merge into trunk

2017-08-29 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61971/ --- Review request for Ambari and Andrii Tkach. Bugs: AMBARI-21308

Re: Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Sid Wagle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61969/#review184066 --- Ship it! Ship It! - Sid Wagle On Aug. 29, 2017, 2:55 p.m.,

Re: Review Request 61970: Log Search UI: add filter by hostname for service logs

2017-08-29 Thread Aleksandr Kovalenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61970/#review184065 --- Ship it! Ship It! - Aleksandr Kovalenko On Авг. 29, 2017,

Review Request 61970: Log Search UI: add filter by hostname for service logs

2017-08-29 Thread Andrii Babiichuk
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61970/ --- Review request for Ambari and Aleksandr Kovalenko. Bugs: AMBARI-21845

Re: Review Request 61968: PATCH and MAINT Orchestrations Should Only Invoke stack-select On Specific Packages

2017-08-29 Thread Nate Cole
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61968/#review184061 --- Ship it! Ship It! - Nate Cole On Aug. 29, 2017, 10:42 a.m.,

Re: Review Request 61968: PATCH and MAINT Orchestrations Should Only Invoke stack-select On Specific Packages

2017-08-29 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61968/#review184059 --- Ship it! Ship It! - Dmitro Lisnichenko On Aug. 29, 2017,

Re: Review Request 61715: package_regex in get_package_from_available() can match wrong pkg

2017-08-29 Thread Dmitro Lisnichenko
> On Aug. 28, 2017, 6:56 p.m., Dmytro Grinenko wrote: > > ambari-common/src/main/python/resource_management/libraries/script/script.py > > Line 455 (original), 455 (patched) > > > > > > it looks that this function

Re: Review Request 61715: package_regex in get_package_from_available() can match wrong pkg

2017-08-29 Thread Dmitro Lisnichenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61715/ --- (Updated Aug. 29, 2017, 6:07 p.m.) Review request for Ambari, Dmytro Grinenko

Re: Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61969/ --- (Updated Сер. 29, 2017, 2:55 після полудня) Review request for Ambari,

Re: Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61969/#review184057 --- Ship it! Ship It! - Jonathan Hurley On Aug. 29, 2017, 10:49

Review Request 61969: Database consistency check fails after upgrade to ambari 2.6.0

2017-08-29 Thread Vitalyi Brodetskyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61969/ --- Review request for Ambari, Jonathan Hurley and Sid Wagle. Bugs: AMBARI-21843

Re: Review Request 61968: PATCH and MAINT Orchestrations Should Only Invoke stack-select On Specific Packages

2017-08-29 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61968/#review184055 ---

Review Request 61968: PATCH and MAINT Orchestrations Should Only Invoke stack-select On Specific Packages

2017-08-29 Thread Jonathan Hurley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61968/ --- Review request for Ambari, Dmitro Lisnichenko and Nate Cole. Bugs:

Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/#review184047 --- Ship it! Ship It! - Jayush Luniya On Aug. 28, 2017, 9:08

Re: Review Request 61917: AMBARI-21824. MultiEverything : Make services as sub-resource of ServiceGroups and use Servicegroup while doing Creation, Update and retrieval of HostComponents and ServiceCo

2017-08-29 Thread Jayush Luniya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61917/#review184003 ---

Re: Review Request 61951: Add keberos service for 3.0.0 stack

2017-08-29 Thread Robert Levas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61951/#review184032 --- Ship it! Ship It! - Robert Levas On Aug. 28, 2017, 1:14

Re: Review Request 61951: Add keberos service for 3.0.0 stack

2017-08-29 Thread Robert Levas
> On Aug. 28, 2017, 4:36 p.m., Robert Levas wrote: > > Why is a new added - `1.10.3-30`? The existing version is `1.10.3-10`? > > Robert Levas wrote: > Oops Why is a new _version_ added - `1.10.3-30`? The existing > version is `1.10.3-10`? > > Eugene Chekanskiy wrote: > 3.0 is

Re: Review Request 61951: Add keberos service for 3.0.0 stack

2017-08-29 Thread Eugene Chekanskiy
> On Aug. 28, 2017, 8:36 p.m., Robert Levas wrote: > > Why is a new added - `1.10.3-30`? The existing version is `1.10.3-10`? > > Robert Levas wrote: > Oops Why is a new _version_ added - `1.10.3-30`? The existing > version is `1.10.3-10`? 3.0 is an independed stack and all code

Re: Review Request 61911: Installing mpack with .tgz extension fails with global name 'path' is not defined

2017-08-29 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61911/#review184026 --- Ship it! Ship It! - Sandor Magyari On Aug. 25, 2017, 9:51