Skip to content

Commit 0ba3a6d

Browse files
Merge pull request #1577 from abays/OSPRH-19353
[OSPRH-19353] Remove deprecated Kustomize fields in config dir
2 parents 2bbd4f2 + 6c51650 commit 0ba3a6d

6 files changed

Lines changed: 39 additions & 35 deletions

File tree

config/crd/kustomization.yaml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -12,22 +12,22 @@ resources:
1212
- bases/lightspeed.openstack.org_openstacklightspeeds.yaml
1313
#+kubebuilder:scaffold:crdkustomizeresource
1414

15-
patchesStrategicMerge:
15+
patches:
1616
# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix.
1717
# patches here are for enabling the conversion webhook for each CRD
18-
#- patches/webhook_in_core_openstackcontrolplanes.yaml
19-
#- patches/webhook_in_client_openstackclients.yaml
20-
#- patches/webhook_in_core_openstackversions.yaml
21-
#- patches/webhook_in_operator_openstacks.yaml
18+
#- path: patches/webhook_in_core_openstackcontrolplanes.yaml
19+
#- path: patches/webhook_in_client_openstackclients.yaml
20+
#- path: patches/webhook_in_core_openstackversions.yaml
21+
#- path: patches/webhook_in_operator_openstacks.yaml
2222
#+kubebuilder:scaffold:crdkustomizewebhookpatch
2323

2424
# [CERTMANAGER] To enable cert-manager, uncomment all the sections with [CERTMANAGER] prefix.
2525
# patches here are for enabling the CA injection for each CRD
26-
#- patches/cainjection_in_core_openstackcontrolplanes.yaml
27-
#- patches/cainjection_in_client_openstackclients.yaml
28-
#- patches/cainjection_in_core_openstackversions.yaml
29-
#- patches/cainjection_in_operator_openstacks.yaml
30-
#- patches/cainjection_in_openstacklightspeeds.yaml
26+
#- path: patches/cainjection_in_core_openstackcontrolplanes.yaml
27+
#- path: patches/cainjection_in_client_openstackclients.yaml
28+
#- path: patches/cainjection_in_core_openstackversions.yaml
29+
#- path: patches/cainjection_in_operator_openstacks.yaml
30+
#- path: patches/cainjection_in_openstacklightspeeds.yaml
3131
#+kubebuilder:scaffold:crdkustomizecainjectionpatch
3232

3333
# the following config is for teaching kustomize how to do kustomization for CRDs.

config/default/kustomization.yaml

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,11 @@ namespace: '{{ .OperatorNamespace }}'
99
namePrefix: openstack-operator-
1010

1111
# Labels to add to all resources and selectors.
12-
#commonLabels:
13-
# someName: someValue
12+
#labels:
13+
#- includeSelectors: true
14+
# includeTemplates: true
15+
# pairs:
16+
# someName: someValue
1417

1518
resources:
1619
#- ../crd
@@ -24,24 +27,25 @@ resources:
2427
# [PROMETHEUS] To enable prometheus monitor, uncomment all sections with 'PROMETHEUS'.
2528
#- ../prometheus
2629

27-
patchesStrategicMerge:
30+
patches:
2831
# Protect the /metrics endpoint by putting it behind auth.
2932
# If you want your controller-manager to expose the /metrics
3033
# endpoint w/o any authn/z, please comment the following line.
31-
- manager_auth_proxy_patch.yaml
34+
- path: manager_auth_proxy_patch.yaml
3235

3336
# Mount the controller config file for loading manager configurations
3437
# through a ComponentConfig type
35-
#- manager_config_patch.yaml
38+
#- path: manager_config_patch.yaml
3639

3740
# [WEBHOOK] To enable webhook, uncomment all the sections with [WEBHOOK] prefix including the one in
3841
# crd/kustomization.yaml
39-
- manager_webhook_patch.yaml
42+
- path: manager_webhook_patch.yaml
4043

4144
# [CERTMANAGER] To enable cert-manager, uncomment all sections with 'CERTMANAGER'.
4245
# Uncomment 'CERTMANAGER' sections in crd/kustomization.yaml to enable the CA injection in the admission webhooks.
4346
# 'CERTMANAGER' needs to be enabled to use ca injection
44-
- webhookcainjection_patch.yaml
47+
- path: mutatingwebhookcainjection_patch.yaml
48+
- path: validatingwebhookcainjection_patch.yaml
4549

4650
# the following config is for teaching kustomize how to do var substitution
4751
vars:
Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
# This patch add annotation to admission webhook config and
2+
# the variables $(CERTIFICATE_NAMESPACE) and $(CERTIFICATE_NAME) will be substituted by kustomize.
3+
apiVersion: admissionregistration.k8s.io/v1
4+
kind: MutatingWebhookConfiguration
5+
metadata:
6+
labels:
7+
app.kubernetes.io/name: mutatingingwebhookconfiguration
8+
app.kubernetes.io/instance: mutating-webhook-configuration
9+
app.kubernetes.io/component: webhook
10+
app.kubernetes.io/created-by: openstack-operator
11+
app.kubernetes.io/part-of: openstack-operator
12+
app.kubernetes.io/managed-by: kustomize
13+
name: mutating-webhook-configuration
14+
annotations:
15+
cert-manager.io/inject-ca-from: $(CERTIFICATE_NAMESPACE)/$(CERTIFICATE_NAME)
Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
# This patch add annotation to admission webhook config and
22
# the variables $(CERTIFICATE_NAMESPACE) and $(CERTIFICATE_NAME) will be substituted by kustomize.
3-
---
43
apiVersion: admissionregistration.k8s.io/v1
54
kind: ValidatingWebhookConfiguration
65
metadata:
@@ -14,17 +13,3 @@ metadata:
1413
name: validating-webhook-configuration
1514
annotations:
1615
cert-manager.io/inject-ca-from: $(CERTIFICATE_NAMESPACE)/$(CERTIFICATE_NAME)
17-
---
18-
apiVersion: admissionregistration.k8s.io/v1
19-
kind: MutatingWebhookConfiguration
20-
metadata:
21-
labels:
22-
app.kubernetes.io/name: mutatingingwebhookconfiguration
23-
app.kubernetes.io/instance: mutating-webhook-configuration
24-
app.kubernetes.io/component: webhook
25-
app.kubernetes.io/created-by: openstack-operator
26-
app.kubernetes.io/part-of: openstack-operator
27-
app.kubernetes.io/managed-by: kustomize
28-
name: mutating-webhook-configuration
29-
annotations:
30-
cert-manager.io/inject-ca-from: $(CERTIFICATE_NAMESPACE)/$(CERTIFICATE_NAME)

config/manifests/kustomization.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ resources:
99
# [WEBHOOK] To enable webhooks, uncomment all the sections with [WEBHOOK] prefix.
1010
# Do NOT uncomment sections with prefix [CERTMANAGER], as OLM does not support cert-manager.
1111
# These patches remove the unnecessary "cert" volume and its manager container volumeMount.
12-
#patchesJson6902:
12+
#patches:
1313
#- target:
1414
# group: apps
1515
# version: v1

config/scorecard/kustomization.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
resources:
22
- bases/config.yaml
3-
patchesJson6902:
3+
patches:
44
- path: patches/basic.config.yaml
55
target:
66
group: scorecard.operatorframework.io
@@ -13,4 +13,4 @@ patchesJson6902:
1313
version: v1alpha3
1414
kind: Configuration
1515
name: config
16-
#+kubebuilder:scaffold:patchesJson6902
16+
#+kubebuilder:scaffold:patches

0 commit comments

Comments
 (0)