Conversation
|
span op changes don't break anything but transaction op changes are breaking in terms of existing dashboards. |
|
also is the new consistent span ops table in notion or on develop already? If develop, can you please add the link to the PR description? |
|
Added link to develop doc to PR description. |
sl0thentr0py
left a comment
There was a problem hiding this comment.
changes look ok, pls don't merge till we decide on roll-out
|
Note to self: document in the readme all the changes to span ops for everyone that has dashboards it could be breaking. |
|
Maybe we can find out how many dashboards would be affected with this change. Possible ways to go: |
|
The definite guide of changed span/transaction |
|
Made a draft again so we do not accidentally merge it. Also related to this one #1665 |
…try/sentry-python into antonpirker/1643-unified-span-ops
Having a unified naming for span ops helps the performance team to discover performance issues.
The ops in the tests are all hard coded, so the tests show if someone trying to change ops string.
A list of all available span ops can be found in the develop docs:
https://develop.sentry.dev/sdk/performance/span-operations/
Fixes #1643