Skip to content

Comments

[9.3] Fix readiness edge case on startup (#140791)#141063

Merged
elasticsearchmachine merged 1 commit intoelastic:9.3from
rjernst:backport/9.3/pr-140791
Jan 23, 2026
Merged

[9.3] Fix readiness edge case on startup (#140791)#141063
elasticsearchmachine merged 1 commit intoelastic:9.3from
rjernst:backport/9.3/pr-140791

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Jan 21, 2026

Backports the following commits to 9.3:

The readiness service watches for cluster state updates to determine
when the node is ready. It must have a master node elected, and file
settings must have been applied. Normally those take a little bit of
time. However, the readiness services is setup to not even allow
starting the tcp listener until after the service start method is
called. This occurs in Node.start(), but _after_ the initial node join.
If the initial join occurs before start, and the state already reflects
the "ready" state, when start is called the service will be marked
active, but nothing will ever start the listener.

This commit adjusts the readiness service to keep track of the last
state applied. It also moves adding the cluster state listener to when
the service is started. Finally, it adjusts the readiness test helpers
to use assertBusy instead of a hand rolled backoff loop.

closes elastic#136955
@rjernst rjernst added :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team labels Jan 21, 2026
@elasticsearchmachine elasticsearchmachine merged commit 18dbb77 into elastic:9.3 Jan 23, 2026
35 checks passed
@rjernst rjernst deleted the backport/9.3/pr-140791 branch January 23, 2026 22:35
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 >bug :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown Team:Core/Infra Meta label for core/infra team v9.3.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants