----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48348/#review136622 -----------------------------------------------------------
Ship it! Ship It! - Miklos Gergely On June 7, 2016, 4:06 p.m., Oliver Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48348/ > ----------------------------------------------------------- > > (Updated June 7, 2016, 4:06 p.m.) > > > Review request for Ambari, Don Bosco Durai, Miklos Gergely, Robert Nettleton, > Sumit Mohanty, and Sebastian Toader. > > > Bugs: AMBARI-17089 > https://issues.apache.org/jira/browse/AMBARI-17089 > > > Repository: ambari > > > Description > ------- > > - Change logfeeder process/files to use sudo user instead of > logfeeder/logfeeder user/group (to make sure logfeeder can read any kind of > the logs). > - solr and logsearch user both moved to hadoop group > > > Diffs > ----- > > > ambari-common/src/main/python/resource_management/libraries/functions/solr_cloud_util.py > b099a1e > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > 47cedef > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/params.py > 1d40ae1 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml > 46ac4c2 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-env.xml > 7943cd0 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-properties.xml > 65dc378 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-solr-env.xml > 73fecb6 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logfeeder.py > c0689f3 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logsearch.py > 2b5fdf7 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logsearch_common.py > d0ac389 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logsearch_solr.py > b55f3d6 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py > 34583ba > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logfeeder.py > 5ca2bd5 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py > 58239c7 > > ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch_solr.py > 6e71334 > ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py > c9a8de0 > ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_logfeeder.py > 54e08e4 > ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_logsearch.py > bfe6921 > ambari-server/src/test/python/stacks/2.4/LOGSEARCH/test_solr.py 0590dca > ambari-server/src/test/python/stacks/2.5/ATLAS/test_atlas_server.py fd75bf7 > ambari-server/src/test/python/stacks/2.5/configs/default.json 1015593 > ambari-web/app/data/HDP2/site_properties.js 794da25 > > Diff: https://reviews.apache.org/r/48348/diff/ > > > Testing > ------- > > lot of ambari-server python errors with global name 'is_empty' is nod defined > error, but it is not related with my change. logsearch related tests passed. > > FT: tested locally with 4 node cluster. > > > Thanks, > > Oliver Szabo > >