-
Notifications
You must be signed in to change notification settings - Fork 888
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
Proposal: move OTLP spec to github.com/open-telemetry/opentelemetry-proto #3415
Comments
@open-telemetry/specs-approvers please comment. |
+1 |
What can we do to keep the OTLP specification documents discoverable? Its definitely easier to find the OTLP specification than it is the OpAmp specification for a casual user. Can we keep a placeholder in the specification repo which links to the proto repository? |
Yes, that's what I want to do. I will not delete the docs from spec repo. I will keep the docs, but will replace the content with links to the new location. This will also ensure any existing external links to the docs will not become 404 after the move. |
Since the OTLP spec will be published via the OTel website, the options of which are discussed in open-telemetry/opentelemetry.io#2642 (comment), I'd propose that discoverability be via the website, and that we avoid keeping a copy of the pages (even if only redirect links), in the OTel spec repo. WDYT? |
I would still prefer to keep the spec repo files (with content deleted and just referring the reader to the new location). I have seen countless examples of people linking directly to files in spec repo. These links will all become 404 if we delete the files. |
+1 |
…oto (#3454) Resolves #3415 Depends on open-telemetry/opentelemetry-proto#458 Depends on open-telemetry/opentelemetry.io#2642
The protocol specification is currently split between this repo which contains the verbal description of top-level message exchanges and https://github.com/open-telemetry/opentelemetry-proto which contains the Protobuf definitions, including verbal description of many fields.
This split is unnecessary and makes reading and understanding the entire OTLP protocol more difficult than it needs to be.
I suggest to move the following files to https://github.com/open-telemetry/opentelemetry-proto:
We have a similar co-location of spec text and proto files for OpAMP and it worked well for the workgroup.
The text was updated successfully, but these errors were encountered: