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

Generated templates don't include fields when parent defined in 2 locations #5075

Closed
exekias opened this issue Sep 1, 2017 · 0 comments
Closed

Generated templates don't include fields when parent defined in 2 locations #5075

exekias opened this issue Sep 1, 2017 · 0 comments
Assignees
@exekias
Labels
bug libbeat v6.0.0-beta1

Comments

@exekias
Copy link
Contributor

exekias commented Sep 1, 2017

Some processors share fields with metricbeat modules (docker & kubernetes), so they define some fields like this:

https://github.com/elastic/beats/blob/master/libbeat/processors/add_docker_metadata/_meta/fields.yml

While the rest of fields for the module are in other fields.yml file:

https://github.com/elastic/beats/blob/master/metricbeat/module/docker/container/_meta/fields.yml

As they share the same root key, the first one in the resulting fields.yml is overridden by the second one. In practice, this means that fields from docker & kubernetes processors are missing from the final template
@exekias exekias self-assigned this Sep 1, 2017
@exekias exekias mentioned this issue Sep 1, 2017
Closed
exekias added a commit to exekias/beats that referenced this issue Sep 1, 2017
@exekias
Combine fields.yml properties when they are defined in different so…
d4a6f71 
…urces

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes elastic#5075
@exekias exekias mentioned this issue Sep 1, 2017
Merged
exekias added a commit to exekias/beats that referenced this issue Sep 1, 2017
@exekias
Combine fields.yml properties when they are defined in different so…
441a91c 
…urces

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes elastic#5075
ruflin pushed a commit that referenced this issue Sep 1, 2017
@exekias @ruflin
Combine fields.yml properties when they are defined in different so…
a96b02a 
…urces (#5077)

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes #5075
exekias added a commit to exekias/beats that referenced this issue Sep 1, 2017
@exekias
Combine fields.yml properties when they are defined in different so…
d27e301 
…urces (elastic#5077)

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes elastic#5075
(cherry picked from commit a96b02a)
@exekias exekias mentioned this issue Sep 1, 2017
Merged
andrewkroh pushed a commit that referenced this issue Sep 1, 2017
@exekias @andrewkroh
Combine fields.yml properties when they are defined in different so…
81dafa4 
…urces (#5077) (#5079)

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes #5075
(cherry picked from commit a96b02a)
leweafan pushed a commit to leweafan/beats that referenced this issue Apr 28, 2023
@exekias @andrewkroh
Combine fields.yml properties when they are defined in different so…
ab16749 
…urces (elastic#5077) (elastic#5079)

We define fields with the same root key across several files, this
change ensures all of them are combined in the final index template.

Fixes elastic#5075
(cherry picked from commit caaa4ca)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@exekias exekias

Labels
bug libbeat v6.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@exekias