Skip to content

Enable Temporal SDK metrics#72

Merged
revied merged 2 commits intoblock:mainfrom
revied:drevie/enable-temporal-sdk-metrics
Apr 30, 2026
Merged

Enable Temporal SDK metrics#72
revied merged 2 commits intoblock:mainfrom
revied:drevie/enable-temporal-sdk-metrics

Conversation

@revied
Copy link
Copy Markdown
Contributor

@revied revied commented Apr 30, 2026

Summary

  • enable the Temporal Go SDK metrics handler with the official Tally bridge
  • expose SDK Prometheus/OpenMetrics on /metrics at port 9090 by default
  • wire the metrics endpoint through env vars, Docker Compose, and the Helm chart
  • document the Temporal SDK metrics endpoint and useful SDK metric names

Notes

This intentionally does not add Version Guard custom metrics. Datadog ingestion still depends on the deployment/platform scrape configuration for the exposed :9090/metrics endpoint.

Test plan

  • go test ./cmd/server
  • go test ./...
  • make lint
  • make test
  • make build-all
  • helm template version-guard charts/version-guard
  • helm lint charts/version-guard

@revied revied requested a review from a team as a code owner April 30, 2026 20:21
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@revied revied force-pushed the drevie/enable-temporal-sdk-metrics branch from 1e91cf6 to 7309cda Compare April 30, 2026 20:43
bakayolo
bakayolo previously approved these changes Apr 30, 2026
@revied revied merged commit a9cbc60 into block:main Apr 30, 2026
8 checks passed
@revied revied deleted the drevie/enable-temporal-sdk-metrics branch April 30, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants