Skip to content

Skips synonym test in mixed cluster BwC tests#119044

Merged
carlosdelest merged 2 commits intoelastic:mainfrom
carlosdelest:fix/synonyms-test-excluded-mixed-cluster
Jan 7, 2025
Merged

Skips synonym test in mixed cluster BwC tests#119044
carlosdelest merged 2 commits intoelastic:mainfrom
carlosdelest:fix/synonyms-test-excluded-mixed-cluster

Conversation

@carlosdelest
Copy link
Member

@carlosdelest carlosdelest commented Dec 19, 2024

Closes #116777

Follow up to #118691

The problematic test should be skipped in the mixed cluster test, not in the YAML rest compatibility. I double checked that all tests failures have been in mixed cluster tests, and that the test is skipped there.

@carlosdelest carlosdelest added >test Issues or PRs that are addressing/adding tests :Search Relevance/Analysis How text is split into tokens auto-backport Automatically create backport pull requests when merged Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v8.16.0 v9.0.0 v8.17.0 v8.18.0 labels Dec 19, 2024
@elasticsearchmachine
Copy link
Collaborator

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

@carlosdelest carlosdelest requested a review from a team January 3, 2025 13:13
Copy link
Contributor

@mayya-sharipova mayya-sharipova left a comment

Choose a reason for hiding this comment

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

Thanks Carlos

@carlosdelest carlosdelest merged commit cc0e53e into elastic:main Jan 7, 2025
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.16 Commit could not be cherrypicked due to conflicts
8.17 Commit could not be cherrypicked due to conflicts
8.15 Commit could not be cherrypicked due to conflicts
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 119044

carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Jan 7, 2025
…mpat (elastic#119044)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Jan 7, 2025
…mpat (elastic#119044)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Jan 7, 2025
…mpat (elastic#119044)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
@carlosdelest
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.x
8.17
8.16
8.15

Questions ?

Please refer to the Backport tool documentation

carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Jan 7, 2025
…mpat (elastic#119044)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
carlosdelest added a commit that referenced this pull request Jan 7, 2025
carlosdelest added a commit that referenced this pull request Jan 7, 2025
elasticsearchmachine pushed a commit that referenced this pull request Jan 7, 2025
…mpat (#119044) (#119664)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
sarog pushed a commit to portsbuild/elasticsearch that referenced this pull request Jan 22, 2025
…mpat (elastic#119044) (elastic#119664)

(cherry picked from commit cc0e53e)

# Conflicts:
#	muted-tests.yml
#	qa/mixed-cluster/build.gradle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Search Relevance/Analysis How text is split into tokens Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch >test Issues or PRs that are addressing/adding tests v8.15.6 v8.16.0 v8.17.0 v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] MixedClusterClientYamlTestSuiteIT test {p0=synonyms/90_synonyms_reloading_for_synset/Reload analyzers for specific synonym set} failing

4 participants