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

Fix istio mapping #17562

Merged
merged 4 commits into from
Apr 7, 2020
Merged

Fix istio mapping #17562

merged 4 commits into from
Apr 7, 2020

Conversation

ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Apr 7, 2020

What does this PR do?

This PR fixes:

  • istio mixer mapping and more specifically the fields handler.name and variety.
  • istio pilot mapping: xds.push.time should be histogram, no.ip should be long, xds.eds.instances should be long.

Why is it important?

Events were not able to be indexed in ES.

Related issues

Signed-off-by: chrismark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark added bug 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 v7.8.0 labels Apr 7, 2020
@ChrsMark ChrsMark self-assigned this Apr 7, 2020
@elasticmachine
Copy link
Collaborator

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

@ChrsMark ChrsMark changed the title Fix istio mixer mapping Fix istio mapping Apr 7, 2020
ChrsMark added 2 commits April 7, 2020 12:28
Signed-off-by: chrismark <chrismarkou92@gmail.com>
Signed-off-by: chrismark <chrismarkou92@gmail.com>
Signed-off-by: chrismark <chrismarkou92@gmail.com>
Copy link
Contributor

@ycombinator ycombinator left a comment

Choose a reason for hiding this comment

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

LGTM. WFG.

@ChrsMark ChrsMark merged commit df311e5 into elastic:master Apr 7, 2020
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Apr 7, 2020
(cherry picked from commit df311e5)
@ChrsMark ChrsMark removed the needs_backport PR is waiting to be backported to other branches. label Apr 7, 2020
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Apr 7, 2020
(cherry picked from commit df311e5)
ChrsMark added a commit that referenced this pull request Apr 7, 2020
(cherry picked from commit df311e5)
ChrsMark added a commit that referenced this pull request Apr 7, 2020
(cherry picked from commit df311e5)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug review Team:Platforms Label for the Integrations - Platforms team v7.7.0 v7.8.0
Projects
None yet
4 participants