-
Notifications
You must be signed in to change notification settings - Fork 472
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
Create Prometheus ServiceMonitor/PodMonitor for collector #1036
Comments
iirc collector internal metrics are currently done only through prometheus? I would love to open an issue in the collector project about emitting OTLP, however. For the operator creating a CRD, I think making a service monitor would make sense. In the manually created one we generated, we had to specify some extra labels so we didn't double scrape targets between the I can take on this work as well if that would be helpful. |
I'm not the smartest kid here, but I wonder if this is what I'm looking for. I've got the kube prometheus stack helm chart, and the otel collector chart running in my cluster. I need prometheus to scrape otel col. Currentlly, I've configured the prometheus exporter on port 8889. I also created a service to expose this port in the cluster. And now I'm trying to create a ServiceMonitor, but it's not quite working. I'm sure I'll figure it out.. but I landed here because I was looking to see if anyone else had done this before. Surely I'm not the first person. |
Hi @alexdresko, creating a ServiceMonitor for the collector may not work until the ports are included for the container (#1011). I included my ServiceMonitor file as a part of the |
@pavolloffay is this the same as #1768? |
This is closed by #1963 |
The operator could create Prometheus ServiceMonitor/PodMonitor which would enable scraping of collector metrics.
This functionality could be conditionally enabled in CR along when the p8s CRDs are present in the cluster.
We should also investigate sending collector internal metrics via {OTLP} exporter configured in the collector.
cc) @kevinearls
The text was updated successfully, but these errors were encountered: