Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Nov 17, 2025

Prevent creating the security index eagerly which sometimes interfere with the test.

Resolves: #115697

Ensure shards for testing are allocated on a spare node so that
migration status is for the test shards only.

Resolves: elastic#115697
@ywangd ywangd requested a review from DaveCTurner November 17, 2025 08:56
@ywangd ywangd added >test Issues or PRs that are addressing/adding tests :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown v9.3.0 v9.1.7 v9.2.2 labels Nov 17, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Nov 17, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

LGTM (commit message needs fixing for the new approach)

@ywangd ywangd added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged labels Nov 17, 2025
@elasticsearchmachine elasticsearchmachine merged commit 5875915 into elastic:main Nov 17, 2025
34 checks passed
@ywangd ywangd deleted the es-115697-fix branch November 17, 2025 22:43
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.1 Commit could not be cherrypicked due to conflicts
9.2 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 138150

ywangd added a commit to ywangd/elasticsearch that referenced this pull request Nov 18, 2025
…lastic#138150)

Prevent creating the security index eagerly which sometimes interfere
with the test.

Resolves: elastic#115697
(cherry picked from commit 5875915)

# Conflicts:
#	muted-tests.yml
@ywangd
Copy link
Member Author

ywangd commented Nov 18, 2025

💚 All backports created successfully

Status Branch Result
9.2
9.1

Questions ?

Please refer to the Backport tool documentation

ywangd added a commit to ywangd/elasticsearch that referenced this pull request Nov 18, 2025
…lastic#138150)

Prevent creating the security index eagerly which sometimes interfere
with the test.

Resolves: elastic#115697
(cherry picked from commit 5875915)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Nov 18, 2025
…138150) (#138211)

Prevent creating the security index eagerly which sometimes interfere
with the test.

Resolves: #115697
(cherry picked from commit 5875915)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this pull request Nov 18, 2025
…138150) (#138212)

Prevent creating the security index eagerly which sometimes interfere
with the test.

Resolves: #115697
(cherry picked from commit 5875915)

# Conflicts:
#	muted-tests.yml
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 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport pending :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v9.1.7 v9.2.2 v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] NodeShutdownIT testStalledShardMigrationProperlyDetected failing

3 participants