You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Oliver Szabo <os...@hortonworks.com> on 2016/04/27 20:37:35 UTC

Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46746/
-----------------------------------------------------------

Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.


Bugs: AMBARI-16141
    https://issues.apache.org/jira/browse/AMBARI-16141


Repository: ambari


Description
-------

- add configuration dependencies
- remove logfeeder-site configuration
- remove unused properties and logsearch kafka support


Diffs
-----

  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml 190ee53 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-site.xml 56552f2 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/metainfo.xml 40c289e 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py 7ed5c42 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py 50018cb 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/templates/output.config.json.j2 8216587 
  ambari-web/app/data/HDP2/site_properties.js 8851a78 

Diff: https://reviews.apache.org/r/46746/diff/


Testing
-------

ambari-web tests:

27448 tests complete (32 seconds)
154 tests pending

ambar-server python tests:

Total run:998
Total errors:0
Total failures:0


Thanks,

Oliver Szabo


Re: Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

Posted by Robert Nettleton <rn...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46746/#review130910
-----------------------------------------------------------


Ship it!




Ship It!

- Robert Nettleton


On April 27, 2016, 6:37 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46746/
> -----------------------------------------------------------
> 
> (Updated April 27, 2016, 6:37 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-16141
>     https://issues.apache.org/jira/browse/AMBARI-16141
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> - add configuration dependencies
> - remove logfeeder-site configuration
> - remove unused properties and logsearch kafka support
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml 190ee53 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-site.xml 56552f2 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/metainfo.xml 40c289e 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py 7ed5c42 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py 50018cb 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/templates/output.config.json.j2 8216587 
>   ambari-web/app/data/HDP2/site_properties.js 8851a78 
> 
> Diff: https://reviews.apache.org/r/46746/diff/
> 
> 
> Testing
> -------
> 
> ambari-web tests:
> 
> 27448 tests complete (32 seconds)
> 154 tests pending
> 
> ambar-server python tests:
> 
> Total run:998
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>


Re: Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

Posted by Miklos Gergely <mi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46746/#review130901
-----------------------------------------------------------


Ship it!




Ship It!

- Miklos Gergely


On April 27, 2016, 6:37 p.m., Oliver Szabo wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46746/
> -----------------------------------------------------------
> 
> (Updated April 27, 2016, 6:37 p.m.)
> 
> 
> Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.
> 
> 
> Bugs: AMBARI-16141
>     https://issues.apache.org/jira/browse/AMBARI-16141
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> - add configuration dependencies
> - remove logfeeder-site configuration
> - remove unused properties and logsearch kafka support
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml 190ee53 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-site.xml 56552f2 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/metainfo.xml 40c289e 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py 7ed5c42 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py 50018cb 
>   ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/templates/output.config.json.j2 8216587 
>   ambari-web/app/data/HDP2/site_properties.js 8851a78 
> 
> Diff: https://reviews.apache.org/r/46746/diff/
> 
> 
> Testing
> -------
> 
> ambari-web tests:
> 
> 27448 tests complete (32 seconds)
> 154 tests pending
> 
> ambar-server python tests:
> 
> Total run:998
> Total errors:0
> Total failures:0
> 
> 
> Thanks,
> 
> Oliver Szabo
> 
>


Re: Review Request 46746: AMBARI-16141 Add configuration dependencies for Logsearch components

Posted by Oliver Szabo <os...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46746/
-----------------------------------------------------------

(Updated April 28, 2016, 4:48 p.m.)


Review request for Ambari, Miklos Gergely, Robert Nettleton, and Sumit Mohanty.


Changes
-------

small fixes


Bugs: AMBARI-16141
    https://issues.apache.org/jira/browse/AMBARI-16141


Repository: ambari


Description
-------

- add configuration dependencies
- remove logfeeder-site configuration
- remove unused properties and logsearch kafka support


Diffs (updated)
-----

  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-env.xml 190ee53 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logfeeder-site.xml 56552f2 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-admin-json.xml ae2c90d 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/configuration/logsearch-solr-env.xml b3ea446 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/metainfo.xml 40c289e 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/logsearch_common.py 56f7792 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/params.py 7ed5c42 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/scripts/setup_logsearch.py 50018cb 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/package/templates/output.config.json.j2 8216587 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/logsearch-admin.json.j2 PRE-CREATION 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/solr-log4j.properties.j2 8666a58 
  ambari-server/src/main/resources/common-services/LOGSEARCH/0.5.0/properties/user_pass.json.j2 abb0ddb 
  ambari-web/app/data/HDP2/site_properties.js 8851a78 

Diff: https://reviews.apache.org/r/46746/diff/


Testing
-------

ambari-web tests:

27448 tests complete (32 seconds)
154 tests pending

ambar-server python tests:

Total run:998
Total errors:0
Total failures:0


Thanks,

Oliver Szabo