Skip to content

Commit 244e2b4

Browse files
authored
Add an overlay for local development on Tilt (#1773)
Adds an overlay that removes the liveness and readiness probes as well as the --leader-elect flag from the operator-controller-controller-manager deployment since those will cause container restarts when the go debugger is stopped at breakpoints. Also edits the Tiltfile to use this new overlay target. Also adds our 1 existing feature flag to the new overlay. Signed-off-by: Tayler Geiger <[email protected]>
1 parent 05415ef commit 244e2b4

File tree

3 files changed

+30
-1
lines changed

3 files changed

+30
-1
lines changed

Tiltfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ load('.tilt-support', 'deploy_repo')
22

33
operator_controller = {
44
'image': 'quay.io/operator-framework/operator-controller',
5-
'yaml': 'config/overlays/cert-manager',
5+
'yaml': 'config/overlays/tilt-local-dev',
66
'binaries': {
77
'./cmd/operator-controller': 'operator-controller-controller-manager',
88
},
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# kustomization file for secure operator-controller
2+
# DO NOT ADD A NAMESPACE HERE
3+
apiVersion: kustomize.config.k8s.io/v1beta1
4+
kind: Kustomization
5+
resources:
6+
- ../../base
7+
components:
8+
- ../../components/tls
9+
# ca must be last or tls will overwrite the namespaces
10+
- ../../components/ca
11+
12+
patches:
13+
- target:
14+
kind: Deployment
15+
name: controller-manager
16+
path: patches/dev-deployment.yaml
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# remove livenessProbe and readinessProbe so container doesn't restart during breakpoints
2+
- op: replace
3+
path: /spec/template/spec/containers/0/livenessProbe
4+
value: null
5+
- op: replace
6+
path: /spec/template/spec/containers/0/readinessProbe
7+
value: null
8+
- op: remove
9+
# remove --leader-elect so container doesn't restart during breakpoints
10+
path: /spec/template/spec/containers/0/args/2
11+
- op: add
12+
path: /spec/template/spec/containers/0/args/-
13+
value: --feature-gates=PreflightPermissions=true

0 commit comments

Comments
 (0)