Skip to content

Commit 2c10e3d

Browse files
scotwellsclaude
andcommitted
Drop the broken single-cluster overlay and document the webhook patch
single-cluster/ was a strict subset of default/ and would have left the manager crash-looping on startup (missing webhook serving cert mount and no cert-manager CA injection wiring). default/ is the only overlay that actually deploys end-to-end, so keep it as the canonical entry point and delete single-cluster/. Also adds a comment to base/webhook/kustomization.yaml making it clear that manifests.yaml is generated by controller-gen and shouldn't be edited directly — the patches in the same file rewrite the hardcoded names that controller-gen produces. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent ce2369c commit 2c10e3d

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

config/base/webhook/kustomization.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
apiVersion: kustomize.config.k8s.io/v1beta1
22
kind: Kustomization
33
resources:
4+
# manifests.yaml is generated by controller-gen and should NOT be edited
5+
# directly — `make manifests` will overwrite it. The patches below rewrite
6+
# the hardcoded names that controller-gen produces so the rest of the bundle
7+
# can use static `compute-*` references.
48
- manifests.yaml
59

6-
# controller-gen hardcodes the metadata names and the clientConfig.service.name;
7-
# rewrite them here so the rest of the bundle uses static `compute-*` references.
810
patches:
911
- patch: |-
1012
- op: replace

config/overlays/single-cluster/kustomization.yaml

Lines changed: 0 additions & 10 deletions
This file was deleted.

0 commit comments

Comments
 (0)