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

Add clarification about indexers/matchers in add_kubernetes_metadata #18714

Merged
merged 3 commits into from
May 25, 2020

Conversation

ChrsMark
Copy link
Member

What does this PR do?

This PR improves the documentation of add_kubernetes_metadata processor regarding the importance of indexers/matches in some Beats like Heartbeat.

Why is it important?

Only Filebeat and Metricbeat have default indexers and matchers. A beat like Heartbeat is not able to start without defining matchers:

2020-05-21T13:10:19.192Z        INFO    add_kubernetes_metadata/kubernetes.go:71        add_kubernetes_metadata: kubernetes env detected, with version: v1.15.9-gke.24
2020-05-21T13:10:19.192Z        DEBUG   [kubernetes]    add_kubernetes_metadata/kubernetes.go:145       Could not initialize kubernetes plugin with zero matcher plugins        {"libbeat.processor": "add_kubernetes_metadata"}

We need to make it clear in the docs, to avoid confusion.

cc: @exekias @david-kow

@elasticmachine
Copy link
Collaborator

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

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 25, 2020
Signed-off-by: chrismark <[email protected]>
@elasticmachine
Copy link
Collaborator

elasticmachine commented May 25, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #18714 updated]

  • Start Time: 2020-05-25T08:59:03.256+0000

  • Duration: 20 min 22 sec

Signed-off-by: chrismark <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs review Team:Platforms Label for the Integrations - Platforms team v7.8.0 v7.8.1 v7.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants