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

Ensure doc_values parameter is passed into Beats field defintions #1488

Merged
merged 9 commits into from Jun 30, 2021

Conversation

ebeahan
Copy link
Member

@ebeahan ebeahan commented Jun 29, 2021

doc_values should be set in the Beat field definition artifacts if the parameter is listed on a field's definition.

This change adds doc_values into the Beats generator allow list (allowed_keys).

Also adds basic unit testing for portions of the Beats generator module.

Closes #1482

@ebeahan ebeahan self-assigned this Jun 29, 2021
@ebeahan ebeahan requested review from a team and andrewkroh June 29, 2021 20:56
Copy link
Contributor

@djptek djptek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ebeahan
Copy link
Member Author

ebeahan commented Jun 30, 2021

Planning to backport to 1.11 in addition to 1.x

@ebeahan ebeahan merged commit 0f5fdad into elastic:master Jun 30, 2021
ebeahan added a commit to ebeahan/ecs that referenced this pull request Jun 30, 2021
…lastic#1488)

* add doc_values to parameter allow list

* update beats artifacts

* add beats generator unit testing

* linting

* additional artifacts

* changelog
ebeahan added a commit to ebeahan/ecs that referenced this pull request Jun 30, 2021
…lastic#1488)

* add doc_values to parameter allow list

* update beats artifacts

* add beats generator unit testing

* linting

* additional artifacts

* changelog
ebeahan added a commit that referenced this pull request Jun 30, 2021
…1488) (#1498)

* add doc_values to parameter allow list

* update beats artifacts

* add beats generator unit testing

* linting

* additional artifacts

* changelog
ebeahan added a commit that referenced this pull request Jun 30, 2021
…1488) (#1497)

* add doc_values to parameter allow list

* update beats artifacts

* add beats generator unit testing

* linting

* additional artifacts

* changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc_values parameter not set in Beats artifact
2 participants