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

Tweaked DownsampleClusterDisruptionIT and #98523

Merged

Conversation

martijnvg
Copy link
Member

@martijnvg martijnvg commented Aug 16, 2023

unmuted DownsampleClusterDisruptionIT.testDownsampleIndexWithRollingRestart() and
DownsampleClusterDisruptionIT.testDownsampleIndexWithFullClusterRestart() tests

Invoking the downsample api failed because the coordinating node was restarted and
the sync request waited forever. Added a timeout, so that in case of this situation invoking
the action can fail and then it will be retried.

Relates to #98485

unmuted DownsampleClusterDisruptionIT.testDownsampleIndexWithRollingRestart() and
 DownsampleClusterDisruptionIT.testDownsampleIndexWithFullClusterRestart() tests

Relates to elastic#98485
@martijnvg martijnvg added >test Issues or PRs that are addressing/adding tests :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data v8.10.0 v8.11.0 labels Aug 16, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Aug 16, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@quux00 quux00 removed the v8.10.0 label Aug 16, 2023
@martijnvg martijnvg added auto-backport-and-merge v8.10.1 auto-backport Automatically create backport pull requests when merged and removed auto-backport-and-merge labels Sep 8, 2023
@martijnvg martijnvg merged commit efc2fb2 into elastic:main Sep 11, 2023
martijnvg added a commit to martijnvg/elasticsearch that referenced this pull request Sep 11, 2023
unmuted DownsampleClusterDisruptionIT.testDownsampleIndexWithRollingRestart() and
 DownsampleClusterDisruptionIT.testDownsampleIndexWithFullClusterRestart() tests

Relates to elastic#98485
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.10

martijnvg added a commit that referenced this pull request Sep 12, 2023
unmuted DownsampleClusterDisruptionIT.testDownsampleIndexWithRollingRestart() and
 DownsampleClusterDisruptionIT.testDownsampleIndexWithFullClusterRestart() tests

Backport of #98523 to 8.10 branch.

Relates to #98485
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 :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.10.1 v8.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants