Skip to content

[OTel] Tornado instrumentation#626

Merged
GSVarsha merged 4 commits into
otel_migrationfrom
otel-tornado-instrumentation
Sep 26, 2024
Merged

[OTel] Tornado instrumentation#626
GSVarsha merged 4 commits into
otel_migrationfrom
otel-tornado-instrumentation

Conversation

@GSVarsha

@GSVarsha GSVarsha commented Sep 24, 2024

Copy link
Copy Markdown
Contributor
coverage: 95%
  • tornado client
  • tornado server

@GSVarsha GSVarsha added Review & Merge currency Keeping supported versions updated. OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry labels Sep 24, 2024
@GSVarsha GSVarsha added this to the H2-2024 milestone Sep 24, 2024
@GSVarsha GSVarsha self-assigned this Sep 24, 2024

@pvital pvital left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It looks good to me.
Just one little minor request you can handle when rebasing with the latest changes. ;-)

Comment thread tests/requirements-310.txt Outdated
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
@GSVarsha GSVarsha force-pushed the otel-tornado-instrumentation branch from 507967e to 1a95036 Compare September 25, 2024 15:59
Signed-off-by: Varsha GS <varsha.gs@ibm.com>
@GSVarsha GSVarsha force-pushed the otel-tornado-instrumentation branch from 1a95036 to fc21cd3 Compare September 25, 2024 16:01
@GSVarsha GSVarsha merged commit 6ba6677 into otel_migration Sep 26, 2024
@GSVarsha GSVarsha deleted the otel-tornado-instrumentation branch September 26, 2024 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

currency Keeping supported versions updated. 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