Skip to content

Commit

Permalink
Merge pull request #195 from FlowFuse/sentry_env
Browse files Browse the repository at this point in the history
Add SENTRY_ENV to deployment
  • Loading branch information
hardillb authored Oct 11, 2023
2 parents 66a2307 + 7995a32 commit 10a4b3d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
5 changes: 3 additions & 2 deletions helm/flowforge/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -81,9 +81,10 @@ Enables FlowForge Telemetry
- `forge.telemetry.posthog.apikey` enables posthog logging if set (not default)
- `forge.telemetry.posthog.apiurl` sets posthog target host (default `https://app.posthog.com`)
- `forge.telemetry.posthog.capture_pageview` (default `true`)
- `forge.telemetry.sentry.frontend_dsn` enables sentry reporting if set
- `forge.telemetry.sentry.backend_dsn` enables sentry reporting if set
- `forge.telemetry.sentry.frontend_dsn` enables sentry reporting if set (default unset)
- `forge.telemetry.sentry.backend_dsn` enables sentry reporting if set (default unset)
- `forge.telemetry.sentry.production_mode` rate limit reporting (default `true`)
- `forge.telemetry.sentry.environment` set SENTRY_ENV environment variable, which overrides NODE_ENV for the reported environment (default unset)
- `forge.telemetry.backend.prometheus.enabled` enables the `/metrics` endpoint on the forge app for scraping by Prometheus

### Support
Expand Down
4 changes: 4 additions & 0 deletions helm/flowforge/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@ spec:
- name: NODE_EXTRA_CA_CERTS
value: /usr/local/ssl-certs/chain.pem
{{- end }}
{{- if .Values.forge.telemetry.sentry.environment }}
- name: SENTRY_ENV
value: {{ .Values.forge.telemetry.sentry.environment }}
{{- end }}
volumeMounts:
- name: config
mountPath: /usr/src/forge/etc
Expand Down
3 changes: 3 additions & 0 deletions helm/flowforge/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,9 @@
},
"production_mode": {
"type": "boolean"
},
"environment": {
"type": "string"
}
}
},
Expand Down

0 comments on commit 10a4b3d

Please sign in to comment.