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

fix(opentelemetry): fix a bug that will cause a failure of sending tracing data to datadog agent #11599

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

liverpool8056
Copy link
Contributor

Summary

fix a bug that will cause a failure of sending tracing data to datadog agent when the value of x-datadog-parent-id header from downstream is a short dec string.

In this PR, a formatter is added for ensuring the field of parent-span-id meet specification of the otel protocol.

Checklist

  • The Pull Request has tests
  • A changelog file has been added to CHANGELOG/unreleased/kong or adding skip-changelog label on PR if unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

  • [Implement ...]

Issue reference

FTI-5375

@github-actions github-actions bot added the author/community PRs from the open-source community (not Kong Inc) label Sep 19, 2023
tracing data to datadog agent when the value of `x-datadog-parent-id`
header from downstream is a short dec string.

In this PR, a formatter is added for ensuring the field of
`parent-span-id` meet specification of the otel protocol.

FTI-5375
@liverpool8056 liverpool8056 force-pushed the fix-opentelemetry-datadog-spanid branch from 57cbde0 to 54be856 Compare September 20, 2023 01:55
@samugi samugi merged commit e0d53a8 into master Sep 20, 2023
29 checks passed
@samugi samugi deleted the fix-opentelemetry-datadog-spanid branch September 20, 2023 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author/community PRs from the open-source community (not Kong Inc) plugins/opentelemetry size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants