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

Deprecate zpages #4

Open
cijothomas opened this issue Aug 16, 2023 · 4 comments
Open

Deprecate zpages #4

cijothomas opened this issue Aug 16, 2023 · 4 comments
Assignees

Comments

@cijothomas
Copy link
Member

Other languages have started deprecating zpages. The spec for it was experimental and has no plan to move to stable spec.
The total download count is <500 all time.

open-telemetry/opentelemetry-specification#3577 Java
open-telemetry/opentelemetry-specification#3564 .NET

@cijothomas
Copy link
Member Author

open-telemetry/opentelemetry-rust#1199 is disabling Zpages from CI check to unblock that PR. This issue is opened to track proper removal, along with deprecation notice in readme/crates.io etc

@TommyCpp
Copy link
Contributor

Golang maintains a zpage implementation so I think there may still be interest it in the future. Not a priority now but I am hesitate to delete it

@djc
Copy link
Contributor

djc commented Aug 17, 2023

If there is interest in the future, it will be easy enough to resurrect from Git -- if no one really wants to spend the effort today to maintain it, we should just drop it. Once a new version of the opentelemetry crates comes out without a new version of the opentelemetry-zpages crate and someone notice, we'll know if someone wants it (enough to be willing to put in some work).

@hdost
Copy link
Contributor

hdost commented Aug 17, 2023

The reason that we were trying to disable to build for this moment is that we have to fix zpages before the build will complete due to some library changes.
I suppose we can put the effort into:

  1. Fixing the build
  2. Add the deprecation/removal notice along with a note of if someone wants to maintain it to reach out in the project.
  3. Release the fixed version with the note
  4. Remove zpages from the repository with CHANGELOG note of the removal.
    That should be enough to close the ticket.

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

4 participants