Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

feat: Run all Django services with ddtrace (Datadog APM) when enabled #7147

Merged
merged 2 commits into from
Apr 5, 2024

Conversation

timmc-edx
Copy link
Contributor

@timmc-edx timmc-edx commented Apr 3, 2024

This expands the changes in edxapp to other Django services.

Also enables Datadog APM in sandboxes when Datadog is enabled overall.

See edx/edx-arch-experiments#573


Configuration Pull Request

Make sure that the following steps are done before merging:

  • Have a Site Reliability Engineer review the PR if you don't own all of the services impacted.
  • If you are adding any new default values that need to be overridden when this change goes live, update internal repos and add an entry to the top of the CHANGELOG.
  • Performed the appropriate testing.
  • Think about how this change will affect Open edX operators and update the wiki page for the next Open edX release if needed

@timmc-edx timmc-edx merged commit 7341100 into master Apr 5, 2024
4 checks passed
@timmc-edx timmc-edx deleted the timmc/dd-apm-generic branch April 5, 2024 13:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants