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

Unable to set health_check endpoint in config #38418

Open
MaartenDeMeyer opened this issue Mar 6, 2025 · 1 comment
Open

Unable to set health_check endpoint in config #38418

MaartenDeMeyer opened this issue Mar 6, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@MaartenDeMeyer
Copy link

Component(s)

No response

Describe the issue you're reporting

Description

I'm deploying the otel collector via a helm chart to my k8s cluster. I'm trying to update beyond v 0.103.0, but I can't get my health_check extension to survive the 0.0.0.0 -> localhost change in v 0.104.0. I can't set the UseLocalHostAsDefaultHost featuregate, since I'm deploying otel as part of a bigger chart and have no control over the CLI. I'm trying to set the health_check's endpoint to 0.0.0.0:13133 explicitly, but no matter what syntax I try, deployment fails with

cannot unmarshal string into Go struct field OpenTelemetryCollectorSpec.spec.config of type v1beta1.Config

chart:

apiVersion: opentelemetry.io/v1beta1
kind: OpenTelemetryCollector
metadata:
  name: otel-collector
spec:
  image: otel/opentelemetry-collector-contrib:latest
  mode: deployment
  config: |
    receivers:
      otlp:
        protocols:
          grpc:
          http:
...
   extensions:
      health_check:
        endpoint: "0.0.0.0:13133"
...
    service:
      telemetry:
        logs:
          level: "debug"
      extensions: [health_check]

Removing the single endpoint line from health_check causes config to pass fine. I've tried with & without various quotes, http:// prefix, checked whitespaces & indentation are correct, and using a health_check/1: entry as described here: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/extension/healthcheckextension/README.md . What am I missing?

@MaartenDeMeyer MaartenDeMeyer added the needs triage New item requiring triage label Mar 6, 2025
@atoulme
Copy link
Contributor

atoulme commented Mar 8, 2025

Sorry, no idea, but this sounds like an issue for the operator project. Try with the latest release and proceed from there to create a small reproduction case to send to the operator issues.

@atoulme atoulme added bug Something isn't working and removed needs triage New item requiring triage labels Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants