Skip to content

[fix] move from keyword to wildcards for ECS fields#4577

Merged
simitt merged 4 commits intoelastic:masterfrom
simitt:fix-field-types
Dec 28, 2020
Merged

[fix] move from keyword to wildcards for ECS fields#4577
simitt merged 4 commits intoelastic:masterfrom
simitt:fix-field-types

Conversation

@simitt
Copy link
Copy Markdown
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
@simitt simitt requested a review from a team December 22, 2020 10:00
@ghost
Copy link
Copy Markdown

ghost 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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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

4 participants