Skip to content

Commit c781e14

Browse files
committed
fix(nextcloud): move ci lint-test values to yaml file
Signed-off-by: WrenIX <[email protected]>
1 parent 7286e3f commit c781e14

File tree

6 files changed

+51
-5
lines changed

6 files changed

+51
-5
lines changed

.github/workflows/lint-test.yaml

+4-4
Original file line numberDiff line numberDiff line change
@@ -72,21 +72,21 @@ jobs:
7272

7373
# test the helm chart with postgresql subchart enabled
7474
- name: PostgreSQL Enabled
75-
helm_args: '--helm-extra-set-args "--set=postgresql.enabled=true --set=postgresql.global.postgresql.auth.password=testing123456 --set=internalDatabase.enabled=false --set=externalDatabase.enabled=True --set=externalDatabase.type=postgresql --set=externalDatabase.password=testing12345"'
75+
helm_args: '--helm-extra-set-args "--values charts/nextcloud/test-values/postgresql.yaml"'
7676

7777
# test the helm chart with nginx container enabled
7878
- name: Nginx Enabled
79-
helm_args: '--helm-extra-set-args "--set=image.flavor=fpm --set=nginx.enabled=true"'
79+
helm_args: '--helm-extra-set-args "--values charts/nextcloud/test-values/nginx.yaml"'
8080

8181
# test the helm chart with horizontal pod autoscaling enabled
8282
- name: Horizontal Pod Autoscaling Enabled
83-
helm_args: '--helm-extra-set-args "--set=hpa.enabled=true --set=hpa.minPods=2 --set=hpa.maxPods=3 --set=hpa.targetCPUUtilizationPercentage=75"'
83+
helm_args: '--helm-extra-set-args "--values charts/nextcloud/test-values/hpa.yaml"'
8484

8585
# test the helm chart with s3 as the primary storage
8686
- name: S3 Enabled as Primary Storage
8787
# we need to skip the clean up so we can test adding a file
8888
helm_args: |
89-
--namespace nextcloud --skip-clean-up --helm-extra-set-args "--set=fullnameOverride=nextcloud --set=nextcloud.objectStore.s3.enabled=true --set=nextcloud.objectStore.s3.accessKey=nextcloud --set=nextcloud.objectStore.s3.secretKey=rootpass123 --set=nextcloud.objectStore.s3.host=minio.nextcloud.svc.cluster.local --set=nextcloud.objectStore.s3.port=9000 --set=nextcloud.objectStore.s3.ssl=false --set=nextcloud.objectStore.s3.bucket=nextcloud --set=nextcloud.objectStore.s3.usePathStyle=true --set=image.flavor=fpm --set=nginx.enabled=true --set=nextcloud.host=nextcloud --set=nextcloud.trustedDomains[0]='*'"
89+
--namespace nextcloud --skip-clean-up --helm-extra-set-args "--values charts/nextcloud/test-values/s3-as-primary-storage.yaml"
9090
9191
steps:
9292
- name: Checkout

charts/nextcloud/Chart.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v2
22
name: nextcloud
3-
version: 6.2.2
3+
version: 6.2.3
44
appVersion: 30.0.2
55
description: A file sharing server that puts the control and security of your own data back into your hands.
66
keywords:

charts/nextcloud/test-values/hpa.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
hpa:
2+
enabled: true
3+
minPods: 2
4+
maxPods: 3
5+
targetCPUUtilizationPercentage: 75
+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
image:
2+
flavor: fpm
3+
4+
nginx:
5+
enabled: true
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
postgresql:
2+
enabled: true
3+
global:
4+
postgresql:
5+
auth:
6+
password: "testing123456"
7+
8+
internalDatabase:
9+
enabled: false
10+
11+
externalDatabase:
12+
enabled: true
13+
type: postgresql
14+
password: "testing123456"
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
fullnameOverride: nextcloud
2+
3+
image:
4+
flavor: fpm
5+
6+
nginx:
7+
enabled: true
8+
9+
nextcloud:
10+
host: nextcloud
11+
trustedDomains:
12+
- '*'
13+
objectStore:
14+
s3:
15+
enabled: true
16+
accessKey: nextcloud
17+
secretKey: rootpass123
18+
host: minio.nextcloud.svc.cluster.local
19+
port: 9000
20+
ssl: false
21+
bucket: nextcloud
22+
usePathStyle: true

0 commit comments

Comments
 (0)