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

Update otel4s to 0.5.0-RC2, use otel4s-sdk in tests #1054

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

iRevive
Copy link
Contributor

@iRevive iRevive commented Feb 28, 2024

The otel4s-sdk-* modules provide a cross-platform implementation of the SDK API.

The traces seem consistent across platforms:

JVM

image

JS

image

private var ioTracerFinalizer: IO[Unit] = _

implicit lazy val ioTracer: Tracer[IO] = {
val tracerAndFinalizer = tracerResource.allocated.unsafeRunSync()
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We cannot use unsafeRunSync() on Native and Scala.js platforms. So the tests will be slightly more verbose.

@mpilquist
Copy link
Member

@iRevive Timing wise, is it worth preparing a separate PR (or building on #1053) that doesn't yet use the SDK? Or do you expect the SDK to be released soon enough after 0.5.0 final that we can do it all at once?

@iRevive
Copy link
Contributor Author

iRevive commented Mar 11, 2024

Hopefully, we will get a release this week. Here are the details: typelevel/otel4s#511 (comment).

@mpilquist
Copy link
Member

Awesome, thanks! +1 on including in 0.5.0 release.

@iRevive iRevive force-pushed the otel4s-sdk branch 2 times, most recently from 865c779 to 63c83a3 Compare March 14, 2024 11:00
@iRevive iRevive changed the title Use otel4s-sdk across all platforms in tests Update otel4s to 0.5.0-RC2, use otel4s-sdk in tests Mar 14, 2024
@iRevive iRevive marked this pull request as ready for review March 14, 2024 11:43
@iRevive
Copy link
Contributor Author

iRevive commented Mar 14, 2024

I updated otel4s to 0.5.0-RC2.

@mpilquist mpilquist merged commit 4f40fc9 into typelevel:main Mar 14, 2024
10 checks passed
@mpilquist
Copy link
Member

Awesome, thanks!!

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.

2 participants