From 563c79c1b92d50600a6ca04f593aa1aaa2bb24cf Mon Sep 17 00:00:00 2001
From: callum-tait-pbx <52924845+callum-tait-pbx@users.noreply.github.com>
Date: Fri, 22 Jan 2021 01:03:25 +0000
Subject: [PATCH] feat/helm: add manager secret to Helm chart (#254)

* feat: adding maanger secret to Helm

* fix: correcting secret data format

* feat: adding in common labels

* fix: updating default values to have config

The auth config needs to be commented out by default as we don't want to deploy both configs empty. This may break stuff, so we want the user to actively uncomment the auth method they want instead

* chore: updating default format of cert

* chore: wording
---
 .../templates/manager_secrets.yaml                 | 14 ++++++++++++++
 charts/actions-runner-controller/values.yaml       | 11 +++++++++++
 2 files changed, 25 insertions(+)
 create mode 100644 charts/actions-runner-controller/templates/manager_secrets.yaml

diff --git a/charts/actions-runner-controller/templates/manager_secrets.yaml b/charts/actions-runner-controller/templates/manager_secrets.yaml
new file mode 100644
index 00000000..de98f5f7
--- /dev/null
+++ b/charts/actions-runner-controller/templates/manager_secrets.yaml
@@ -0,0 +1,14 @@
+{{- if or .Values.authSecret.enabled }}
+apiVersion: v1
+kind: Secret
+metadata:
+  name: controller-manager
+  namespace: {{ .Release.Namespace }}
+  labels:
+    {{- include "actions-runner-controller.labels" . | nindent 4 }}
+type: Opaque
+data:
+{{- range $k, $v := .Values.authSecret }}
+  {{ $k }}: {{ $v | toString | b64enc }}
+{{- end }} 
+{{- end }}
\ No newline at end of file
diff --git a/charts/actions-runner-controller/values.yaml b/charts/actions-runner-controller/values.yaml
index 3203d2b5..c507efbd 100644
--- a/charts/actions-runner-controller/values.yaml
+++ b/charts/actions-runner-controller/values.yaml
@@ -8,6 +8,17 @@ replicaCount: 1
 
 syncPeriod: 10m
 
+# Only 1 authentication method can be deployed at a time
+# Uncomment the configuration you are applying and fill in the details
+authSecret:
+  enabled: false
+  ### GitHub Apps Configuration
+  #github_app_id: ""
+  #github_app_installation_id: ""
+  #github_app_private_key: |
+  ### GitHub PAT Configuration
+  #github_token: ""
+
 image:
   repository: summerwind/actions-runner-controller
   # Overrides the manager image tag whose default is the chart appVersion if the tag key is commented out