Skip to content

[TEST] Trace logging in testDownsampleTwiceSameInterval#107346

Merged
elasticsearchmachine merged 3 commits intoelastic:mainfrom
kkrik-es:fix/105437
Apr 11, 2024
Merged

[TEST] Trace logging in testDownsampleTwiceSameInterval#107346
elasticsearchmachine merged 3 commits intoelastic:mainfrom
kkrik-es:fix/105437

Conversation

@kkrik-es
Copy link
Copy Markdown
Member

@kkrik-es kkrik-es commented Apr 11, 2024

This is no longer needed in testRollupIndex, no test failures over a week.

Fixes #105437
Related to #105485

kkrik-es added 2 commits April 8, 2024 11:16
This is no longer needed in testRollupIndex, no test failures
over a week.
@kkrik-es kkrik-es 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 Team:StorageEngine labels Apr 11, 2024
@kkrik-es kkrik-es self-assigned this Apr 11, 2024
# Conflicts:
#	x-pack/plugin/ilm/qa/multi-node/src/javaRestTest/java/org/elasticsearch/xpack/ilm/actions/DownsampleActionIT.java
@kkrik-es kkrik-es changed the title Trace logging in testDownsampleTwiceSameInterval [TEST] Trace logging in testDownsampleTwiceSameInterval Apr 11, 2024
@kkrik-es kkrik-es added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Apr 11, 2024
@kkrik-es kkrik-es marked this pull request as ready for review April 11, 2024 08:41
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

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

@elasticsearchmachine elasticsearchmachine merged commit 4707dee into elastic:main Apr 11, 2024
@kkrik-es kkrik-es deleted the fix/105437 branch April 11, 2024 09:18
craigtaverner pushed a commit to craigtaverner/elasticsearch that referenced this pull request Apr 11, 2024
This is no longer needed in testRollupIndex, no test failures over a
week.

Fixes elastic#105437 Related to elastic#105485
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :StorageEngine/Downsampling Downsampling (replacement for rollups) - Turn fine-grained time-based data into coarser-grained data Team:StorageEngine >test Issues or PRs that are addressing/adding tests v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] DownsampleActionIT testRollupIndex failing

3 participants