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

OpenTelemetry and Correlation Context #16

Closed
mwear opened this issue Feb 11, 2020 · 8 comments
Closed

OpenTelemetry and Correlation Context #16

mwear opened this issue Feb 11, 2020 · 8 comments

Comments

@mwear
Copy link
Member

mwear commented Feb 11, 2020

OpenTelemetry and Correlation Context

OpenTelemetry is proposing to propagate correlations according to the editor's draft of the correlation context spec, however, it is still in its early stages. As we discussed at today's meeting, we need to come up with a plan that sets this spec up for success and mitigates any potential challenges from early adoption. I'm creating this issue for us to collect ideas and discuss our options.

Two things we can consider are:

  • Using a different header name for OpenTelemetry (temporarily)
  • Versioning the header

Different header name

We could use a header such as otcorrelationcontext which would give us the freedom to make breaking changes to the spec without considerations for backwards compatibility. The upgrade path for OpenTelemtry would have to be intentional and would require introducing a suitable propagator for correlation context once the spec has been finalized. There is always the risk OpenTelemetry and the W3C spec could diverge and we would be left with multiple formats indefinitely. Even if the projects are aligned, we might be left with both formats regardless.

Versioning the header

If we can agree on a header name and a versioning scheme that will work with our future header representations, we could potentially implement and improve the spec incrementally and maintain backwards compatibility. The main challenges would be deciding on a versioning scheme that will work with the header defined in the editor's draft and how we intend to represent it in the future, such as using an IETF dictionary. Versioning is a challenge we are going to have to face as we finalize the spec either way, so maybe now is as good a time as any to discuss it?

Other options

I'm sure some exist. Please add them to this issue.

@mwear
Copy link
Member Author

mwear commented Feb 11, 2020

Option 3 - Use IETF Dictionary now

We could discuss going all in on the IETF dictionary (see #15) and update the editor's draft to use it now. That way we wouldn't have to change formats. Embedding the version in the dictionary could be a possibility. There would be some work to do in OpenTelemetry to support it, but possibly worth it. An upside is that it could make the spec process go quickly and smoothly.

@codefromthecrypt
Copy link

why is opentelemetry making issues in this repo? isn't open telemetry a separate project from w3c specification?

@danielkhan
Copy link
Contributor

OTel ist just one of the first implementors and they have a need for correlation context and we are trying to suggest something that won't break in the future.

@codefromthecrypt
Copy link

just seems some troubling left hand here and right hand there, where w3c efforts have been conflated with open-telemetry in chat and issues lists.

Maybe consider putting this issue in otel and inviting the folks who are not already there to it? It would be welcome to see some amount of independence between these two things.

@dyladan
Copy link
Member

dyladan commented Feb 26, 2020

There is an issue in the opentelemetry spec. I think this issue is more along the lines of "what can we do in order to support otel as quickly as possible" rather than "what should otel do".

@codefromthecrypt
Copy link

codefromthecrypt commented Feb 26, 2020 via email

@SergeyKanzhelev
Copy link
Member

I believe w3c distributed tracing group cannot make any commitments for open telemetry. We need to follow the standard maturity levels and gather feedback from various parties. I think the point of this issue is that it will be great if we can put more energy into this spec. If this is it, let's close it?

@danielkhan
Copy link
Contributor

I think this is safe to close. We agreed on baggage meanwhile.

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

No branches or pull requests

5 participants