fix: make monitoring optional based on license

README.md

@@ -64,6 +64,10 @@ To use IRSA make sure to NOT set the two key variables and add the appropriate r
               value: "arn:aws:iam::123456789:role/mattermost-role"
 ```
 
+### Monitoring
+
+Mattermost supports emitting metrics to Prometheus, but only if yo uhave a license. This package configures the necessary service monitor to enable metrics, but only when a license has been provided via the `MM_LICENSE` var. By default (no license), it does not provision the Service Monitor as it will show unhealthy because metrics is not enabled via the license.
+
 ## Flavors
 
 | Flavor | Description | Example Creation |

chart/templates/uds-package.yaml

@@ -45,12 +45,13 @@ spec:
         MM_EMAILSETTINGS_ENABLESIGNINWITHUSERNAME: "{{ .Values.sso.enable_sign_in_with_username | toString }}"
   {{- end }}
   monitor:
+  {{- if ne .Values.mattermostLicense "" }}
     - selector:
         app.kubernetes.io/name: mattermost-enterprise-edition
       targetPort: 8067
       portName: mattermost-app-metrics
       description: Metrics
-
+  {{- end }}
   network:
     expose:
       - service: mattermost-enterprise-edition

chart/values.yaml

@@ -26,7 +26,7 @@ postgres:
     cluster-name: pg-cluster
   namespace: postgres
   port: 5432
-
+mattermostLicense: ""
 sso:
   enabled: true
   # Options: "gitlab", "openid_connect"

values/config-values.yaml

@@ -6,3 +6,4 @@ postgres:
   password: "###ZARF_VAR_DB_PASSWORD###"
 
 subdomain: "###ZARF_VAR_SUBDOMAIN###"
+mattermostLicense: "###ZARF_VAR_MM_LICENSE###"

zarf.yaml

@@ -19,6 +19,9 @@ variables:
     description: "Secret Key for S3 compatible storage"
   - name: DB_PASSWORD
     description: "Database Password for Mattermost"
+  - name: MM_LICENSE
+    description: "License for Mattermost"
+    default: ""
 
 components:
   - name: mattermost