Skip to content

Conversation

@danhermann
Copy link
Contributor

Addresses (finally!) the missing tests mentioned in this PR reivew: #66534 (review)

@danhermann danhermann added >test Issues or PRs that are addressing/adding tests :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP v8.0.0 v7.13.0 labels Feb 23, 2021
@danhermann danhermann requested a review from martijnvg February 23, 2021 13:54
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Feb 23, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@danhermann
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/2

@danhermann
Copy link
Contributor Author

@elasticmachine update branch

@danhermann danhermann merged commit e52d60f into elastic:master Feb 24, 2021
@danhermann danhermann deleted the missing_ingest_processor_yaml_tests branch February 24, 2021 17:31
danhermann added a commit to danhermann/elasticsearch that referenced this pull request Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v7.13.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants