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

[Stack Monitoring] Add missing metricbeat configuration to integrations #4666

Closed
4 tasks done
klacabane opened this issue Nov 16, 2022 · 8 comments
Closed
4 tasks done
Assignees
Labels
bug Something isn't working, use only for issues Integration:beat Beat Integration:elasticsearch Elasticsearch Integration:kibana Kibana Integration:logstash Logstash Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services v8.7.0

Comments

@klacabane
Copy link
Contributor

klacabane commented Nov 16, 2022

Summary

The metricbeat modules (elasticsearch, kibana,..) allows one to specify ssl configuration (certificate_authorities, verification_mode..) and other parameters like period or scope.

Since the elastic integration spawns the metricbeat module under the hood, one should be able to set any accepted metricbeat parameters from the integration.

Implementation hints

  • The parameters are defined under the integration's manifest vars block
  • The parameters are then transformed into a metricbeat configuration in the data streams stream.yml.hbs file

See relevant PR that added the scope parameter: #3275

AC

  • All metricbeat parameters can be set from the integration form
@klacabane klacabane added bug Something isn't working, use only for issues Integration:elasticsearch Elasticsearch Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services labels Nov 16, 2022
@klacabane klacabane changed the title [elasticsearch] add ssl.certificate_authorities configuration [elasticsearch] add ssl.* configuration Nov 16, 2022
@miltonhultgren
Copy link
Contributor

We should also add support for API keys.

@mohamedhamed-ahmed mohamedhamed-ahmed self-assigned this Dec 19, 2022
@klacabane klacabane changed the title [elasticsearch] add ssl.* configuration [Stack Monitoring] Add missing metricbeat configuration to integrations Dec 22, 2022
@klacabane
Copy link
Contributor Author

Updated the issue to account for the other Stack Monitoring packages

@elasticmachine
Copy link

Package beat - 0.1.0 containing this change is available at https://epr.elastic.co/search?package=beat

@elasticmachine
Copy link

Package elasticsearch - 1.2.0-preview1 containing this change is available at https://epr.elastic.co/search?package=elasticsearch

@elasticmachine
Copy link

Package kibana - 2.2.0-preview1 containing this change is available at https://epr.elastic.co/search?package=kibana

@elasticmachine
Copy link

Package logstash - 2.2.0-preview1 containing this change is available at https://epr.elastic.co/search?package=logstash

@joshdover
Copy link
Contributor

We should also add support for API keys.

Did we end up doing this? I'm not seeing in my latest pull of main

@mohamedhamed-ahmed
Copy link
Contributor

We should also add support for API keys.

Did we end up doing this? I'm not seeing in my latest pull of main

Hi @joshdover
We have a ticket opened for this, but not picked up yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working, use only for issues Integration:beat Beat Integration:elasticsearch Elasticsearch Integration:kibana Kibana Integration:logstash Logstash Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services v8.7.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants