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

Add HTTP/2 HttpServer metric for active streams #2356

Closed
samueldlightfoot opened this issue Jul 5, 2022 · 2 comments · Fixed by #2357
Closed

Add HTTP/2 HttpServer metric for active streams #2356

samueldlightfoot opened this issue Jul 5, 2022 · 2 comments · Fixed by #2357
Labels
help wanted We need contributions on this type/enhancement A general enhancement
Milestone
1.0.21

Comments

@samueldlightfoot
Copy link
Contributor

Similar to that available for the HttpClient, it would be nice to expose the current number of active streams for a server as a metric.

Motivation

Desired solution

Extend the current server metrics recorders.

Considered alternatives

Additional context
@samueldlightfoot samueldlightfoot added status/need-triage A new issue that still need to be evaluated as a whole type/enhancement A general enhancement labels Jul 5, 2022
@violetagg violetagg added help wanted We need contributions on this and removed status/need-triage A new issue that still need to be evaluated as a whole labels Jul 5, 2022
@violetagg violetagg added this to the 1.0.x Backlog milestone Jul 5, 2022
@violetagg
Copy link
Member

@samueldlightfoot I would be great if you decide to provide PR!

@samueldlightfoot
Copy link
Contributor Author

Sure thing - will raise shortly.

@samueldlightfoot samueldlightfoot mentioned this issue Jul 5, 2022
Merged
@violetagg violetagg modified the milestones: 1.0.x Backlog, 1.0.21 Jul 6, 2022
@violetagg violetagg linked a pull request Jul 6, 2022 that will close this issue
Capture HTTP/2 server active stream metrics #2357
Merged
violetagg pushed a commit that referenced this issue Jul 7, 2022
@samueldlightfoot
Capture HTTP/2 server active stream metrics (#2357)
3a14fc6 
Fixes #2356
violetagg added a commit that referenced this issue Jul 7, 2022
@violetagg
Capture HTTP/2 server active stream metrics
23a9f4e 
This is a backport for PR #2357

Fixes #2356
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted We need contributions on this type/enhancement A general enhancement
Projects
None yet
Milestone
1.0.21
Development

Successfully merging a pull request may close this issue.

Capture HTTP/2 server active stream metrics samueldlightfoot/reactor-netty
2 participants
@violetagg @samueldlightfoot