Skip to content

[bp/1.28] datadog: fix span error property#31704

Merged
phlax merged 1 commit into
envoyproxy:release/v1.28from
DataDog:david.goffredo/backport-v1.28-datadog-fix-span-error-property
Jan 9, 2024
Merged

[bp/1.28] datadog: fix span error property#31704
phlax merged 1 commit into
envoyproxy:release/v1.28from
DataDog:david.goffredo/backport-v1.28-datadog-fix-span-error-property

Conversation

@dgoffredo
Copy link
Copy Markdown
Contributor

This is a backport of #31669 onto the v1.28 release branch.

These changes fix a bug in the Datadog tracing extension. In releases before v1.27, when the "error" tag was set on a Datadog span, a corresponding private property on the span would also be set to indicate the presence of an error. Release v1.27 included a rewrite of the Datadog tracing extension that removed this behavior, and the removal is a bug. These changes restore the old behavior.

@phlax phlax self-assigned this Jan 8, 2024
@dgoffredo dgoffredo force-pushed the david.goffredo/backport-v1.28-datadog-fix-span-error-property branch from 6ceb379 to e6b7ec5 Compare January 9, 2024 13:14
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks @dgoffredo

@phlax phlax enabled auto-merge (rebase) January 9, 2024 13:17
@dgoffredo
Copy link
Copy Markdown
Contributor Author

Thanks, @phlax. You've been very helpful getting all of these Datadog bugs patched.

@phlax
Copy link
Copy Markdown
Member

phlax commented Jan 9, 2024

you might need to do a commit --amend to kick this one it seems to have totally fallen through the github cracks

Signed-off-by: David Goffredo <david.goffredo@datadoghq.com>
auto-merge was automatically disabled January 9, 2024 14:11

Head branch was pushed to by a user without write access

@dgoffredo dgoffredo force-pushed the david.goffredo/backport-v1.28-datadog-fix-span-error-property branch from e6b7ec5 to 735dc21 Compare January 9, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants