Skip to content
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

[metricbeat:elasticsearch module] Missing jvm.mem.pools metrics from node.stats #40539

Closed
przemek-grzedzielski opened this issue Aug 15, 2024 · 1 comment · Fixed by #40571
Closed
Assignees
Labels
Team:Monitoring Stack Monitoring team

Comments

@przemek-grzedzielski
Copy link

I found that metricbeat v 8.13.4 is not ingesting some of the elasticsearch.node.stats.jvm.mem.* metrics, in particular: elasticsearch.node.stats.jvm.mem.pools.old.max.bytes and elasticsearch.node.stats.jvm.mem.pools.old.used.bytes.

Note that these (and other missing metrics) are listed in the Metricbeat 8.x documentation. They're also present in the mappings:

They're also exposed via Elasticsearch APIs as per the node_stats API documentation (also tested that this is the case).

@nkvoll traced the changes back to this commit or this Pull Request.

It seems that these fields have been removed inadvertently and the ask is that they be brought back since a number of services depend on them.


For confirmed bugs, please report:

  • Version: 8.13.4
  • Operating System: Ubuntu 22.04
@przemek-grzedzielski przemek-grzedzielski added the Team:Monitoring Stack Monitoring team label Aug 15, 2024
@przemek-grzedzielski przemek-grzedzielski changed the title [elasticsearch module] Missing jvm.mem.pools metrics from node.stats [metricbeat:elasticsearch module] Missing jvm.mem.pools metrics from node.stats Aug 15, 2024
@consulthys
Copy link
Contributor

@przemek-grzedzielski The actual PR where the elasticsearch.node.stats.jvm.mem.pools.* fields were actually removed is #23901 but the PR doesn't contain any reason why.

My hunch is that those fields have been removed because they were nowhere used in the Stack Monitoring UI, which looking at the SM UI source code seems to confirm this.

Maybe @pickypg has an idea or some background context since he was involved at that time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Monitoring Stack Monitoring team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants