Skip to content

Comments

[8.19] Fix name of started time field in shutdown status (#139910)#140287

Merged
elasticsearchmachine merged 5 commits intoelastic:8.19from
rjernst:backport/8.19/pr-139910
Jan 16, 2026
Merged

[8.19] Fix name of started time field in shutdown status (#139910)#140287
elasticsearchmachine merged 5 commits intoelastic:8.19from
rjernst:backport/8.19/pr-139910

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented Jan 7, 2026

Backports the following commits to 8.19:

This commit adjusts the field name indicated the time a shutdown started
to be consistent with other millis based times, using the `_millis`
suffix with an underscore. Note that this API is private so making
rest changes without bwc is ok.

closes elastic#139543
@rjernst rjernst added :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown >refactoring 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 7, 2026
@mosche
Copy link
Contributor

mosche commented Jan 15, 2026

Docs tests are failing with

java.lang.RuntimeException: Failure at [reference/shutdown/apis/shutdown-get:26]: stashed value not found for key [body.nodes.0.shutdown_startedmillis]	

@mosche
Copy link
Contributor

mosche commented Jan 15, 2026

fixed in 62a85ac

@rjernst rjernst enabled auto-merge (squash) January 16, 2026 22:17
@elasticsearchmachine elasticsearchmachine merged commit ecc561a into elastic:8.19 Jan 16, 2026
28 checks passed
@rjernst rjernst deleted the backport/8.19/pr-139910 branch January 16, 2026 23:16
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 :Core/Infra/Node Lifecycle Node startup, bootstrapping, and shutdown >refactoring Team:Core/Infra Meta label for core/infra team v8.19.11

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants