Skip to content

(new) Adapt common utilities to OTel specifications.#552

Merged
GSVarsha merged 7 commits into
otel_migrationfrom
new_otel_utilities
Jul 8, 2024
Merged

(new) Adapt common utilities to OTel specifications.#552
GSVarsha merged 7 commits into
otel_migrationfrom
new_otel_utilities

Conversation

@pvital

@pvital pvital commented Jul 8, 2024

Copy link
Copy Markdown
Member

Based on changes made on #550.

Adapt the following to OTel specifications, and fix some rebasing issues:

  • inject()
  • extract()
  • traceparent
  • tracestate
  • get_current_span()
  • get_active_tracer()
  • get_tracer_tuple()

Signed-off-by: Varsha GS varsha.gs@ibm.com
Co-authored-by: Paulo Vital paulo.vital@ibm.com

GSVarsha and others added 3 commits July 8, 2024 08:58
… OTel

Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Co-authored-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Co-authored-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
@pvital pvital changed the base branch from master to otel_migration July 8, 2024 07:12
pvital added 2 commits July 8, 2024 09:15
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital force-pushed the new_otel_utilities branch from f0ac61e to 3b112e8 Compare July 8, 2024 07:16
pvital added 2 commits July 8, 2024 09:21
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
Signed-off-by: Paulo Vital <paulo.vital@ibm.com>
@pvital pvital force-pushed the new_otel_utilities branch from 3b112e8 to b3b1484 Compare July 8, 2024 07:21
@pvital pvital requested a review from GSVarsha July 8, 2024 07:22
@pvital pvital added OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry Review & Merge labels Jul 8, 2024
@pvital pvital marked this pull request as ready for review July 8, 2024 07:36

@GSVarsha GSVarsha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me!

@GSVarsha GSVarsha merged commit a01cac8 into otel_migration Jul 8, 2024
@GSVarsha GSVarsha deleted the new_otel_utilities branch July 8, 2024 08:11
@GSVarsha GSVarsha added this to the H2-2024 milestone Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry Review & Merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants