Skip to content

Conversation

@ChrsMark
Copy link
Member

This is a follow-up of #16609 to improve the docs of the metricset by:

  1. providing a basic configuration without TLS
  2. providing configuration samples using TLS support (the manual testing guide was updated with this case too on the initial PR)

@ChrsMark ChrsMark added enhancement docs review needs_backport PR is waiting to be backported to other branches. [zube]: In Review v7.7.0 Team:Platforms Label for the Integrations - Platforms team labels Mar 13, 2020
@ChrsMark ChrsMark requested a review from exekias March 13, 2020 11:05
@ChrsMark ChrsMark self-assigned this Mar 13, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/integrations-platforms (Team:Platforms)

tls_config:
cert_file: "/etc/prometheus/my_key.pem"
key_file: "/etc/prometheus/my_key.key"
insecure_skip_verify: true
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it would make sense to comment this one out, and leave it as an example for the case when you cannot validate the host with a CA, WDYT?

Copy link
Member Author

Choose a reason for hiding this comment

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

nice catch, thanks!

@ChrsMark ChrsMark requested review from a team and exekias March 16, 2020 08:08
@ChrsMark ChrsMark merged commit 71b4116 into elastic:master Mar 16, 2020
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Mar 16, 2020
@ChrsMark ChrsMark removed the needs_backport PR is waiting to be backported to other branches. label Mar 16, 2020
ChrsMark added a commit that referenced this pull request Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs enhancement review Team:Platforms Label for the Integrations - Platforms team v7.7.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants