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

Change synapse server operation name to match other servers #3566

Merged
merged 2 commits into from
May 24, 2022

Conversation

devinsba
Copy link
Contributor

@devinsba devinsba commented May 23, 2022

What Does This Do

Changes the synapse server operation name to match the patterns for other servers. Adds a config to revert to the previous behavior.

Set dd.integration.synapse.legacy-operation-name=true to get back the previous behavior

@devinsba devinsba added inst: others All other instrumentations tag: breaking change Breaking changes labels May 23, 2022
@devinsba devinsba requested a review from a team as a code owner May 23, 2022 15:01
@devinsba devinsba force-pushed the bds/synapse-operation-name branch 4 times, most recently from b7c5c65 to f15ceab Compare May 23, 2022 17:48
@devinsba devinsba force-pushed the bds/synapse-operation-name branch from f15ceab to de81985 Compare May 23, 2022 18:28
@devinsba devinsba merged commit 7721cf8 into master May 24, 2022
@devinsba devinsba deleted the bds/synapse-operation-name branch May 24, 2022 15:01
@github-actions github-actions bot added this to the 0.102.0 milestone May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inst: others All other instrumentations tag: breaking change Breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants