Skip to content

chore(deps): Update otel/metric to stable API#2607

Merged
vmihailenco merged 1 commit into
redis:masterfrom
ash2k:update-otel
May 24, 2023
Merged

chore(deps): Update otel/metric to stable API#2607
vmihailenco merged 1 commit into
redis:masterfrom
ash2k:update-otel

Conversation

@ash2k

@ash2k ash2k commented May 23, 2023

Copy link
Copy Markdown
Contributor

Pick up https://github.com/open-telemetry/opentelemetry-go/releases/tag/v1.16.0.

The example also needs to be updated, but uptrace-go needs to be updated first.

Closes #2578.

@vmihailenco

vmihailenco commented May 23, 2023

Copy link
Copy Markdown
Collaborator

Thanks! uptrace-go update is blocked by open-telemetry/opentelemetry-go-contrib#3874

@pellared

pellared commented May 23, 2023

Copy link
Copy Markdown

Are you not using uptrace-go only in OTel example?

Personally, I would suggest moving the example here: https://github.com/uptrace/uptrace-go/tree/master/example and change the existing example to use only the official "vanilla" OTel Go components.

PS. I am not a maintainer here. I just noticed this PR via open-telemetry/opentelemetry-go-contrib#3874 and I hope that this idea may help you. Feel free to ignore my comment for whatever reason 😉

@vmihailenco

Copy link
Copy Markdown
Collaborator

@ash2k uptrace-go is updated - could you update this PR?

Personally, I would suggest moving the example here

We had lots of examples in uptrace-go, but it was hard to update them and it resulted in a lot of noise in the repo for uptrace-go users.

Besides, the example here is in a separate module so it does not affect the redis client.

@ash2k

ash2k commented May 24, 2023

Copy link
Copy Markdown
Contributor Author

@vmihailenco PTAL

@vmihailenco vmihailenco merged commit 9a9423d into redis:master May 24, 2023
@ash2k ash2k deleted the update-otel branch May 24, 2023 10:40
@ash2k

ash2k commented May 24, 2023

Copy link
Copy Markdown
Contributor Author

@vmihailenco Any plans to release a new version soon? :)

@vmihailenco

Copy link
Copy Markdown
Collaborator

@ash2k #2608

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.

3 participants