Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enables plugins to define default logging configuration for their needs. #6805

Closed
wants to merge 1 commit into from
Closed

Enables plugins to define default logging configuration for their needs. #6805

wants to merge 1 commit into from

Conversation

uboness
Copy link
Contributor

@uboness uboness commented Jul 9, 2014

  • adds support for multiple logging configurations under the config dir (will pick up any logging.xxx in the config folder tree)

  • plugins can now define a top level config directory that will be copied under es config dir and will be renamed after the plugin name (same as the support we have the plugin "bin" dirs)

    Closes Enable plugins to define default logging configurations for their needs #6802

- adds support for multiple logging configurations under the config dir (will pick up any logging.xxx in the config folder tree)
- plugins can now define a top level config directory that will be copied under es config dir and will be renamed after the plugin name (same as the support we have the plugin "bin" dirs)

 Closes #6802
@uboness uboness added v2.0.0 and removed v1.2.3 labels Jul 9, 2014
@kimchy kimchy removed the v1.2.3 label Jul 9, 2014
@kimchy
Copy link
Member

kimchy commented Jul 9, 2014

LGTM

@uboness
Copy link
Contributor Author

uboness commented Jul 10, 2014

closed 610900b

@clintongormley
Copy link

Closed by 610900b

/**
*
*/
public class LoggingConfigurationTests {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one should extend ElasticsearchTestCase, otherwise it will use the junit runner instead of the randomized one.

@uboness uboness deleted the enhance/logging branch July 10, 2014 11:59
@jpountz jpountz removed the review label Jul 16, 2014
@clintongormley clintongormley changed the title Enables plugins to define default logging configuration for their needs. Plugins: Enables plugins to define default logging configuration for their needs. Jul 16, 2014
@clintongormley clintongormley added the :Core/Infra/Plugins Plugin API and infrastructure label Jun 7, 2015
@clintongormley clintongormley changed the title Plugins: Enables plugins to define default logging configuration for their needs. Enables plugins to define default logging configuration for their needs. Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable plugins to define default logging configurations for their needs
5 participants