OpenTracing Shim GA

[carlosalberto]

This is an issue to track the upcoming OpenTracing GA release:

• OpenTracing Shim: Propagate baggage from multiple parents #4906
• OpenTracing Shim: All parents must be recorded as links #4907
• OpenTracing Shim: Support setting ALL tag/attribute types #4908
• OpenTracing Shim: Align Shim Layer creation arguments #4909
• OpenTracing Shim: Proper handle tag/attribute ERROR #4910
• OpenTracing Shim: Do not perform semantic conventions mapping #5004
• OpenTracing Shim: Ignore invalid arguments #5005
• OpenTracing Shim: Properly implement Tracer.Close() #5081
• Verify OT shim context propagation is functioning properly. #2182
• TracerShim#extract() returning current context instead of null in case of empty carrier #4344

(Will be updating it with additional details as we go)

[oppegard]

@carlosalberto, just checking in on the GA status of this. We're in the process of adopting the shim, but bringing in a dependency with alpha status will raise some concerns during code review.

[carlosalberto]

hey @oppegard, thanks for inquiring. So I'm doing a final pass on the Spec/Shim to make sure all looks right. So far a pending item is open-telemetry/opentelemetry-specification#3048 is the only standing item (once accepted, I will implement the update for the Java Shim).

More broadly speaking, I'd expect to have all ready by end of January at max, so the February release includes a stable artifact ;)

[oppegard]

Thanks for the good news @carlosalberto!

[jack-berg]

@open-telemetry/java-approvers / @carlosalberto what do you say we mark the opentracing shim as stable?

[carlosalberto]

@jack-berg Thanks for reaching out. We are mostly there, but I'd ask a pair of more days in order for me to do a full review of the current code base in case there's anything missing (I know we can fix/tune things later if/as needed, but still). Tomorrow EOD I should have a proper answer (hopefully with PRs if needed).

[jack-berg]

No problem. The next release is on 3/10. Let's (minimally) wait until the following April release.

[jack-berg]

Added #2182 to ensure we don't loose track of it.

[jack-berg]

Added #4344 for tracking.

[formanojhr]

Thanks for the good news @carlosalberto!

@carlosalberto @jack-berg Just following up on my colleague @oppegard 's request for update on open tracing shim ready for GA early this year(2023). Is this something we can use in prod? I understand that there can be a few issue but overall is it in a stable state for java language?

[jack-berg]

Closing because the opentracing shinm was marked as stable in #5371. The unchecked boxes in this tracking issue continue to be open, but the shim is stable and "GA".