-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove newrelic from edx-platform dependencies #829
Comments
Maybe this whole ticket should just be replaced by a DEPR ticket? |
jristau1984
changed the title
Remove newrelic from edx-platform dependencies
Propose DEPR to remove newrelic from edx-platform dependencies
Oct 21, 2024
Tim confirmed there are several other repos with this reference in python code, but edx-platform is not in that list. Remove NR from the edx-platform requirements list. DEPR from django-utils can be done at a future time. |
jristau1984
changed the title
Propose DEPR to remove newrelic from edx-platform dependencies
Remove newrelic from edx-platform dependencies
Oct 21, 2024
Completed, and ticket updated to scope the work down. |
github-project-automation
bot
moved this from Ready For Development
to Done
in Arch-BOM
Oct 21, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
newrelic should not be listed as a dependency of edx-platform -- we rely on the abstraction layer in edx-django-utils (and that library also declares a dependency on it).
A/C:
newrelic
references in Python code in edx-platform or the things that are installed into it, even in try-import blocks.(Later we can look into DEPRing the
newrelic
dependency in edx-django-utils as well -- we should probably have extra-deps of edx-django-utils that pull in Datadog, New Relic, or OTel as needed, and which deployers can add to their additional package list.)The text was updated successfully, but these errors were encountered: