Skip to content

Commit

Permalink
Extra Containers / PVCs Support (#34)
Browse files Browse the repository at this point in the history
* firefly-perfnode Chart and Extra Containers / PVCs Support

Signed-off-by: hfuss <[email protected]>

* comments

Signed-off-by: hfuss <[email protected]>

* headers

Signed-off-by: hfuss <[email protected]>

* moved firefly-perfnode chart to firefly-perf-cli repo

Signed-off-by: hfuss <[email protected]>
  • Loading branch information
onelapahead authored Mar 24, 2022
1 parent d3ce70b commit 1e2533d
Show file tree
Hide file tree
Showing 5 changed files with 32 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@
# Created by https://www.toptal.com/developers/gitignore/api/helm,macos,linux,windows,visualstudiocode,intellij
# Edit at https://www.toptal.com/developers/gitignore?templates=helm,macos,linux,windows,visualstudiocode,intellij

local-values.yaml
local-values*.yaml

### Helm ###
# Chart dependencies
**/charts/*.tgz
**/local-values.yaml
**/local-values*.yaml

### Intellij ###
# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio, WebStorm and Rider
Expand Down
2 changes: 1 addition & 1 deletion charts/firefly/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ name: firefly
description: A Helm chart for deploying FireFly and FireFly HTTPS Dataexchange onto Kubernetes.
type: application
appVersion: "0.14.0"
version: "0.4.0"
version: "0.4.1"

maintainers:
- name: hfuss
Expand Down
11 changes: 11 additions & 0 deletions charts/firefly/templates/core/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,10 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.core.podSecurityContext | nindent 8 }}
{{- if .Values.core.initContainers }}
initContainers:
{{- tpl .Values.core.initContainers . | nindent 8 }}
{{- end }}
containers:
- name: firefly
securityContext:
Expand Down Expand Up @@ -95,6 +99,9 @@ spec:
name: firefly-config
resources:
{{- toYaml .Values.core.resources | nindent 12 }}
{{- if .Values.core.extraContainers }}
{{- tpl .Values.core.extraContainers . | nindent 8 }}
{{- end }}
{{- with .Values.core.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
Expand All @@ -111,3 +118,7 @@ spec:
- name: firefly-config
secret:
secretName: {{ include "firefly.fullname" . }}-config
{{- if .Values.core.volumeClaimTemplates }}
volumeClaimTemplates:
{{- tpl .Values.core.volumeClaimTemplates . | nindent 4 }}
{{- end }}
10 changes: 10 additions & 0 deletions charts/firefly/templates/ethconnect/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@ spec:
{{- end }}
securityContext:
{{- toYaml .Values.ethconnect.podSecurityContext | nindent 8 }}
{{- if .Values.ethconnect.initContainers }}
initContainers:
{{- tpl .Values.ethconnect.initContainers . | nindent 8 }}
{{- end }}
containers:
- name: ethconnect
securityContext:
Expand Down Expand Up @@ -89,6 +93,9 @@ spec:
- mountPath: /etc/ethconnect/config.yaml
name: config
subPath: config.yaml
{{- if .Values.ethconnect.extraContainers }}
{{- tpl .Values.ethconnect.extraContainers . | nindent 8 }}
{{- end }}
{{- with .Values.ethconnect.nodeSelector }}
nodeSelector:
{{- toYaml . | nindent 8 }}
Expand Down Expand Up @@ -121,4 +128,7 @@ spec:
requests:
storage: {{ .size }}
{{- end }}
{{- if .Values.ethconnect.extraVolumeClaimTemplates }}
{{- tpl .Values.ethconnect.extraVolumeClaimTemplates . | nindent 4 }}
{{- end }}
{{- end }}
8 changes: 8 additions & 0 deletions charts/firefly/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,10 @@ core:
# runAsNonRoot: true
# runAsUser: 1000

initContainers: ""
extraContainers: ""
volumeClaimTemplates: ""

service:
type: ClusterIP
httpPort: 5000
Expand Down Expand Up @@ -519,6 +523,10 @@ ethconnect:
type: ClusterIP
apiPort: 5102

initContainers: ""
extraContainers: ""
extraVolumeClaimTemplates: ""

resources: {}
# We usually recommend not to specify default resources and to leave this as a conscious
# choice for the user. This also increases chances charts run on environments with little
Expand Down

0 comments on commit 1e2533d

Please sign in to comment.