Skip to content

[7.x] Remove Node Shutdown API feature flag (#76588)#76680

Merged
elasticsearchmachine merged 2 commits intoelastic:7.xfrom
AthenaEryma:backport/7.x/pr-76588
Aug 18, 2021
Merged

[7.x] Remove Node Shutdown API feature flag (#76588)#76680
elasticsearchmachine merged 2 commits intoelastic:7.xfrom
AthenaEryma:backport/7.x/pr-76588

Conversation

@AthenaEryma
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* Remove Node Shutdown API feature flag

This PR removes the Node Shutdown API feature flag.

The Node Shutdown API will now always be available.

* Check if xpack is enabled in cleanup

When I removed the feature flag, I assumed that we would always have the
Node Shutdown APIs, but that turns out not to be the case if xpack isn't
enabled. This case was caught by the logic to handle the case where the
feature flag wasn't enabled by accident.

This commit adds the check we always should have had.

* Also check version before tyring cleanup
@AthenaEryma AthenaEryma added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v7.16.0 labels Aug 18, 2021
@elasticsearchmachine elasticsearchmachine merged commit d091a03 into elastic:7.x Aug 18, 2021
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 v7.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants