search

jaegertracing / helm-charts

Helm Charts for Jaeger backend
Apache License 2.0
258 stars 340 forks source link

[Bug]: Setting Jaeger Operator webhook port doesn't change the operator flag to bind correctly #473

Closed stevehipwell closed 1 year ago

stevehipwell commented 1 year ago

What happened?

When setting webhooks.port for the Jaeger Operator Helm chart it doesn't set the --webhook-bind-port flag which means it doesn't work.

Steps to reproduce

  1. Set webhooks.port to a non-standard value
  2. Add a Jager CR

Expected behavior

I'd expect the Jager CR to be created successfully.

Relevant log output

Error: cannot patch "jaeger-operator-jaeger" with kind Jaeger: Internal error occurred: failed calling webhook "mjaeger.kb.io": failed to call webhook: Post "https://jaeger-operator-webhook-service.jaeger-operator.svc:443/mutate-jaegertracing-io-v1-jaeger?timeout=10s": dial tcp 100.65.139.131:10250: connect: connection refused

Screenshot

No response

Additional context

No response

Jaeger backend version

No response

SDK

No response

Pipeline

No response

Stogage backend

No response

Operating system

No response

Deployment model

No response

Deployment configs

No response