From 33c4d0f2c64de617570aec5c0b64ce1fb1dae42a Mon Sep 17 00:00:00 2001
From: Yash Mehrotra <yashmehrotra95@gmail.com>
Date: Mon, 20 May 2024 15:31:27 +0530
Subject: [PATCH] feat: use json logs in chart by default

---
 chart/templates/deployment.yaml | 3 +++
 chart/values.yaml               | 2 ++
 2 files changed, 5 insertions(+)

diff --git a/chart/templates/deployment.yaml b/chart/templates/deployment.yaml
index abeb6d42..9930998c 100644
--- a/chart/templates/deployment.yaml
+++ b/chart/templates/deployment.yaml
@@ -66,6 +66,9 @@ spec:
             - --disable-postgrest={{ .Values.disablePostgrest }}
             - --change-retention-days={{ .Values.configChangeRetentionDays }}
             - --analysis-retention-days={{ .Values.configAnalysisRetentionDays }}
+            {{- if .Values.jsonLogs }}
+            - --json-logs
+            {{- end }}
             {{- if .Values.db.runMigrations}}
             - --db-migrations
             {{- end}}
diff --git a/chart/values.yaml b/chart/values.yaml
index 92feb6b6..4cac7efb 100644
--- a/chart/values.yaml
+++ b/chart/values.yaml
@@ -26,6 +26,8 @@ image:
 configChangeRetentionDays: 60
 configAnalysisRetentionDays: 60
 
+jsonLogs: true
+
 # a list of configmaps to load scrape rules from, the configmap should have a single entry called "config.yaml"
 scrapeRuleConfigMaps:
   - config-db-rules