Skip to content

Missing field and wrong value in ClusterHealthResponse #4193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 7, 2025

Conversation

l-trotta
Copy link
Contributor

@l-trotta l-trotta commented Apr 3, 2025

active_shards_percent_as_number is not a string nor float, it's a double - server code.
also there's the optional string version of this field, active_shards_percent - server code

Copy link
Contributor

github-actions bot commented Apr 3, 2025

Following you can find the validation results for the API you have changed.

API Status Request Response
cluster.health 🟢 162/162 162/162

You can validate this API yourself by using the make validate target.

Copy link
Contributor

github-actions bot commented Apr 3, 2025

Following you can find the validation results for the API you have changed.

API Status Request Response
cluster.health 🟢 162/162 162/162

You can validate this API yourself by using the make validate target.

@l-trotta l-trotta merged commit 33f1884 into main Apr 7, 2025
8 checks passed
@l-trotta l-trotta deleted the fix-cluster-health-resp branch April 7, 2025 11:53
github-actions bot pushed a commit that referenced this pull request Apr 7, 2025
github-actions bot pushed a commit that referenced this pull request Apr 7, 2025
github-actions bot pushed a commit that referenced this pull request Apr 7, 2025
l-trotta added a commit that referenced this pull request Apr 7, 2025
(cherry picked from commit 33f1884)

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Apr 7, 2025
(cherry picked from commit 33f1884)

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Apr 7, 2025
(cherry picked from commit 33f1884)

Co-authored-by: Laura Trotta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants