Skip to content

Commit fabcd79

Browse files
authored
migrate to helm-unittest/helm-unittest (#359)
* migrate to helm-unittest/helm-unittest * fix crd
1 parent 731d9c5 commit fabcd79

File tree

2 files changed

+30
-20
lines changed

2 files changed

+30
-20
lines changed

.github/workflows/chart-test.yml

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@ name: Chart Test
1818
on:
1919
pull_request:
2020
branches:
21-
- '*'
21+
- "*"
2222
paths:
23-
- 'charts/**'
24-
- '.github/workflows/chart-test.yml'
23+
- "charts/**"
24+
- ".github/workflows/chart-test.yml"
2525

2626
env:
2727
GOPRIVATE: github.com/streamnative
@@ -37,7 +37,7 @@ jobs:
3737
uses: actions/checkout@v3
3838
with:
3939
fetch-depth: 0
40-
40+
4141
- name: Check if the CRDs are update to date
4242
run: |
4343
diff config/crd/bases/resource.streamnative.io_pulsarconnections.yaml charts/pulsar-resources-operator/crds/resource.streamnative.io_pulsarconnections.yaml && \
@@ -63,14 +63,14 @@ jobs:
6363

6464
- name: Install plugin unittest
6565
run: |
66-
helm plugin install https://github.com/quintush/helm-unittest
66+
helm plugin install https://github.com/helm-unittest/helm-unittest
6767
6868
# Python is required because `ct lint` runs Yamale (https://github.com/23andMe/Yamale) and
6969
# yamllint (https://github.com/adrienverge/yamllint) which require Python
7070
- name: Set up Python
7171
uses: actions/setup-python@v4
7272
with:
73-
python-version: '3.10'
73+
python-version: "3.10"
7474

7575
- name: Set up chart-testing
7676
uses: helm/[email protected]
@@ -158,23 +158,23 @@ jobs:
158158
run: |
159159
echo "Waiting for pods to be ready..."
160160
kubectl wait --for=condition=Ready pods --all -n chart-testing --timeout=300s
161-
161+
162162
echo "Checking pod status..."
163163
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.phase}' | tr ' ' '\n' | grep -v "Running" | wc -l) -ne 0 ]]; then
164164
echo "Some pods are not running:"
165165
kubectl get pods -n chart-testing
166166
kubectl describe pods -n chart-testing
167167
exit 1
168168
fi
169-
169+
170170
echo "Checking for pod restarts..."
171171
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.containerStatuses[*].restartCount}' | tr ' ' '\n' | grep -v "^0$" | wc -l) -ne 0 ]]; then
172172
echo "Some pods have restarted:"
173173
kubectl get pods -n chart-testing
174174
kubectl describe pods -n chart-testing
175175
exit 1
176176
fi
177-
177+
178178
echo "All pods are running properly!"
179179
kubectl get pods -n chart-testing
180180
@@ -237,31 +237,31 @@ jobs:
237237
run: |
238238
echo "Waiting for pods to be ready..."
239239
kubectl wait --for=condition=Ready pods --all -n chart-testing --timeout=300s
240-
240+
241241
echo "Checking pod status..."
242242
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.phase}' | tr ' ' '\n' | grep -v "Running" | wc -l) -ne 0 ]]; then
243243
echo "Some pods are not running:"
244244
kubectl get pods -n chart-testing
245245
kubectl describe pods -n chart-testing
246246
exit 1
247247
fi
248-
248+
249249
echo "Checking for pod restarts..."
250250
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.containerStatuses[*].restartCount}' | tr ' ' '\n' | grep -v "^0$" | wc -l) -ne 0 ]]; then
251251
echo "Some pods have restarted:"
252252
kubectl get pods -n chart-testing
253253
kubectl describe pods -n chart-testing
254254
exit 1
255255
fi
256-
256+
257257
echo "All pods are running properly!"
258258
kubectl get pods -n chart-testing
259259
260260
- name: Setup tmate session
261261
uses: mxschmitt/action-tmate@v3
262262
timeout-minutes: 10
263263
if: failure()
264-
264+
265265
install-on-v1-28:
266266
runs-on: ubuntu-22.04
267267
needs: lint-test
@@ -316,23 +316,23 @@ jobs:
316316
run: |
317317
echo "Waiting for pods to be ready..."
318318
kubectl wait --for=condition=Ready pods --all -n chart-testing --timeout=300s
319-
319+
320320
echo "Checking pod status..."
321321
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.phase}' | tr ' ' '\n' | grep -v "Running" | wc -l) -ne 0 ]]; then
322322
echo "Some pods are not running:"
323323
kubectl get pods -n chart-testing
324324
kubectl describe pods -n chart-testing
325325
exit 1
326326
fi
327-
327+
328328
echo "Checking for pod restarts..."
329329
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.containerStatuses[*].restartCount}' | tr ' ' '\n' | grep -v "^0$" | wc -l) -ne 0 ]]; then
330330
echo "Some pods have restarted:"
331331
kubectl get pods -n chart-testing
332332
kubectl describe pods -n chart-testing
333333
exit 1
334334
fi
335-
335+
336336
echo "All pods are running properly!"
337337
kubectl get pods -n chart-testing
338338
@@ -395,27 +395,27 @@ jobs:
395395
run: |
396396
echo "Waiting for pods to be ready..."
397397
kubectl wait --for=condition=Ready pods --all -n chart-testing --timeout=300s
398-
398+
399399
echo "Checking pod status..."
400400
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.phase}' | tr ' ' '\n' | grep -v "Running" | wc -l) -ne 0 ]]; then
401401
echo "Some pods are not running:"
402402
kubectl get pods -n chart-testing
403403
kubectl describe pods -n chart-testing
404404
exit 1
405405
fi
406-
406+
407407
echo "Checking for pod restarts..."
408408
if [[ $(kubectl get pods -n chart-testing -o jsonpath='{.items[*].status.containerStatuses[*].restartCount}' | tr ' ' '\n' | grep -v "^0$" | wc -l) -ne 0 ]]; then
409409
echo "Some pods have restarted:"
410410
kubectl get pods -n chart-testing
411411
kubectl describe pods -n chart-testing
412412
exit 1
413413
fi
414-
414+
415415
echo "All pods are running properly!"
416416
kubectl get pods -n chart-testing
417417
418418
- name: Setup tmate session
419419
uses: mxschmitt/action-tmate@v3
420420
timeout-minutes: 10
421-
if: failure()
421+
if: failure()

charts/pulsar-resources-operator/crds/resource.streamnative.io_pulsarconnections.yaml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,11 @@ spec:
143143
For confidential clients, this would be the client secret.
144144
For public clients using JWT authentication, this would be the path to the JSON credentials file.
145145
properties:
146+
file:
147+
description: |-
148+
File points to a local file path whose contents will be used as the value.
149+
This is useful when running the operator locally without creating Kubernetes secrets.
150+
type: string
146151
secretRef:
147152
description: SecretKeyRef indicates a secret name and
148153
key
@@ -187,6 +192,11 @@ spec:
187192
This can be either a direct token value or a reference to a secret containing the token.
188193
If using a secret, the token should be stored under the specified key in the secret.
189194
properties:
195+
file:
196+
description: |-
197+
File points to a local file path whose contents will be used as the value.
198+
This is useful when running the operator locally without creating Kubernetes secrets.
199+
type: string
190200
secretRef:
191201
description: SecretKeyRef indicates a secret name and key
192202
properties:

0 commit comments

Comments
 (0)