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] move from keyword to wildcards for ECS fields #4577

Merged
merged 4 commits into from
Dec 28, 2020

Conversation

simitt
Copy link
Contributor

@simitt simitt commented Dec 22, 2020

Motivation/summary

Types have not been changed from keyword to wildcard when pulling in the latest beats update. Index Pattern generation is broken since then, as the same field name is defined with different types.

This PR aligns the type with ECS types introduced in beats for overwritten fields.

Checklist

How to test these changes

Check that template has wildcard as type for changed fields

Related issues

closes #4448

Align type with ECS types for overwritten fields.

closes elastic#4448
@apmmachine
Copy link
Contributor

apmmachine commented Dec 22, 2020

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #4577 updated

  • Start Time: 2020-12-24T14:42:05.198+0000

  • Duration: 53 min 44 sec

Test stats 🧪

Test Results
Failed 0
Passed 4613
Skipped 124
Total 4737

Steps errors 2

Expand to view the steps failures

Compress
  • Took 0 min 0 sec . View more details on here
  • Description: tar --exclude=coverage-files.tgz -czf coverage-files.tgz coverage
Compress
  • Took 0 min 0 sec . View more details on here
  • Description: tar --exclude=system-tests-linux-files.tgz -czf system-tests-linux-files.tgz system-tests

@codecov-io
Copy link

codecov-io commented Dec 22, 2020

Codecov Report

Merging #4577 (c3345fd) into master (3858a27) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4577   +/-   ##
=======================================
  Coverage   75.96%   75.96%           
=======================================
  Files         161      161           
  Lines        9789     9789           
=======================================
  Hits         7436     7436           
  Misses       2353     2353           

@simitt
Copy link
Contributor Author

simitt commented Dec 22, 2020

Tests are failing because of a corrupted Kibana container.

@simitt simitt merged commit d8792d8 into elastic:master Dec 28, 2020
simitt added a commit to simitt/apm-server that referenced this pull request Dec 28, 2020
Align type with ECS types for overwritten fields.

closes elastic#4448
simitt added a commit to simitt/apm-server that referenced this pull request Dec 28, 2020
Align type with ECS types for overwritten fields.

closes elastic#4448
simitt added a commit that referenced this pull request Dec 28, 2020
)

Align type with ECS types for overwritten fields.

closes #4448
simitt added a commit that referenced this pull request Dec 28, 2020
…4589)

Align type with ECS types for overwritten fields.

closes #4448
@jalvz
Copy link
Contributor

jalvz commented Jan 12, 2021

wildcard types exist in the templates, works as expected

@simitt simitt deleted the fix-field-types branch August 20, 2021 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to ECS 1.7 - support wildcards
5 participants