Skip to content

[CI] [8.19] Fix ThreadPoolMergeExecutorServiceDiskSpaceTests testUnavailableBudgetBlocksNewMergeTasksFromStartingExecution#130110

Merged
elasticsearchmachine merged 1 commit intoelastic:8.19from
albertzaharovits:backport-819-fix-129148-take2
Jun 26, 2025
Merged

[CI] [8.19] Fix ThreadPoolMergeExecutorServiceDiskSpaceTests testUnavailableBudgetBlocksNewMergeTasksFromStartingExecution#130110
elasticsearchmachine merged 1 commit intoelastic:8.19from
albertzaharovits:backport-819-fix-129148-take2

Conversation

@albertzaharovits
Copy link
Copy Markdown
Contributor

The test might produce over-budget tasks that cannot run even if all the other tasks that were blocked (and hold up budget) while running complete.
Rather than prevent submitting such over-budget tasks, this fix simply sets the merge task's queue available budget to Long.MAX_VALUE, in order to ensure that all merge tasks run before the test ends.

Fixes #129148
Backport of #130001

…tBlocksNewMergeTasksFromStartingExecution (elastic#130001)

The test might produce over-budget tasks that cannot run even if all the
other tasks that were blocked (and hold up budget) while running
complete. Rather than prevent submitting such over-budget tasks, this
fix simply sets the merge task's queue available budget to
`Long.MAX_VALUE`, in order to ensure that all merge tasks run before the
test ends.

Fixes elastic#129148
@albertzaharovits albertzaharovits self-assigned this Jun 26, 2025
@albertzaharovits albertzaharovits added >test Issues or PRs that are addressing/adding tests :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. backport v8.19.0 labels Jun 26, 2025
@albertzaharovits albertzaharovits changed the title [8.19] Fix ThreadPoolMergeExecutorServiceDiskSpaceTests testUnavailableBudgetBlocksNewMergeTasksFromStartingExecution [CI] [8.19] Fix ThreadPoolMergeExecutorServiceDiskSpaceTests testUnavailableBudgetBlocksNewMergeTasksFromStartingExecution Jun 26, 2025
@albertzaharovits albertzaharovits added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 26, 2025
@elasticsearchmachine elasticsearchmachine merged commit 4c591f5 into elastic:8.19 Jun 26, 2025
22 checks passed
@albertzaharovits albertzaharovits deleted the backport-819-fix-129148-take2 branch June 26, 2025 13:22
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!) backport :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. >test Issues or PRs that are addressing/adding tests v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants