-
Notifications
You must be signed in to change notification settings - Fork 330
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
Add cluster info endpoint for /_elastic
#3594
Add cluster info endpoint for /_elastic
#3594
Conversation
_elastic
/_elastic
cb35e38
to
13dcd2a
Compare
13dcd2a
to
de78578
Compare
node_version_handler(build_info, runtime_info).or(node_config_handler(config)) | ||
node_version_handler(build_info, runtime_info) | ||
.or(node_config_handler(config.clone())) | ||
.or(es_compat_info_handler(build_info, config)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not the place to put ES handler right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these are the top-level handlers for cluster info. That's why it is built in ES folder but mounted here. functionally they belong to the same group. But I can just mount it there if you think this is more appropriate to you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But we want to keep all elasticsearch handlers mounted at the same place, under the _elastic
path, that's the current logic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed to mount all elastic-related endpoints from the same location.
5275593
to
a038a05
Compare
a038a05
to
2e39321
Compare
Can we add a rest-api test for this? It is (hopefully) relatively easy https://github.com/quickwit-oss/quickwit/blob/main/quickwit/rest-api-tests/README.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/_elastic
endpoint for showing basic cluster and build info/version
,/cluster
, and/config
Closes GET on
/api/v1/_elastic
should return basic info of the cluster #3582