Skip to content

Commit

Permalink
Merge pull request #37 from loicwinkopp/icinga-stack-icinga2-extra-en…
Browse files Browse the repository at this point in the history
…v-vars

icinga-stack: extraEnvVars value for icinga2
  • Loading branch information
mocdaniel authored Feb 17, 2024
2 parents ad90734 + aa295a8 commit 8216931
Show file tree
Hide file tree
Showing 7 changed files with 66 additions and 1 deletion.
3 changes: 3 additions & 0 deletions charts/icinga-stack/charts/icinga2/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ spec:
- name: ICINGA_DISABLE_CONFD
value: {{ .Values.config.disable_confd | int | quote }}
{{- include "icinga2.envSecrets" . | nindent 12 }}
{{- if .Values.extraEnvVars }}
{{- toYaml .Values.extraEnvVars | nindent 12 }}
{{- end }}
ports:
- name: api
containerPort: {{ .Values.service.port }}
Expand Down
4 changes: 3 additions & 1 deletion charts/icinga-stack/charts/icinga2/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -309,4 +309,6 @@ securityContext: {}
# - ALL
# readOnlyRootFilesystem: true
# runAsNonRoot: true
# runAsUser: 1000
# runAsUser: 1000

extraEnvVars: []
3 changes: 3 additions & 0 deletions charts/icinga-stack/charts/icingadb/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,9 @@ spec:
{{- end }}
- name: ICINGADB_DATABASE_DATABASE
value: {{ .Values.global.databases.icingadb.database | default "mysql" | quote }}
{{- if .Values.extraEnvVars }}
{{- toYaml .Values.extraEnvVars | nindent 12 }}
{{- end }}
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{- with .Values.nodeSelector }}
Expand Down
2 changes: 2 additions & 0 deletions charts/icinga-stack/charts/icingadb/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -50,3 +50,5 @@ securityContext: {}
# readOnlyRootFilesystem: true
# runAsNonRoot: true
# runAsUser: 1000

extraEnvVars: []
27 changes: 27 additions & 0 deletions charts/icinga-stack/tests/icinga2_statefulset_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -891,3 +891,30 @@ tests:
path: "cert.crt"
- key: "influxdb-key-key"
path: "cert.key"

- it: deploys an Icinga2 StatefulSet with extra environment variables
values:
- required_values.yaml
set:
icinga2:
config:
ticket_salt:
value: "insecuresalt"
extraEnvVars:
- name: ENV_VAR_1
value: env_var_1
- name: ENV_VAR_2
value: env_var_2
release:
name: my-icinga
asserts:
- contains:
path: spec.template.spec.containers[0].env
content:
name: ENV_VAR_1
value: env_var_1
- contains:
path: spec.template.spec.containers[0].env
content:
name: ENV_VAR_2
value: env_var_2
24 changes: 24 additions & 0 deletions charts/icinga-stack/tests/icingadb_deployment_test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -66,3 +66,27 @@ tests:
secretKeyRef:
name: database-icingadb
key: password

- it: deploys an IcingaDB deployment with extra environment variables
values:
- required_values.yaml
set:
icingadb:
extraEnvVars:
- name: ENV_VAR_1
value: env_var_1
- name: ENV_VAR_2
value: env_var_2
release:
name: my-icinga
asserts:
- contains:
path: spec.template.spec.containers[0].env
content:
name: ENV_VAR_1
value: env_var_1
- contains:
path: spec.template.spec.containers[0].env
content:
name: ENV_VAR_2
value: env_var_2
4 changes: 4 additions & 0 deletions charts/icinga-stack/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -358,6 +358,8 @@ icingadb:
# runAsNonRoot: true
# runAsUser: 1000

extraEnvVars: []

icingaweb2:
enabled: true

Expand Down Expand Up @@ -492,6 +494,8 @@ icingaweb2:
# runAsNonRoot: true
# runAsUser: 1000

extraEnvVars: []

global:
api:
# host: # only needed if Icinga2 runs out of cluster
Expand Down

0 comments on commit 8216931

Please sign in to comment.