Skip to content

Commit

Permalink
fix: propagate custom webhook service ports in WebhookConfigurations (#…
Browse files Browse the repository at this point in the history
…263)

Setting the `operator.webhookPort` value when installing the operator chart could result in calls failing calls to the Mutating and Validation webhooks of the operator.

This, in turn, could prevent, for example, the creation of `Dash0OperatorConfiguration` or `Dash0Monitoring` resources.
  • Loading branch information
mmanciop authored Feb 6, 2025
1 parent da62291 commit 64fc498
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,7 @@ webhooks:
service:
name: {{ template "dash0-operator.chartName" . }}-webhook-service
namespace: {{ .Release.Namespace }}
port: {{ .Values.operator.webhookPort }}
path: /v1alpha1/inject/dash0
failurePolicy: Ignore
rules:
Expand Down Expand Up @@ -339,6 +340,7 @@ webhooks:
service:
name: {{ template "dash0-operator.chartName" . }}-webhook-service
namespace: {{ .Release.Namespace }}
port: {{ .Values.operator.webhookPort }}
path: /v1alpha1/validate/operator-configuration
admissionReviewVersions:
- v1
Expand Down Expand Up @@ -371,6 +373,7 @@ webhooks:
service:
name: {{ template "dash0-operator.chartName" . }}-webhook-service
namespace: {{ .Release.Namespace }}
port: {{ .Values.operator.webhookPort }}
path: /v1alpha1/validate/monitoring
admissionReviewVersions:
- v1
Expand Down

0 comments on commit 64fc498

Please sign in to comment.