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

[extension/zpages] Implement rpcz page or delete mentions #9328

Closed
atoulme opened this issue Jan 21, 2024 · 0 comments · Fixed by #9329
Closed

[extension/zpages] Implement rpcz page or delete mentions #9328

atoulme opened this issue Jan 21, 2024 · 0 comments · Fixed by #9329

Comments

@atoulme
Copy link
Contributor

atoulme commented Jan 21, 2024

Is your feature request related to a problem? Please describe.
The zpages extension README mentions /debug/rpcz page:

The Rpcz route is available to help examine statistics of remote procedure calls (RPCs) 
that are properly instrumented. For example when using gRPC

However, this page is not implemented.

Describe the solution you'd like
Remove the rpcz mention or draft a plan to implement it.

Describe alternatives you've considered
I have done a bit of search in git history but I have not found a mention of rcpz being implemented before, there's no mention in issues or pull requests either.
@atoulme atoulme changed the title [extension/zpages] Implement rpcz page [extension/zpages] Implement rpcz page or delete mentions Jan 21, 2024
@atoulme atoulme mentioned this issue Jan 21, 2024
Merged
bogdandrutu pushed a commit that referenced this issue Jan 22, 2024
@atoulme
[extension/zpages] remove mention of rpcz (#9329)
8a374e7 
**Description:**
Remove mention of rpcz from tests and README.

**Link to tracking Issue:**
Fixes #9328
TylerHelmuth pushed a commit to TylerHelmuth/opentelemetry-collector that referenced this issue Jan 23, 2024
@atoulme @TylerHelmuth
[extension/zpages] remove mention of rpcz (open-telemetry#9329)
d610150 
**Description:**
Remove mention of rpcz from tests and README.

**Link to tracking Issue:**
Fixes open-telemetry#9328
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[extension/zpages] remove mention of rpcz atoulme/opentelemetry-collector
1 participant
@atoulme