Skip to content

Commit e18041d

Browse files
jkhelilcursoragent
andcommitted
fix(kustomize): move platform-specific CRDs to shared base
Newer kustomize versions enforce a security restriction that rejects direct file references pointing outside the kustomization root. config/kubernetes/base and config/openshift/base both referenced CRD files via ../../base/generated-crds/..., which violates that restriction and caused kustomize build to silently fail during releases (the error was swallowed because the pipeline does not set pipefail). Move tektondashboards and tektonaddons CRDs from their platform- specific kustomization.yaml files into config/base/kustomization.yaml where controller-gen already writes all generated CRDs. The extra CRDs on each platform are harmless — their reconcilers are platform-gated. Signed-off-by: Jawed khelil <jkhelil@redhat.com> Assisted-by: Claude Sonnet 4.6 (via Cursor) Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent 818e85d commit e18041d

3 files changed

Lines changed: 2 additions & 2 deletions

File tree

config/base/kustomization.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,4 +44,6 @@ resources:
4444
- generated-crds/operator.tekton.dev_tektonpruners.yaml
4545
- generated-crds/operator.tekton.dev_tektonresults.yaml
4646
- generated-crds/operator.tekton.dev_tektonschedulers.yaml
47+
- generated-crds/operator.tekton.dev_tektondashboards.yaml
48+
- generated-crds/operator.tekton.dev_tektonaddons.yaml
4749
- generated-crds/operator.tekton.dev_tektontriggers.yaml

config/kubernetes/base/kustomization.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ apiVersion: kustomize.config.k8s.io/v1beta1
3030
kind: Kustomization
3131
resources:
3232
- ../../base/
33-
- ../../base/generated-crds/operator.tekton.dev_tektondashboards.yaml
3433
- ../../webhooks/
3534
- operator_service.yaml
3635
- pipelinesascode.yaml

config/openshift/base/kustomization.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ patchesStrategicMerge:
5252

5353
resources:
5454
- ../../base/
55-
- ../../base/generated-crds/operator.tekton.dev_tektonaddons.yaml
5655
- ../../webhooks
5756
- operator_service.yaml
5857
- operator_servicemonitor.yaml

0 commit comments

Comments
 (0)