Skip to content

Move instrumentation opentracing shim#109

Closed
NathanielRN wants to merge 15 commits intoopen-telemetry:masterfrom
NathanielRN:move-instrumentation-opentracing-shim
Closed

Move instrumentation opentracing shim#109
NathanielRN wants to merge 15 commits intoopen-telemetry:masterfrom
NathanielRN:move-instrumentation-opentracing-shim

Conversation

@NathanielRN
Copy link
Copy Markdown
Contributor

Description

Changes for package instrumentation/opentelemetry-instrumentation-opentracing-shim.

Adds remaining changes needed to get instrumentation packages to tag v0.15b0 at open-telemetry/opentelemetry-python@725655a of the Core Repo

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tests will be added in a future PR

Checklist:

  • Followed the style guidelines of this project
    - [ ] Changelogs have been updated
    - [ ] Unit tests have been added
    - [ ] Documentation has been updated

lzchen and others added 15 commits August 4, 2020 19:10
…#922)

This uses the OpenTelemetry context management mechanism to store a
ScopeShim object in order to make active return the same object as the
one returned by start_active_span.
Co-authored-by: Yusuke Tsutsumi <tsutsumi.yusuke@gmail.com>
* dropping support for python 3.4
…text (open-telemetry#1146)

Co-authored-by: Diego Hurtado <ocelotl@users.noreply.github.com>
@NathanielRN NathanielRN requested a review from a team November 2, 2020 19:21
Copy link
Copy Markdown
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

The OpenTracing bridge must remain in the core repo.

@NathanielRN NathanielRN closed this Nov 2, 2020
@NathanielRN NathanielRN deleted the move-instrumentation-opentracing-shim branch July 21, 2021 16:39
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.

6 participants