incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

[Feature Request] Support push metrics to Prometheus Pushgateway

Open xianjingfeng opened this issue 2 years ago • 4 comments

The push mode will be more convenient and more real-time. We have implemented it. If necessary, I will create a PR

xianjingfeng avatar Aug 19 '22 14:08 xianjingfeng

Why will the push mode be more convenient and more real-time? Could you give some examples?

jerqi avatar Aug 22 '22 02:08 jerqi

1.We need install node exporter in every host, and if not, real-time performance cannot be ensured. 2.We need change the configration of node exporter after we add an instance.

xianjingfeng avatar Aug 22 '22 07:08 xianjingfeng

In our production environment, we use crontab to fetch the servers' metrics every minute and push the metrics to our prometheus server. I don't get the difference if we use exporter.

jerqi avatar Aug 22 '22 07:08 jerqi

This depends on operation and maintenance. If automation tools is well, it may not be inconvenient.

xianjingfeng avatar Aug 22 '22 07:08 xianjingfeng

@jerqi Can you share how do you collect metrics in your k8s cluster?

xianjingfeng avatar Dec 01 '22 14:12 xianjingfeng

@jerqi Can you share how do you collect metrics in your k8s cluster?

We plan to use a k8s sidecar to collect the metrics.

jerqi avatar Dec 01 '22 14:12 jerqi

I think we should support export metrics, like hadoop,hbase etc. Users can implement custom export plugin to export metrics to their monitoring system. WDYT? @jerqi

xianjingfeng avatar Dec 01 '22 14:12 xianjingfeng

I think we should support export metrics, like hadoop,hbase etc. Users can implement custom export plugin to export metrics to their monitoring system. WDYT? @jerqi

I'm not familiar with this aspect. If other mature systems have this , I think it will be ok for me. @zuston Do you have some suggestion?

jerqi avatar Dec 01 '22 14:12 jerqi

The similar mechanism is supported by other systems, like Flink: https://nightlies.apache.org/flink/flink-docs-release-1.13/docs/deployment/metric_reporters/

I think this is a good improvement of observability

zuston avatar Dec 02 '22 08:12 zuston

Let's reopen this?

zuston avatar Dec 02 '22 08:12 zuston

Let's reopen this?

OK.

jerqi avatar Dec 02 '22 08:12 jerqi

Should we have a concept of MetricsReporter refer to Flink first?

jerqi avatar Dec 02 '22 08:12 jerqi