Skip to content

Fixing ingest simulate yaml rest test when there is a global legacy template (#115559) #115599

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

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

masseyke
Copy link
Member

The ingest simulate yaml rest test Test mapping addition works with indices without templates tests what happens when an index has a mapping but matches no template at all. However, randomly and rarely a global match-all legacy template is applied to the cluster. When this happens, the assumptions for the test fail since the index matches a template. This PR removes that global legacy template so that the test works as intended.
Closes #115412
Closes #115472

…emplate (elastic#115559)

The ingest simulate yaml rest test `Test mapping addition works with
indices without templates` tests what happens when an index has a
mapping but matches no template at all. However, randomly and rarely a
global match-all legacy template is applied to the cluster. When this
happens, the assumptions for the test fail since the index matches a
template. This PR removes that global legacy template so that the test
works as intended. Closes elastic#115412 Closes elastic#115472
@masseyke masseyke added >test Issues or PRs that are addressing/adding tests :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.17.0 labels Oct 24, 2024
@elasticsearchmachine elasticsearchmachine merged commit d3705e6 into elastic:8.x Oct 24, 2024
15 checks passed
@masseyke masseyke deleted the backport/8.x/pr-115559 branch October 24, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP >test Issues or PRs that are addressing/adding tests v8.17.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants