Skip to content
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

Automatically set code owner for celery tasks #729

Open
robrap opened this issue Jul 15, 2024 · 1 comment
Open

Automatically set code owner for celery tasks #729

robrap opened this issue Jul 15, 2024 · 1 comment
Labels

Comments

@robrap
Copy link
Contributor

robrap commented Jul 15, 2024

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 openedx/edx-platform#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 robrap added the on-call label Jul 15, 2024
@robrap
Copy link
Contributor Author

robrap commented Aug 28, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Backlog
Development

No branches or pull requests

1 participant