Skip to content

Fix failing test DownsampleActionIT.testRollupIndex() #95214

Merged
csoulios merged 1 commit intoelastic:mainfrom
csoulios:fix-test
Apr 18, 2023
Merged

Fix failing test DownsampleActionIT.testRollupIndex() #95214
csoulios merged 1 commit intoelastic:mainfrom
csoulios:fix-test

Conversation

@csoulios
Copy link
Copy Markdown
Contributor

@csoulios csoulios commented Apr 13, 2023 •
edited
Loading

Test DownsampleActionIT.testRollupIndex() has been failing after PR #94835 got merged.

Most probably, because The PR removed the following line from the testRollupIndex() method

updateClusterSettings(client(), Settings.builder().put("indices.lifecycle.poll_interval", "5s").build());

To fix the test, we set the indices.lifecycle.poll_interval setting before all tests run.

Closes #95156

@csoulios csoulios added >test-failure Triaged test failures from CI :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data auto-backport-and-merge v8.8.0 v8.7.2 labels Apr 13, 2023
@elasticsearchmachine elasticsearchmachine added Team:Data Management (obsolete) DO NOT USE. This team no longer exists. Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Apr 13, 2023
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

elasticsearchmachine commented Apr 13, 2023

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

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

elasticsearchmachine commented Apr 13, 2023

Pinging @elastic/es-data-management (Team:Data Management)

martijnvg
martijnvg approved these changes Apr 17, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

@csoulios csoulios merged commit c986317 into elastic:main Apr 18, 2023
@csoulios csoulios deleted the fix-test branch April 18, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@martijnvg martijnvg martijnvg approved these changes

@salvatore-campagna salvatore-campagna salvatore-campagna approved these changes

Assignees

No one assigned

Labels

:Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Data Management (obsolete) DO NOT USE. This team no longer exists. >test-failure Triaged test failures from CI v8.8.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CI] DownsampleActionIT testRollupIndex failing

4 participants

@csoulios @elasticsearchmachine @martijnvg @salvatore-campagna