Skip to content

Commit

Permalink
feat: use json logs in chart by default
Browse files Browse the repository at this point in the history
  • Loading branch information
yashmehrotra authored and moshloop committed May 20, 2024
1 parent 2534e1c commit 33c4d0f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
3 changes: 3 additions & 0 deletions chart/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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}}
Expand Down
2 changes: 2 additions & 0 deletions chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 33c4d0f

Please sign in to comment.