Skip to content

Conversation

@gmarouli
Copy link
Contributor

@gmarouli gmarouli commented Apr 5, 2024

The health API was available for experimentation under the _internal/_health before it became GA at 8.7.0. For this reason we introduce another node feature to capture this change.

Fixes #106933

@gmarouli gmarouli added >test Issues or PRs that are addressing/adding tests :Data Management/Health v8.14.0 labels Apr 5, 2024
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Apr 5, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@gmarouli gmarouli requested a review from nielsbauman April 5, 2024 12:40
Copy link
Contributor

@nielsbauman nielsbauman left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for taking the time to figure this one out :)

@gmarouli
Copy link
Contributor Author

gmarouli commented Apr 8, 2024

@elasticmachine update branch

@gmarouli gmarouli added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Apr 8, 2024
@elasticsearchmachine elasticsearchmachine merged commit f66ca2d into elastic:main Apr 8, 2024
@gmarouli gmarouli deleted the fix-health-api-test branch April 8, 2024 08:26
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!) :Data Management/Health Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] HealthNodeUpgradeIT testHealthNode {upgradedNodes=0} failing

4 participants