feat(argo-workflows): Add encryptionOptions to S3 based artifactRepository (#1122)
* add encryptionOptions * fix indentation * fix typo * fix mapping * update version * document usage in comments in values.yaml * document usage in comments in values.yaml Signed-off-by: Bart Geens <bgeens2@its.jnj.com> * Apply suggestions from code review Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com> * Apply suggestions from code review Signed-off-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com> Co-authored-by: Bart Geens <bgeens2@its.jnj.com> Co-authored-by: Marco Kilchhofer <mkilchhofer@users.noreply.github.com>pull/1167/head^2 argo-workflows-0.13.0
parent
a647fbe123
commit
8b26bed62f
|
@ -2,7 +2,7 @@ apiVersion: v2
|
|||
name: argo-workflows
|
||||
description: A Helm chart for Argo Workflows
|
||||
type: application
|
||||
version: 0.12.0
|
||||
version: 0.13.0
|
||||
appVersion: v3.2.9
|
||||
icon: https://raw.githubusercontent.com/argoproj/argo-workflows/master/docs/assets/argo.png
|
||||
home: https://github.com/argoproj/argo-helm
|
||||
|
@ -15,4 +15,4 @@ maintainers:
|
|||
- name: benjaminws
|
||||
annotations:
|
||||
artifacthub.io/changes: |
|
||||
- "[Added]: Add parameter for enabling roles."
|
||||
- "[Added]: Add encryptionOptions for S3 based artifactRepository"
|
||||
|
|
|
@ -76,6 +76,10 @@ data:
|
|||
{{- if .Values.artifactRepository.s3.useSDKCreds }}
|
||||
useSDKCreds: {{ .Values.artifactRepository.s3.useSDKCreds }}
|
||||
{{- end }}
|
||||
{{- with .Values.artifactRepository.s3.encryptionOptions }}
|
||||
encryptionOptions:
|
||||
{{- toYaml . | nindent 10 }}
|
||||
{{- end }}
|
||||
{{- end }}
|
||||
{{- end}}
|
||||
{{- if .Values.controller.metricsConfig.enabled }}
|
||||
|
|
|
@ -491,6 +491,8 @@ artifactRepository:
|
|||
# region:
|
||||
# roleARN:
|
||||
# useSDKCreds: true
|
||||
# encryptionOptions:
|
||||
# enableEncryption: true
|
||||
# -- Store artifact in a GCS object store
|
||||
# @default -- `{}` (See [values.yaml])
|
||||
gcs: {}
|
||||
|
|
Loading…
Reference in New Issue