Skip to content

[OTel] PubSub & Cloud Storage instrumentation#623

Merged
pvital merged 4 commits into
otel_migrationfrom
otel/gcloud
Sep 30, 2024
Merged

[OTel] PubSub & Cloud Storage instrumentation#623
pvital merged 4 commits into
otel_migrationfrom
otel/gcloud

Conversation

@CagriYonca

Copy link
Copy Markdown
Contributor
  • Pubsub coverage: 90%
  • Cloud Storage coverage: 91%

@CagriYonca CagriYonca added the OTel_migration Migration the code dependency from OpenTracing to OpenTelemetry label Sep 20, 2024
@CagriYonca CagriYonca added this to the H2-2024 milestone Sep 20, 2024
@CagriYonca CagriYonca self-assigned this Sep 20, 2024
Comment thread src/instana/instrumentation/google/cloud/storage.py Outdated
Comment thread tests/clients/test_google-cloud-pubsub.py Outdated

@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.

A few minor requests. If there are other higher priority tasks, please feel free to make these changes later as well.

@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.

In addition to @GSVarsha's requests, I have only one that should be applied to multiple files.

Comment thread src/instana/instrumentation/google/cloud/pubsub.py Outdated
Comment thread src/instana/instrumentation/google/cloud/pubsub.py Outdated
@CagriYonca CagriYonca force-pushed the otel/gcloud branch 3 times, most recently from 4594648 to bfcb741 Compare September 27, 2024 11:41

@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.

LGTM!

@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.

@pvital pvital merged commit 5cd344c into otel_migration Sep 30, 2024
@pvital pvital deleted the otel/gcloud branch September 30, 2024 08:55
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants