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

Feedback related to metrics #5497

Open
ckdarby opened this issue Oct 9, 2020 · 1 comment
Open

Feedback related to metrics #5497

ckdarby opened this issue Oct 9, 2020 · 1 comment

Comments

@ckdarby
Copy link

ckdarby commented Oct 9, 2020

While rolling out Kubernetes scaling for Presto we weren't thrilled about some aspects or weren't able to find information in the docs to make the below suggestions moot.

These are the notes I made while we went through the experience.

Metrics be exposed as rest endpoints.

We'd rather avoid running JMX exporter for prometheus and having ports open for the JMX. With rest endpoints, it is straight forward to funnel everything through an ingress controller and force authentication, logging, and auditing.

Coordinator offers a single endpoint to pull in all node metrics

We'd rather be able to send a request to the coordinator and have it handle pulling in the individual worker metrics than us building tooling around asking k8s for all existing workers and making the individual requests out.

It appears someone else in the community asked a similar feature, here.

Cluster level metrics vs node level metrics

There are times we're interested in monitoring individual health of each node but also metrics that would be across the cluster. Cluster level metrics like total memory in use across all queries, total queued queries, etc. When using JMX it is very confusing when pointing at a worker node if it is reporting the stats of what it is executing or if they're the stats across the whole cluster or if it has asked the coordinator, etc.

@hashhar
Copy link
Member

hashhar commented Feb 26, 2024

There's now OpenMetrics support available. No docs yet (in progress) but for now instructions can be found at #1581 (comment).

It should address the first and last points however the middle point is not addressed by it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants