You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
During the New Relic work, we temporarily disabled spans for middleware. @timmc-edx mentioned he'd like to leave it this way.
AC:
Create ADR for disabling (or enabling) middleware instrumentation in edxapp.
Update comments around disabling, or ticket re-enabling, depending on decision.
Communicate the decision to other teams.
Notes/Questions:
Middleware instrumentation is currently disabled in edxapp, and likely enabled in other services, so we can still screenshot the differences.
What do we lose without the middleware spans when working on or debugging middleware issues or errors?
What are the workarounds if we leave this disabled? Could it makes sense documenting temporarily enabling in an environment if we are working on a middleware issue?
The text was updated successfully, but these errors were encountered:
I don't know that it needs to be assigned to me—I don't have strong feelings around it, and I think anyone can make the decision. I'll just link this from 692 for now.
Thanks @timmc-edx. I unassigned you. That said - I thought you had an opinion somewhere to leave this disabled. If you do have any thoughts, you can document them in a comment on this ticket.
During the New Relic work, we temporarily disabled spans for middleware. @timmc-edx mentioned he'd like to leave it this way.
AC:
Notes/Questions:
The text was updated successfully, but these errors were encountered: