Skip to content

Commit

Permalink
Add setting for OTEL_EXPORTER_OTLP_ENDPOINT to proxy
Browse files Browse the repository at this point in the history
OpenTelemetry tracing support was added to proxy in
neondatabase/neon#3458
  • Loading branch information
hlinnaka committed Apr 24, 2023
1 parent 45b4de3 commit 4bc788f
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
1 change: 1 addition & 0 deletions charts/neon-proxy/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ Kubernetes: `^1.18.x-x`
| settings.extraDomains | list | `[]` | domains used in extra TLS certs for client postgres connections |
| settings.metricCollectionEndpoint | string | `""` | (url) endpoint used to send metrics to. If null, metrics will not be sent. |
| settings.metricCollectionInterval | string | `""` | (string) how often metrics should be sent. |
| settings.otelExporterOtlpEndpoint | string | `""` | OpenTelemetry collector URL (will be converted into `OTEL_EXPORTER_OTLP_ENDPOINT` environment variable) |
| settings.sentryEnvironment | string | `"development"` | "development" or "production". It will be visible in sentry in order to filter issues |
| settings.sentryUrl | string | `""` | url (will be converted into `SENTRY_DSN` environment variable) used by sentry to collect error/panic events in neon-proxy |
| settings.uri | string | `""` | |
Expand Down
4 changes: 4 additions & 0 deletions charts/neon-proxy/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,10 @@ spec:
{{- end }}
{{- if .Values.settings }}
env:
{{- with .Values.settings.otelExporterOtlpEndpoint }}
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: {{ . }}
{{- end }}
{{- with .Values.settings.sentryUrl }}
- name: SENTRY_DSN
value: {{ . }}
Expand Down
2 changes: 2 additions & 0 deletions charts/neon-proxy/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ settings:
domain: ""
# settings.extraDomains -- domains used in extra TLS certs for client postgres connections
extraDomains: []
# settings.otelExporterOtlpEndpoint -- OpenTelemetry collector URL (will be converted into `OTEL_EXPORTER_OTLP_ENDPOINT` environment variable)
otelExporterOtlpEndpoint: ""
# settings.sentryUrl -- url (will be converted into `SENTRY_DSN` environment variable) used by sentry to collect error/panic events in neon-proxy
sentryUrl: ""
# settings.sentryEnvironment -- "development" or "production". It will be visible in sentry in order to filter issues
Expand Down

0 comments on commit 4bc788f

Please sign in to comment.