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

Fix and unmute CrossClusterEsqlRCS1UnavailableRemotesIT tests #120388

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

pawankartik-elastic
Copy link
Contributor

Fixes #118350.

Solution inline with #119977.

@pawankartik-elastic pawankartik-elastic added >non-issue Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch :Search Foundations/CCS v9.0.0 labels Jan 17, 2025
@pawankartik-elastic pawankartik-elastic changed the title Unmute CrossClusterEsqlRCS1UnavailableRemotesIT.testEsqlRcs1UnavailableRemoteScenarios Fix and unmute CrossClusterEsqlRCS1UnavailableRemotesIT.testEsqlRcs1UnavailableRemoteScenarios Jan 17, 2025
@pawankartik-elastic pawankartik-elastic changed the title Fix and unmute CrossClusterEsqlRCS1UnavailableRemotesIT.testEsqlRcs1UnavailableRemoteScenarios Fix and unmute CrossClusterEsqlRCS1UnavailableRemotesIT tests Jan 17, 2025
@pawankartik-elastic pawankartik-elastic marked this pull request as ready for review January 17, 2025 18:21
@elasticsearchmachine
Copy link
Collaborator

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

assertThat(ex.getMessage(), containsString("connect_transport_exception"));
assertThat(
ex.getMessage(),
anyOf(containsString("connect_transport_exception"), containsString("node_disconnected_exception"))
Copy link
Contributor

Choose a reason for hiding this comment

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

This should also have the third variant added we saw in the enrich tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Pushed a commit to fix this.

@quux00
Copy link
Contributor

quux00 commented Jan 17, 2025

Let's add auto-backport to the labels and v8.18.0 after you get the original test backported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>non-issue :Search Foundations/CCS Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] CrossClusterEsqlRCS1UnavailableRemotesIT testEsqlRcs1UnavailableRemoteScenarios failing
3 participants