Uh oh!
There was an error while loading. Please reload this page.
Unified naming for span ops - #1661
Conversation
span op changes don't break anything but transaction op changes are breaking in terms of existing dashboards. |
sl0thentr0py
commented
Oct 5, 2022
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? |
antonpirker
commented
Oct 5, 2022
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
antonpirker
commented
Oct 5, 2022
Note to self: document in the readme all the changes to span ops for everyone that has dashboards it could be breaking. |
antonpirker
commented
Oct 5, 2022
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 |
antonpirker
commented
Oct 6, 2022
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