From 225bae2d5783cb30d3b44f7693a42be967bfc3ef Mon Sep 17 00:00:00 2001
From: Kornesh Kanan <kornesh.kanan@gmail.com>
Date: Fri, 8 Feb 2019 15:10:28 +0800
Subject: [PATCH] Add archiveLogs option to Argo workflow controller config map

---
 charts/argo/templates/workflow-controller-config-map.yaml | 3 +++
 charts/argo/values.yaml                                   | 2 ++
 2 files changed, 5 insertions(+)

diff --git a/charts/argo/templates/workflow-controller-config-map.yaml b/charts/argo/templates/workflow-controller-config-map.yaml
index 4c97cfc6..fdf05c7c 100644
--- a/charts/argo/templates/workflow-controller-config-map.yaml
+++ b/charts/argo/templates/workflow-controller-config-map.yaml
@@ -17,6 +17,9 @@ data:
     {{- end }}
     artifactRepository:
       {{- if or .Values.minio.install .Values.useDefaultArtifactRepo }}
+      {{- if .Values.artifactRepository.archiveLogs }}
+      archiveLogs: {{ .Values.artifactRepository.archiveLogs }}
+      {{- end }}
       s3:
         {{- if .Values.useStaticCredentials }}
         accessKeySecret:
diff --git a/charts/argo/values.yaml b/charts/argo/values.yaml
index bb0985fa..7f014dfe 100644
--- a/charts/argo/values.yaml
+++ b/charts/argo/values.yaml
@@ -71,6 +71,8 @@ ui:
 useDefaultArtifactRepo: false
 useStaticCredentials: true
 artifactRepository:
+  # archiveLogs will archive the main container logs as an artifact
+  archiveLogs: true
   s3:
     # Note the `key` attribute is not the actual secret, it's the PATH to
     # the contents in the associated secret, as defined by the `name` attribute.