DM-45394 mobu: Config from a single YAML configmap

applications/mobu/README.md

@@ -14,10 +14,11 @@ Continuous integration testing
 |-----|------|---------|-------------|
 | affinity | object | `{}` | Affinity rules for the mobu frontend pod |
 | config.autostart | list | `[]` | Autostart specification. Must be a list of mobu flock specifications. Each flock listed will be automatically started when mobu is started. |
-| config.debug | bool | `false` | If set to true, include the output from all flocks in the main mobu log and disable structured JSON logging. |
-| config.githubCiApp | object | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration |
-| config.githubRefreshApp | object | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_refresh_app.html#add-phalanx-configuration |
+| config.githubCiApp | string | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration |
+| config.githubRefreshApp | string | disabled. | Configuration for the GitHub refresh app integration. See https://mobu.lsst.io/operations/github_refresh_app.html#add-phalanx-configuration |
+| config.logLevel | string | `"INFO"` | Log level. Set to 'DEBUG' to include the output from all flocks in the main mobu log. |
 | config.pathPrefix | string | `"/mobu"` | Prefix for mobu's API routes. |
+| config.profile | string | `"production"` | One of 'production' or 'development'. 'production' configures structured JSON logging, and 'development' configures unstructured human readable logging. |
 | config.slackAlerts | bool | `true` | Whether to send alerts and status to Slack. |
 | fullnameOverride | string | `""` | Override the full name for resources (includes the release name) |
 | global.baseUrl | string | Set by Argo CD | Base URL for the environment |

applications/mobu/templates/configmap-config.yaml

@@ -0,0 +1,9 @@
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: {{ include "mobu.fullname" . }}-config
+  labels:
+    {{- include "mobu.labels" . | nindent 4 }}
+data:
+  config.yaml: |
+    {{- toYaml .Values.config | nindent 4 }}

applications/mobu/templates/deployment.yaml

@@ -13,8 +13,10 @@ spec:
     type: "Recreate"
   template:
     metadata:
-      {{- with .Values.podAnnotations }}
       annotations:
+        # Force the pod to restart when the config map is updated.
+        checksum/config: {{ include (print $.Template.BasePath "/configmap-config.yaml") . | sha256sum }}
+      {{- with .Values.podAnnotations }}
         {{- toYaml . | nindent 8 }}
       {{- end }}
       labels:
@@ -31,10 +33,6 @@ spec:
                   name: {{ template "mobu.fullname" . }}-secret
                   key: "ALERT_HOOK"
             {{- end }}
-            {{- if .Values.config.autostart }}
-            - name: "MOBU_AUTOSTART_PATH"
-              value: "/etc/mobu/autostart.yaml"
-            {{- end }}
             - name: "MOBU_ENVIRONMENT_URL"
               value: {{ .Values.global.baseUrl }}
             - name: "MOBU_GAFAELFAWR_TOKEN"
@@ -43,17 +41,13 @@ spec:
                   name: {{ template "mobu.fullname" . }}-gafaelfawr-token
                   key: "token"
             {{- if .Values.config.githubRefreshApp }}
-            - name: "MOBU_GITHUB_REFRESH_APP_CONFIG_PATH"
-              value: "/etc/mobu/github-refresh-app.yaml"
             - name: "MOBU_GITHUB_REFRESH_APP_WEBHOOK_SECRET"
               valueFrom:
                 secretKeyRef:
                   name: {{ template "mobu.fullname" . }}-secret
                   key: "github-refresh-app-webhook-secret"
             {{- end}}
             {{- if .Values.config.githubCiApp }}
-            - name: "MOBU_GITHUB_CI_APP_CONFIG_PATH"
-              value: "/etc/mobu/github-ci-app.yaml"
             - name: "MOBU_GITHUB_CI_APP_ID"
               valueFrom:
                 secretKeyRef:
@@ -70,17 +64,6 @@ spec:
                   name: {{ template "mobu.fullname" . }}-secret
                   key: "github-ci-app-webhook-secret"
             {{- end}}
-            - name: "MOBU_PATH_PREFIX"
-              value: {{ .Values.config.pathPrefix | quote }}
-            {{- if .Values.config.debug }}
-            - name: "MOBU_LOGGING_PROFILE"
-              value: "development"
-            - name: "MOBU_LOG_LEVEL"
-              value: "DEBUG"
-            {{- else }}
-            - name: "MOBU_LOGGING_PROFILE"
-              value: "production"
-            {{- end }}
           image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
           imagePullPolicy: {{ .Values.image.pullPolicy | quote }}
           ports:
@@ -114,29 +97,8 @@ spec:
         runAsGroup: 1000
       volumes:
         - name: "config"
-          projected:
-            sources:
-            {{- if .Values.config.autostart }}
-            - configMap:
-                name: {{ include "mobu.fullname" . }}-autostart
-                items:
-                  - key: autostart.yaml
-                    path: autostart.yaml
-            {{- end }}
-            {{- if .Values.config.githubCiApp }}
-            - configMap:
-                name: {{ include "mobu.fullname" . }}-github-ci-app
-                items:
-                  - key: github-ci-app.yaml
-                    path: github-ci-app.yaml
-            {{- end }}
-            {{- if .Values.config.githubRefreshApp }}
-            - configMap:
-                name: {{ include "mobu.fullname" . }}-github-refresh-app
-                items:
-                  - key: github-refresh-app.yaml
-                    path: github-refresh-app.yaml
-            {{- end }}
+          configMap:
+            name: "mobu-config"
         - name: "tmp"
           emptyDir: {}
       {{- with .Values.nodeSelector }}

applications/mobu/values-idfdemo.yaml

@@ -1,3 +1,4 @@
 config:
-  debug: true
+  logLevel: DEBUG
+  profile: development
   autostart: []

applications/mobu/values-idfdev.yaml

@@ -2,7 +2,8 @@ image:
   tag: "tickets-DM-44145-app-metrics"
   pullPolicy: "Always"
 config:
-  debug: true
+  logLevel: DEBUG
+  profile: development
   githubRefreshApp:
     acceptedGithubOrgs:
       - lsst-sqre

applications/mobu/values-roundtable-dev.yaml

@@ -1,5 +1,6 @@
 config:
-  debug: true
+  logLevel: DEBUG
+  profile: development
   autostart:
     - name: "gitlfs"
       count: 1

applications/mobu/values-usdfdev.yaml

@@ -1,5 +1,6 @@
 config:
-  debug: true
+  logLevel: DEBUG
+  profile: development
   githubRefreshApp:
     acceptedGithubOrgs:
       - lsst-sqre

applications/mobu/values-usdfint.yaml

@@ -1,5 +1,4 @@
 config:
-  debug: true
   githubRefreshApp:
     acceptedGithubOrgs:
       - lsst-sqre

applications/mobu/values-usdfprod.yaml

@@ -1,5 +1,4 @@
 config:
-  debug: true
   autostart:
     - name: "firefighter"
       count: 1

applications/mobu/values.yaml

@@ -29,17 +29,22 @@ config:
   # -- Configuration for the GitHub refresh app integration.
   # See https://mobu.lsst.io/operations/github_refresh_app.html#add-phalanx-configuration
   # @default -- disabled.
-  githubRefreshApp: {}
+  githubRefreshApp: null
 
   # -- Configuration for the GitHub CI app integration.
   # -- Configuration for the GitHub refresh app integration.
   # See https://mobu.lsst.io/operations/github_ci_app.html#add-phalanx-configuration
   # @default -- disabled.
-  githubCiApp: {}
+  githubCiApp: null
 
-  # -- If set to true, include the output from all flocks in the main mobu log
-  # and disable structured JSON logging.
-  debug: false
+  # -- Log level. Set to 'DEBUG' to include the output from all flocks in the
+  # main mobu log.
+  logLevel: INFO
+
+  # -- One of 'production' or 'development'. 'production' configures structured
+  # JSON logging, and 'development' configures unstructured human readable
+  # logging.
+  profile: production
 
   # -- Whether to send alerts and status to Slack.
   slackAlerts: true