-
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
DEPR and move code owner monitoring code with updates #784
Comments
Giving this a P3 because it is important, but it is not urgent. |
robrap
added a commit
that referenced
this issue
Oct 23, 2024
Initial rollout of moving code_owner monitoring code from edx-django-utils to this plugin. - Adds near duplicate of code owner middleware from edx-django-utils. - Adds code owner for celery using Datadog span processing of celery.run spans. - Uses temporary span tags names using `_2`, like `code_owner_2`, for rollout and comparison with the original span tags. See #784
7 tasks
Presumably this was marked blocked while I was out. Marking in-progress again. |
robrap
added a commit
that referenced
this issue
Nov 14, 2024
Initial rollout of moving code_owner monitoring code from edx-django-utils to this plugin. - Adds near duplicate of code owner middleware from edx-django-utils. - Adds code owner for celery using Datadog span processing of celery.run spans. - Uses temporary span tags names using `_2`, like `code_owner_2`, for rollout and comparison with the original span tags. See #784
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
code_owner
andcode_owner_squad
would have the same value, andcode_owner_theme
would be dropped. In theory, we could also dropcode_owner_squad
, but that might force too much unnecessary work on other teams.Implementation thoughts:
code_owner
, but don't get our new test tag.ACs:
The text was updated successfully, but these errors were encountered: