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

Cherry-pick #20571 to 7.x: Change cluster_ip field from ip to keyword #20596

Merged
merged 1 commit into from
Aug 14, 2020

Conversation

ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Aug 14, 2020

Cherry-pick of PR #20571 to 7.x branch. Original message:

What does this PR do?

This PR changes the type of cluster_ip field of state_service metricset of kubernetes Metricbeat module from ip to keyword.

Why is it important?

cluster_ip can come with a value like None which is meaningful in Kubernetes networking. We need to change the type of this filed so as to be possible to index such values.

Related issues

@ChrsMark ChrsMark added [zube]: In Review backport Team:Platforms Label for the Integrations - Platforms team labels Aug 14, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Aug 14, 2020
@elasticmachine
Copy link
Collaborator

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

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Aug 14, 2020
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20596 opened]

  • Start Time: 2020-08-14T07:18:24.906+0000

  • Duration: 77 min 15 sec

Test stats 🧪

Test Results
Failed 0
Passed 3081
Skipped 686
Total 3767

@ChrsMark ChrsMark merged commit 47aa809 into elastic:7.x Aug 14, 2020
@zube zube bot removed the [zube]: Done label Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Team:Platforms Label for the Integrations - Platforms team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants