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.

Sign up for GitHub

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

Jump to bottom

ZPAGES no longer supported in opentelemetry-cpp #3786

Closed
marcalff opened this issue Dec 8, 2023 · 3 comments · Fixed by #3789
Closed

ZPAGES no longer supported in opentelemetry-cpp #3786

marcalff opened this issue Dec 8, 2023 · 3 comments · Fixed by #3789
Assignees
@arminru
Labels
[label deprecated] triaged-accepted [label deprecated] Issue triaged and accepted by OTel community, can proceed with creating a PR spec:miscellaneous For issues that don't match any other spec label

Comments

@marcalff
Copy link
Member

marcalff commented Dec 8, 2023

opentelemetry-cpp removed the ZPages implementation:

As a result, the file experimental/trace/zpages.md needs to be updated, as it contains:

Within OpenTelemetry, zPages are being developed in C++.
@marcalff marcalff added the spec:miscellaneous For issues that don't match any other spec label label Dec 8, 2023
@marcalff marcalff mentioned this issue Dec 8, 2023
Closed
@tigrannajaryan
Copy link
Member

Do any other SDKs implement zpages? I don't think zpages got much momentum. If that is the case we should consider removing them altogether from the spec.

@MrAlias
Copy link
Contributor

MrAlias commented Dec 8, 2023

We have it implemented in Go: https://github.com/open-telemetry/opentelemetry-go-contrib/tree/main/zpages

@jack-berg jack-berg added the [label deprecated] triaged-accepted [label deprecated] Issue triaged and accepted by OTel community, can proceed with creating a PR label Dec 13, 2023
@jack-berg
Copy link
Member

@marcalff can you open a PR to remove the c++ reference from the zpages page?

@arminru arminru assigned arminru and unassigned tigrannajaryan Dec 13, 2023
@arminru arminru mentioned this issue Dec 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arminru arminru

Labels
[label deprecated] triaged-accepted [label deprecated] Issue triaged and accepted by OTel community, can proceed with creating a PR spec:miscellaneous For issues that don't match any other spec label
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update languages implementing zPages arminru/opentelemetry-specification
5 participants
@tigrannajaryan @MrAlias @arminru @marcalff @jack-berg