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

Properly communicate the stability for OTLP #1147

Closed
carlosalberto opened this issue Oct 27, 2020 · 4 comments
Closed

Properly communicate the stability for OTLP #1147

carlosalberto opened this issue Oct 27, 2020 · 4 comments
Labels
area:versioning Related to specification versioning priority:p2 Medium priority level release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs spec:protocol Related to the specification/protocol directory

Comments

@carlosalberto
Copy link
Contributor

Based on feedback during the SIG call today, we need to be even more explicit about OTLP/protos still being in (relative) flux, and the fact it shouldn't be used in production systems these days. See https://github.com/open-telemetry/opentelemetry-proto#maturity-level - Also, there's the need to add Logs to this table, as either Experimental or Alpha.

Specially important as we are approaching Kubecon and we need to make things the community expectations are the proper ones ;)

cc @tigrannajaryan

@carlosalberto carlosalberto added spec:protocol Related to the specification/protocol directory priority:p2 Medium priority level release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs labels Oct 27, 2020
@andrewhsu andrewhsu added the area:versioning Related to specification versioning label Oct 27, 2020
@andrewhsu
Copy link
Member

Just wanted to make this point: it'd be good to have OTLP used in production with the understanding that at this point some portions of the proto may change drastically between releases, like logs, and some portions don't have an intention of changing and have been hardened a lot more, like trace. Feedback from the user from running this in production is valuable in both cases but different expectations ought to be set for different parts of the protocol.

From the conversation that evolved from the spec sig mtg today, perhaps documentation or blog posts may help in setting this expectation.

@jsuereth
Copy link
Contributor

I want to call out two things I think may send mixed signals:

  1. The fact that everything is called "v1" in the proto repository: https://github.com/open-telemetry/opentelemetry-proto
    • While the readme in the repository outlines the expected compatibility level of the protocol, it's possible someone is consuming those protos without having to read the readme and sees the "v1" in the type.
    • I understand there are legacy reasons why this was done. I think we could take action on non-GA items within the protocol, e.g. when GA release "v1", then we should not include items that are not stable.
  2. Communication around adoption of OpenTelemetry and usage of the collector.
    • If people find OTLP configuration in releases of the open telemetry collector, or language SDKs, they may not read the warnings that have been placed around them by the OT project itself.
    • When advertising the collector in blog posts and conference talks, we should be careful to ALWAYS denote that some components are not stable, but others are so folks know to check before assuming. Announcements on readiness and calls to try something out tend to imply a level of maturity we may not be adhering too.

@tigrannajaryan
Copy link
Member

Added a clarification for OTLP Logs: open-telemetry/opentelemetry-proto#228

bogdandrutu pushed a commit to open-telemetry/opentelemetry-collector that referenced this issue Oct 28, 2020
@tigrannajaryan
Copy link
Member

We now have stability labels in both proto repo and in this repo in the "protocol" directory. I believe this issue is resolved. If any additional communication is needed please re-open or create a new issue that describes what else is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:versioning Related to specification versioning priority:p2 Medium priority level release:allowed-for-ga Editorial changes that can still be added before GA since they don't require action by SIGs spec:protocol Related to the specification/protocol directory
Projects
None yet
Development

No branches or pull requests

4 participants