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.

Sign up for GitHub

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

Jump to bottom

Change docker container field names to match ECS #10826

Closed
wants to merge 4 commits into from
Closed

Change docker container field names to match ECS #10826

wants to merge 4 commits into from

Conversation

kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented Feb 19, 2019 •
edited
Loading

Migrate the docker container fields to ECS container fields:

  • docker.container.id -> container.id
  • docker.container.image -> container.image.name
  • docker.container.name -> container.name
  • docker.container.labels -> container.labels

This PR also migrates docker container metricset to use ReporterV2 interface. Please see #10774 for more info.

@kaiyan-sheng kaiyan-sheng requested a review from a team as a code owner February 19, 2019 15:38
@sayden sayden mentioned this pull request Feb 19, 2019
Closed
62 tasks
@kaiyan-sheng kaiyan-sheng self-assigned this Feb 19, 2019
@kaiyan-sheng kaiyan-sheng added Metricbeat Metricbeat Team:Integrations Label for the Integrations team review labels Feb 19, 2019
@kaiyan-sheng
Copy link
Contributor Author

This is replaced by #10927 which is better :-)

@kaiyan-sheng kaiyan-sheng deleted the docker_ecs branch February 25, 2019 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@kaiyan-sheng kaiyan-sheng

Labels
Metricbeat Metricbeat review Team:Integrations Label for the Integrations team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kaiyan-sheng