Skip to content

Conversation

@mark-vieira
Copy link
Contributor

On lower core count machines (such as macbooks) we hit hard limits on thread pool sizes. This adjusts this test to include thread pools for which hard limits apply.

Closes #133854

@mark-vieira mark-vieira added >test Issues or PRs that are addressing/adding tests :Core/Infra/Core Core issues without another label auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Sep 3, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Sep 3, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Member

@rjernst rjernst 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 2bc5498 into elastic:main Sep 4, 2025
33 checks passed
@mark-vieira mark-vieira deleted the fix-thread-pool-settings-tests branch September 4, 2025 03:23
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!) :Core/Infra/Core Core issues without another label Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UpdateThreadPoolSettingsTests > testFixedExecutorType FAILED

3 participants