Skip to content

Commit ddaf56b

Browse files
authored
Fix: enable webhooks in e2e (#171)
1 parent 1565b2c commit ddaf56b

File tree

3 files changed

+8
-5
lines changed

3 files changed

+8
-5
lines changed

.github/workflows/run-tests.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,8 +158,11 @@ jobs:
158158
- name: load-and-deploy-operator
159159
run: |
160160
kind load docker-image kind/ydb-operator:current
161-
- name: pull-and-load-ydb-image
161+
- name: pull-and-load-other-images
162162
run: |
163+
docker pull k8s.gcr.io/ingress-nginx/kube-webhook-certgen:v1.0
164+
kind load docker-image k8s.gcr.io/ingress-nginx/kube-webhook-certgen:v1.0
165+
163166
# TODO would be cool to parse YDB image from manifests to avoid duplicating information
164167
docker pull cr.yandex/crptqonuodf51kdj7a7d/ydb:22.4.44
165168
kind load docker-image cr.yandex/crptqonuodf51kdj7a7d/ydb:22.4.44

deploy/ydb-operator/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,9 +103,9 @@ webhook:
103103
injectCA: false
104104
# self-signed root certificate
105105
rootCert:
106-
duration: "" # default to be 5y
106+
duration: "" # default is 5y
107107
admissionCert:
108-
duration: "" # default to be 1y
108+
duration: "" # default is 1y
109109
# issuerRef:
110110
# name: "issuer"
111111
# kind: "ClusterIssuer"

e2e/operator-values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ image:
44
pullPolicy: Never
55

66
webhook:
7-
enabled: false
7+
enabled: true
88

99
patch:
10-
enabled: false
10+
enabled: true

0 commit comments

Comments
 (0)