Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

fix: for open-telemetry status code spec change#362

Closed
vi3k6i5 wants to merge 14 commits intogoogleapis:masterfrom
vi3k6i5:ot_status_update_2
Closed

fix: for open-telemetry status code spec change#362
vi3k6i5 wants to merge 14 commits intogoogleapis:masterfrom
vi3k6i5:ot_status_update_2

Conversation

@vi3k6i5
Copy link
Copy Markdown
Contributor

@vi3k6i5 vi3k6i5 commented Jun 9, 2021

fix: for open-telemetry status code spec change

Fixes #357 🦕

Open telemetry has changed their specs for status code. Since the open telemetry library for python is still in beta it doesn't make much sense to use/support older version of that library.

Couple of reference:

'StatusCanonicalCode' is not part of 'opentelemetry.trace.status'
In memory exporter not resetting after test case ends

@vi3k6i5 vi3k6i5 requested a review from a team June 9, 2021 07:03
@product-auto-label product-auto-label bot added the api: spanner Issues related to the googleapis/python-spanner API. label Jun 9, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 9, 2021
@vi3k6i5
Copy link
Copy Markdown
Contributor Author

vi3k6i5 commented Jun 9, 2021

Duplicate of #358 So closing it.

@vi3k6i5 vi3k6i5 closed this Jun 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/python-spanner API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update open telemetry tracing code according to new spec.

1 participant