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

[CM] Make the monitoring reloadable and configurable by CM. #11576

Closed
ph opened this issue Apr 1, 2019 · 6 comments
Closed

[CM] Make the monitoring reloadable and configurable by CM. #11576

ph opened this issue Apr 1, 2019 · 6 comments
Assignees

Comments

@ph
Copy link
Contributor

ph commented Apr 1, 2019

The beats can be configured to send the collected monitoring information to Elasticsearch, this option is globally set in the YAML config with the following:

xpack.monitoring:
  enabled: true
  elasticsearch:
    hosts: ["https://example.com:9200", "https://example2.com:9200"]
    username: beats_system
    password: somepassword

We have to do the following changes:

  • Allow a beat to enable monitoring from CM.
  • Allow to reconfigures monitoring in a running process. (Previously monitoring is set once at boot time.)
  • Add a new type of config block to allow to reconfigure the monitoring from the manager.

Notes: The monitoring uses an internal pipeline so we may be able to reuse existing logic.

@ycombinator
Copy link
Contributor

ycombinator commented Apr 4, 2019

Note that now (with #9260 merged) Beats can ship their monitoring data directly to the monitoring Elasticsearch cluster. To do this, users must set the monitoring.* options instead of the xpack.monitoring.* options.

@michalpristas
Copy link
Contributor

the question is, do we want to configure both options from CM?

@ph
Copy link
Contributor Author

ph commented Apr 5, 2019

@ycombinator Thanks for the ping, looking at this, I believe the objective is to force the user to go with monitoring.* and not use xpack.monitoring.*. If this is the case I would just support the new way of defining it.

@ycombinator
Copy link
Contributor

@ph you are correct, we will deprecate xpack.monitoring.* in favor of monitoring.* in the near future. I'm working on docs and config file updates now.

@ycombinator
Copy link
Contributor

I'm working on docs and config file updates now.

Doc updates PR: #11678
Config file updates PR: #11677

@michalpristas
Copy link
Contributor

closiong this one as well

cc @ph

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants