Skip to content
This repository was archived by the owner on Apr 24, 2024. It is now read-only.

Commit 632bbaa

Browse files
committed
Fix issue with missing "--api-export-name=" in args set in deployment after kustomize
Signed-off-by: Frederic Giloux <[email protected]>
1 parent ee3be8d commit 632bbaa

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

config/default/kustomization.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,10 @@ vars:
7272
# kind: Service
7373
# version: v1
7474
# name: webhook-service
75-
76-
configurations:
77-
- kustomizeconfig.yaml
75+
- name: API_EXPORT_NAME
76+
objref:
77+
apiVersion: apis.kcp.dev/v1alpha1
78+
kind: APIExport
79+
name: data.my.domain
80+
fieldref:
81+
fieldPath: metadata.name

config/default/kustomizeconfig.yaml

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

config/default/manager_auth_proxy_patch.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ spec:
2929
memory: 64Mi
3030
- name: manager
3131
args:
32-
- data.my.domain
32+
- "--api-export-name=$(API_EXPORT_NAME)"
3333
- "--health-probe-bind-address=:8081"
3434
- "--metrics-bind-address=127.0.0.1:8080"
3535
- "--leader-elect"

config/manager/manager.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ spec:
3030
- command:
3131
- /manager
3232
args:
33-
- --api-export-name data.my.domain
33+
- --api-export-name=data.my.domain
3434
- --leader-elect
3535
image: controller:latest
3636
name: manager

0 commit comments

Comments
 (0)