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

feat: OTel tracing media ops initial instrumentation #1280

Merged
merged 3 commits into from
May 30, 2024

Conversation

cojenco
Copy link
Contributor

@cojenco cojenco commented May 23, 2024

No description provided.

@cojenco cojenco requested review from a team as code owners May 23, 2024 20:59
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label May 23, 2024
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/python-storage API. label May 23, 2024
@cojenco cojenco assigned cojenco and unassigned coryan May 23, 2024
@cojenco cojenco merged commit f5c7eee into googleapis:otel-trace May 30, 2024
7 checks passed
cojenco added a commit that referenced this pull request Jun 26, 2024
* feat: introduce OpenTelemetry Tracing decorators (#1257)

* feat: introduce OpenTelemetry Tracing decorators

* update test coverage

* add tests, update fixture

* update noxfile, extras; remove print

* update unit test

* review comments

* feat: instrument metadata ops with OTel tracing (#2) (#1267)

* feat: instrument metadata ops with Otel tracing

* update README plus test

* update decorator name per review session

* fix typo in readme

* feat: OTel tracing media ops initial instrumentation (#1280)

* feat: OTel tracing media ops initial instrumentation

* use download class name as span name

* avoid asserting filtered warnings from otel per open-telemetry/opentelemetry-python#3164

* add copyright and preview note

* comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/python-storage API. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants