Skip to content

Commit

Permalink
[feature/issue_377] Allow additional env variables to be added (#378)
Browse files Browse the repository at this point in the history
* [feature/issue_377] Allow additional env variables to be added

Signed-off-by: Balamurugan Balasubramanian <[email protected]>

* [feature/issue_377] Allow additional env variables to be added , extended to the subcharts - ui and kyverno plugin and bumped the chart version

Signed-off-by: Balamurugan Balasubramanian <[email protected]>

---------

Signed-off-by: Balamurugan Balasubramanian <[email protected]>
Co-authored-by: Balamurugan Balasubramanian <[email protected]>
  • Loading branch information
kbcbals and Balamurugan Balasubramanian authored Dec 6, 2023
1 parent a747418 commit e9719c8
Show file tree
Hide file tree
Showing 7 changed files with 20 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/policy-reporter/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ description: |
It creates Prometheus Metrics and can send rule validation events to different targets like Loki, Elasticsearch, Slack or Discord
type: application
version: 2.21.3
version: 2.21.4
appVersion: 2.17.4

icon: https://github.com/kyverno/kyverno/raw/main/img/logo.png
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,9 @@ spec:
key: password
optional: false
{{- end }}
{{- with .Values.envVars }}
{{- . | toYaml | trim | nindent 10 }}
{{- end }}
volumes:
- name: config-file
secret:
Expand Down
3 changes: 3 additions & 0 deletions charts/policy-reporter/charts/kyvernoPlugin/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,9 @@ podAnnotations: {}
# Key/value pairs that are attached to pods.
podLabels: {}

# Allow additional env variables to be added
envVars: []

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
3 changes: 3 additions & 0 deletions charts/policy-reporter/charts/ui/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,9 @@ spec:
valueFrom:
fieldRef:
fieldPath: metadata.namespace
{{- with .Values.envVars }}
{{- . | toYaml | trim | nindent 10 }}
{{- end }}
volumes:
- name: config-file
configMap:
Expand Down
5 changes: 4 additions & 1 deletion charts/policy-reporter/charts/ui/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ api:

# use redis as external log storage instead of an in memory store
# recommended when using a HA setup with more then one replica
# to get all logs on each instance
# to get all logs on each instance
redis:
enabled: false
address: ""
Expand Down Expand Up @@ -134,6 +134,9 @@ podAnnotations: {}
# Key/value pairs that are attached to pods.
podLabels: {}

# Allow additional env variables to be added
envVars: []

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
3 changes: 3 additions & 0 deletions charts/policy-reporter/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,9 @@ spec:
fieldRef:
fieldPath: metadata.name
{{- end }}
{{- with .Values.envVars }}
{{- . | toYaml | trim | nindent 10 }}
{{- end }}
volumes:
- name: sqlite
{{- if .Values.sqliteVolume }}
Expand Down
3 changes: 3 additions & 0 deletions charts/policy-reporter/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,9 @@ podAnnotations: {}
# Key/value pairs that are attached to pods.
podLabels: {}

# Allow additional env variables to be added
envVars: []

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 e9719c8

Please sign in to comment.