-
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
[APM] Remove direct references to newrelic #621
Comments
Maybe most of this should be about informing maintainers? |
Yeah. I've gone ahead and just notified each team; they can migrate on their own schedule. I think the only thing left is this |
Could we just document that as not being available for certain backends, in addition to your intended communications? |
It's already documented: https://github.com/openedx/edx-django-utils/tree/master/edx_django_utils/monitoring#choice-of-monitoring-tools I haven't gone through and checked if other such newrelic-specific functions are used in the codebase, though. |
Filed openedx/edx-django-utils#419 to cover the remaining NR-specific functions. |
We have some code in edx-platform (and possibly elsewhere) that directly imports and uses
newrelic
rather than going through edx-django-utils (e.g. in StaticContentServer). These all need to be switched over to edx-django-utils.A/C:
newrelic
references in codebase identifiedWorklist:
newrelic
refs in edxapp: feat: Remove direct New Relic references (use configured telemetry) openedx/edx-platform#34781import.*newrelic
ornewrelic.*import
:The text was updated successfully, but these errors were encountered: