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.

Sign up for GitHub

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

Jump to bottom

[CI] LangMustacheClientYamlTestSuiteIT test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors} failing #112580

Closed
elasticsearchmachine opened this issue Sep 6, 2024 · 6 comments · Fixed by #112683
Closed

[CI] LangMustacheClientYamlTestSuiteIT test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors} failing #112580

elasticsearchmachine opened this issue Sep 6, 2024 · 6 comments · Fixed by #112683
Assignees
@cbuescher
Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

Build Scans:

Reproduction Line:

./gradlew ":modules:lang-mustache:yamlRestTestV7CompatTest" --tests "org.elasticsearch.script.mustache.LangMustacheClientYamlTestSuiteIT.test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors}" -Dtests.seed=DC50FAC49A21145D -Dtests.locale=da-Latn-DK -Dtests.timezone=Hongkong -Druntime.java=22

Applicable branches:
main

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: Failure at [lang_mustache/50_multi_search_template:141]: field [responses.1.error.root_cause.0.reason] was expected to match the provided regex but didn't
Expected: Unexpected.end.of.input
     but: was "[1:22] Unexpected end of file"

Issue Reasons:

  • [main] 2 failures in test test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors} (0.8% fail rate in 257 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Core/Infra/Core Core issues without another label >test-failure Triaged test failures from CI labels Sep 6, 2024
elasticsearchmachine added a commit that referenced this issue Sep 6, 2024
@elasticsearchmachine
Mute org.elasticsearch.script.mustache.LangMustacheClientYamlTestSuit…
d2433c4 
…eIT test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors} #112580
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 2 failures in test test {yaml=lang_mustache/50_multi_search_template/Multi-search template with errors} (0.8% fail rate in 257 executions)

Build Scans:

@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team needs:risk Requires assignment of a risk label (low, medium, blocker) labels Sep 6, 2024
@ldematte
Copy link
Contributor

ldematte commented Sep 9, 2024 •
edited
Loading

This started failing after #111548 got merged, forwarding to the search team (Probably a string that needs to be handled different for BwC/ rest compatibility tests?)

@ldematte ldematte added :Search/Search Search-related issues that do not fall into other categories and removed :Core/Infra/Core Core issues without another label labels Sep 9, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Search Meta label for search team label Sep 9, 2024
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-search (Team:Search)

@elasticsearchmachine elasticsearchmachine removed the Team:Core/Infra Meta label for core/infra team label Sep 9, 2024
@benwtrent
Copy link
Member

@cbuescher ^? This one looks weird for sure.

@cbuescher cbuescher assigned cbuescher and unassigned ldematte Sep 9, 2024
cbuescher added a commit to cbuescher/elasticsearch that referenced this issue Sep 9, 2024
@cbuescher
Fix failing LangMustacheClientYamlTestSuiteIT yamlRestTestV7CompatTest
3fa3623 
The error message changed slightly to include the failure location. While this
was added to the main test yml file, the yamlRestTestV7CompatTest resources seem
to get copied from v7 files and we then modify them if we expect any changes.

Closes elastic#112580
@cbuescher cbuescher mentioned this issue Sep 9, 2024
Merged
@cbuescher
Copy link
Member

Thanks, I opened a PR that I believe fixes this.

davidkyle pushed a commit that referenced this issue Sep 12, 2024
@cbuescher @davidkyle
Fix failing LangMustacheClientYamlTestSuiteIT yamlRestTestV7CompatTest (
7f66489 
#112683)

The error message changed slightly to include the failure location. While this
was added to the main test yml file, the yamlRestTestV7CompatTest resources seem
to get copied from v7 files and we then modify them if we expect any changes.

Closes #112580
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cbuescher cbuescher

Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Search/Search Search-related issues that do not fall into other categories Team:Search Meta label for search team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix failing LangMustacheClientYamlTestSuiteIT yamlRestTestV7CompatTest cbuescher/elasticsearch
4 participants
@ldematte @benwtrent @cbuescher @elasticsearchmachine