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

Deprecate/remove xpack.monitoring.ui.container.elasticsearch.enabled setting? #35582

Open
ycombinator opened this issue Apr 25, 2019 · 5 comments
Labels
discuss Team:Monitoring Stack Monitoring team

Comments

@ycombinator
Copy link
Contributor

Today, users must set the xpack.monitoring.ui.container.elasticsearch.enabled setting in their kibana.yml if their Elasticsearch nodes are running in a containerized environment. Otherwise CPU stats in the Monitoring UI are not shown correctly.

I'm wondering if we can discover that Elasticsearch is running in a containerized environment from data in .monitoring-es-* itself and therefore get rid of this kibana.yml setting.

@ycombinator ycombinator added discuss Team:Monitoring Stack Monitoring team labels Apr 25, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring

@cachedout
Copy link
Contributor

I think @pickypg might have some thoughts here.

@pickypg
Copy link
Member

pickypg commented Apr 26, 2019

I'd love to see us do that. We discussed doing something like this awhile ago, like from Metricbeat data just having it report the data correctly from the onset and avoiding the setting altogether (8.0 presumably).

The biggest issue with automatically determining it at query time is that:

  1. Per node page, it's not so bad -- it's one query and adjustment.
  2. Per the node listing page, you have to be sure that it's the same across all nodes because it's possible, albeit rare, for a cluster to have some nodes getting CPU throttled containers and some not.

@ycombinator
Copy link
Contributor Author

like from Metricbeat data just having it report the data correctly from the onset

Meaning, this is not reported today by the internal node_stats ES collector?

@pickypg
Copy link
Member

pickypg commented Apr 26, 2019

Exactly. The internal ES node stats report the CPU as a system value and, separately, as the Cgroup value.

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

No branches or pull requests

4 participants