Skip to content

Comments

[6.x] Remove /api/_xpack/usage and /api/_kibana/v1/stats (#20800)#21008

Merged
tsullivan merged 1 commit intoelastic:6.xfrom
tsullivan:backport/6.x/pr-20800
Jul 20, 2018
Merged

[6.x] Remove /api/_xpack/usage and /api/_kibana/v1/stats (#20800)#21008
tsullivan merged 1 commit intoelastic:6.xfrom
tsullivan:backport/6.x/pr-20800

Conversation

@tsullivan
Copy link
Member

Backports the following commits to 6.x:

/api/_xpack/usage was added as a target for 6.4.0 but it will not be
used. Instead, the /api/stats response will include usage info on
everything that gets registered with the usage service in
/src/server/usage

/api/_kibana/v1/stats is a GET API that was added in 6.2, during a point
where we thought providing usage stats through a public API would be OK
for capturing internally, with the benefit of having it be visible.

However, we've pivoted away from that idea because it doesn't line up
too well with the existing flow of data, where usage stats are combined
with the "Kibana stats" such as process uptime and number of requests.
We want to shift how we collect stats from Kibana, but it will be
gradual. It might be a while before we have an architecture that makes
sense for a standalone public API for the usage stats

This endpoint was never documented, and isn't used anywhere in the code.
It does incur a maintenance cost though.

Therefore, instead of waiting for a next major version to remove this
API, I'm removing it for 6.4. It will be marked in the release notes as
a breaking change. Since it was never documented, it should not provide
a problem.
@tsullivan tsullivan added the backport This PR is a backport of another PR label Jul 19, 2018
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@tsullivan tsullivan merged commit 9a6e665 into elastic:6.x Jul 20, 2018
@tsullivan tsullivan deleted the backport/6.x/pr-20800 branch July 20, 2018 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants