Skip to content

Commit

Permalink
fix(nextcloud): move ci lint-test values to yaml file
Browse files Browse the repository at this point in the history
Signed-off-by: WrenIX <[email protected]>
  • Loading branch information
wrenix committed Nov 15, 2024
1 parent 79d20ec commit f5e0918
Show file tree
Hide file tree
Showing 6 changed files with 51 additions and 5 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/lint-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,21 +72,21 @@ jobs:

# test the helm chart with postgresql subchart enabled
- name: PostgreSQL Enabled
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"'
helm_args: '--helm-extra-set-args "--values charts/nextcloud/test-values/postgresql.yaml"'

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

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

# test the helm chart with s3 as the primary storage
- name: S3 Enabled as Primary Storage
# we need to skip the clean up so we can test adding a file
helm_args: |
--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]='*'"
--namespace nextcloud --skip-clean-up --helm-extra-set-args "--values charts/nextcloud/test-values/s3-as-primary-storage.yaml"
steps:
- name: Checkout
Expand Down
2 changes: 1 addition & 1 deletion charts/nextcloud/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: nextcloud
version: 6.2.3
version: 6.2.4
appVersion: 30.0.2
description: A file sharing server that puts the control and security of your own data back into your hands.
keywords:
Expand Down
5 changes: 5 additions & 0 deletions charts/nextcloud/test-values/hpa.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
hpa:
enabled: true
minPods: 2
maxPods: 3
targetCPUUtilizationPercentage: 75
5 changes: 5 additions & 0 deletions charts/nextcloud/test-values/nginx.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
image:
flavor: fpm

nginx:
enabled: true
14 changes: 14 additions & 0 deletions charts/nextcloud/test-values/postgresql.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
postgresql:
enabled: true
global:
postgresql:
auth:
password: "testing123456"

internalDatabase:
enabled: false

externalDatabase:
enabled: true
type: postgresql
password: "testing123456"
22 changes: 22 additions & 0 deletions charts/nextcloud/test-values/s3-as-primary-storage.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
fullnameOverride: nextcloud

image:
flavor: fpm

nginx:
enabled: true

nextcloud:
host: nextcloud
trustedDomains:
- '*'
objectStore:
s3:
enabled: true
accessKey: nextcloud
secretKey: rootpass123
host: minio.nextcloud.svc.cluster.local
port: 9000
ssl: false
bucket: nextcloud
usePathStyle: true

0 comments on commit f5e0918

Please sign in to comment.