Skip to content

Commit b5ed105

Browse files
Techassisbernauer
andcommitted
fix: Only select Deployment in Service
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.tech>
1 parent 504cf96 commit b5ed105

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

deploy/helm/secret-operator/templates/controller-deployment.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ spec:
1717
{{- toYaml . | nindent 8 }}
1818
{{- end }}
1919
labels:
20+
# TODO (@Techassi): Gate this behind the maintance.customResourceDefinition.maintain field
21+
webhook.stackable.tech/conversion: enabled
2022
{{- include "operator.selectorLabels" . | nindent 8 }}
2123
spec:
2224
{{- with .Values.image.pullSecrets }}
@@ -48,6 +50,8 @@ spec:
4850
# (which is turn pulls in https://github.com/stackabletech/operator-rs/blob/main/crates/stackable-operator/src/cli.rs)
4951
# You can read there about the expected values and purposes.
5052

53+
# TODO (@Techassi): Gate the DISABLE_CRD_MAINTENANCE env var behind maintance.customResourceDefinition.maintain field
54+
5155
# Sometimes products need to know the operator image, e.g. the opa-bundle-builder OPA
5256
# sidecar uses the operator image.
5357
- name: OPERATOR_IMAGE

deploy/helm/secret-operator/templates/service.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
---
32
apiVersion: v1
43
kind: Service
@@ -11,7 +10,9 @@ metadata:
1110
{{- include "operator.labels" . | nindent 4 }}
1211
spec:
1312
selector:
14-
{{- include "operator.selectorLabels" . | nindent 6 }}
13+
# TODO (@Techassi): Gate this behind the maintance.customResourceDefinition.maintain field
14+
webhook.stackable.tech/conversion: enabled
15+
{{- include "operator.selectorLabels" . | nindent 4 }}
1516
ports:
1617
- name: conversion-webhook
1718
protocol: TCP

0 commit comments

Comments
 (0)