temporalio / samples-server

MIT License
37 stars 32 forks source link

fix: support DD_SITE env var #70

Closed sourcec0de closed 9 months ago

sourcec0de commented 9 months ago

What was changed

Added support for the DD_SITE environmental variable.

Why?

Not all users are part of the default datadog site. https://docs.datadoghq.com/getting_started/site/

  1. Closes https://github.com/temporalio/samples-server/issues/69

  2. How was this tested:

    • [ ] Building and deploying on our own infrastructure
    • [ ] WIll add a unit test to ensure the value is set on the context.
  3. Any docs updates needed?

    • [ ] update the readme to include details about the DD_SITE config option and add it to the helm chart.
CLAassistant commented 9 months ago

CLA assistant check
All committers have signed the CLA.