search

edx / edx-arch-experiments

A plugin to include applications under development by the architecture team at edx
GNU Affero General Public License v3.0
0 stars 3 forks source link

Automatically set code owner for celery tasks #729

Open robrap opened 2 months ago

robrap commented 2 months ago

This is a possible on-call improvement so that we don't need to add custom code to each celery task to get its code owner marked.

This was tried in https://github.com/openedx/edx-platform/issues/33179, but it didn't work for New Relic instrumentation. It may be that Datadog is using the signals, and this might possibly work.

Related, the community probably doesn't care about codeowner, but 2U doesn't yet have its own universal 2u-django-utils.

robrap commented 1 month ago

If this code is only used by 2U in a plugin, we could try the signal handler, but could also potentially use Datadog's span creation callbacks that we've used for other diagnostics.

Note: This is related to our ability to potentially DEPR all the codeowner code.