[TS-3183] Clean up (and/or eliminate?) some proxy.node metrics

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Fix Version/s: 8.0.0
• Component/s: Metrics
• Labels: incompatible
• Assignee: Unassigned
• Reporter: Leif Hedstrom
• Created: 10/Nov/14 20:52
• Updated: 25/Oct/16 22:05

Description

There's two types of metrics it seems, proxy.node and proxy.process. Unfortunately, in at least a few cases, the same metrics get duplicated into both. I'm not sure why that makes sense, ever, so I think we should clean that up. That would certainly make it less confusing.

Secondly, it seems proxy.node metrics, which are registered via RecordsConfig.cc, are primarily used for computed metrics. These are metrics synthesized via the stats.config.xml mechanism. Albeit being useful, what's bad with the current design here is that adding new metrics to the XML config file still requires a change to RecordsConfig.cc. This makes no sense to me, so perhaps the proxy.node metrics should all be eliminated from RecordsConfig.cc, and exclusively owned (registered and updated) via the XML config format.

Elimination, or renaming, of any of these metrics would be an incompatible change. Therefore, it would go into 6.0.0.

Issue Links

• blocks: TS-4026 Remove clustering feature in 7.0.0 release
• blocks: TS-3671 ☂ Critical issues for v7.0.0 release
• is blocked by: TS-1882 ATS doesn't warn about unknown config items
• is related to: TS-3830 Metrics created via stats.config.xml should not need to be registered in RecordsConfig.cc
• is related to: TS-4099 Replace XML statistics with Lua
• relates to: TS-2470 Remove PCT metrics