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

Missing notation for Services broke integration with ServiceMonitor #2877

Closed
iblancasa opened this issue Apr 18, 2024 · 1 comment · Fixed by #2878
Closed

Missing notation for Services broke integration with ServiceMonitor #2877

iblancasa opened this issue Apr 18, 2024 · 1 comment · Fixed by #2878
Labels
bug Something isn't working needs triage

Comments

@iblancasa
Copy link
Contributor

iblancasa commented Apr 18, 2024

Component(s)

collector

What happened?

Description

When we create a collector instance with prometheus exporter, the created ServiceMonitor requires the following label selectors:

  selector:
    matchLabels:
      app.kubernetes.io/component: opentelemetry-collector
      app.kubernetes.io/instance: chainsaw-otlp-metrics.cluster-collector
      app.kubernetes.io/managed-by: opentelemetry-operator
      app.kubernetes.io/part-of: opentelemetry
      operator.opentelemetry.io/collector-monitoring-service: Exists

The label operator.opentelemetry.io/collector-monitoring-service: Exists is available in the Monitoring service but not in the collector nor collector-headless services. Due to that, the ServiceMonitor cannot match the Service for the OTEL Collector and scrape the metrics from the Prometheus exporter.

Kubernetes Version

.

Operator version

.

Collector version

.

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")

Log output

No response

Additional context

No response

@iblancasa iblancasa added bug Something isn't working needs triage labels Apr 18, 2024
@iblancasa
Copy link
Contributor Author

I'm working on it.

VineethReddy02 added a commit to VineethReddy02/opentelemetry-operator that referenced this issue May 20, 2024
pavolloffay pushed a commit that referenced this issue May 20, 2024
* Prepare release 0.100.0

Signed-off-by: Vineeth Pothulapati <vineethpothulapati@outlook.com>

* update the chlog

* update the chlog with #2877 merge

---------

Signed-off-by: Vineeth Pothulapati <vineethpothulapati@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant