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: introduce OpenTelemetry Tracing decorators #1257

Merged
merged 6 commits into from
Apr 24, 2024

Conversation

cojenco
Copy link
Contributor

@cojenco cojenco commented Apr 12, 2024

part 1 introduce OTel tracing module and decorators

Fixes internal b/330754859

@cojenco cojenco requested review from a team as code owners April 12, 2024 00:06
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: storage Issues related to the googleapis/python-storage API. labels Apr 12, 2024
@cojenco cojenco added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 12, 2024
@cojenco cojenco removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 18, 2024
google/cloud/storage/_http.py Show resolved Hide resolved
google/cloud/storage/_opentelemetry_tracing.py Outdated Show resolved Hide resolved
google/cloud/storage/_opentelemetry_tracing.py Outdated Show resolved Hide resolved
@cojenco cojenco merged commit e6ed8e8 into googleapis:otel-trace Apr 24, 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: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants