GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/1049
TS-4892: Metric wrong for http.current_active_client_connections
Moved the m_active flag to ProxyClientSession. Added set_session_activate
and clear_session_activate methods which will only increment/decrement the
current_active_client_connections counter if the m_active flag changes value.
In all cases, the set_session_action is called from HttpSM when the request
header is successfully parsed.
For Http1, the clear_session_active is called when the transaction is
completed and when the session is closed.
For Http2, the clear_session_active is called when the number of active
streams goes to zero. The session may not necessarily be closing at this
point, but it is idle until another stream comes in.
We have been running with this patch in production
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shinrich/trafficserver ts-4892
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/1049.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1049
----
commit cb096bcbb5132980f78bcfc1aecbfc5dac36405f
Author: Susan Hinrichs <shinr...@ieee.org>
Date: 2016-09-26T18:32:20Z
TS-4892: Wrong metrix for
proxy.process.http.current_active_client_connections.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
