Skip to content

Commit

Permalink
Add helm config for PersistentStorge
Browse files Browse the repository at this point in the history
  • Loading branch information
hardillb committed Jul 3, 2024
1 parent c389540 commit ca5abf1
Show file tree
Hide file tree
Showing 4 changed files with 46 additions and 1 deletion.
11 changes: 11 additions & 0 deletions helm/flowforge/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,17 @@ Enables FlowForge Telemetry
- `forge.fileStore.labels` allows to add custom labels to the file-server related objects (e.g. deployment, services, etc.) (default `{}`)
- `forge.fileStore.podLabels` allows to add custom labels to the file-server pod (default `{}`)

### Persistent Storage

This is a replacement for the File Storage, both can be run at once to enable migration

- `forge.persistentStorage.enabled` (default `false`)
- `forge.persistentStorage.size` The size of the Persistent Volume that will be mounted into each instance e.g. `5Gi` (default not set)
- `forge.persistentStorage.storageClass` The name of the Kubernetes Storage Class used to provision the volumes (default not set)
- `forge.persistentStorage.storageClassEFSTag` The name of a AWS Tag used to find EFS Volumes when needing to scale (default not set)

One of either `storageClass` or `storageClassEFSTag` needs to be set.

### Private Certificate Authority

- `forge.privateCA.configMapName` name of ConfigMap to store the CA Cert bundle (default `ff-ca-certs`)
Expand Down
15 changes: 14 additions & 1 deletion helm/flowforge/templates/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ data:
{{- end }}
{{- if .Values.forge.customHostname }}
customHostname:
enabled: {{ .Values.forge.customHostname.enabled }}
enabled: {{ .Values.forge.customHostname.enabled | default false }}
{{ if default false .Values.forge.customHostname.enabled -}}
cnameTarget: {{ required "A value is required for .Values.forge.customHostname.cnameTarget" .Values.forge.customHostname.cnameTarget }}
{{- end -}}
Expand All @@ -75,6 +75,19 @@ data:
ingressClass: {{ .Values.forge.customHostname.ingressClass }}
{{- end }}
{{- end }}
{{- if .Values.forge.persistentStorage }}
storage:
enabled: {{ .Values.forge.persistentStorage.enabled }}
{{- if .Values.forge.persistentStorage.storageClass }}
storageClass: {{ .Values.forge.persistentStorage.storageClass }}
{{- end }}
{{- if .Values.forge.persistentStorage.storageClassEFSTag }}
storageClassEFSTag: {{ .Values.forge.persistentStorage.storageClassEFSTag }}
{{- end }}
{{- if .Values.forge.persistentStorage.size }}
size: {{ .Values.forge.persistentStorage.size }}
{{- end }}
{{- end }}
{{- if .Values.forge.email }}
email:
enabled: true
Expand Down
18 changes: 18 additions & 0 deletions helm/flowforge/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -426,6 +426,24 @@
}
}
},
"persistentStorage": {
"type": "object",
"properties": {
"enabled": {
"type": "boolean"
},
"storageClass": {
"type": "string"
},
"storageClassEFSTag": {
"type": "string"
},
"size": {
"type": "string"
}
},
"required": [ "enabled" ]
},
"fileStore": {
"type": "object",
"properties": {
Expand Down
3 changes: 3 additions & 0 deletions helm/flowforge/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,9 @@ forge:
podLabels: {}
ingress.annotations: {}

persistentStorage:
enabled: false

fileStore:
enabled: false
type: localfs
Expand Down

0 comments on commit ca5abf1

Please sign in to comment.