2021-03-26 09:14:12 -05:00
|
|
|
package metrics
|
|
|
|
|
|
|
|
// ResettingSample converts an ordinary sample into one that resets whenever its
|
|
|
|
// snapshot is retrieved. This will break for multi-monitor systems, but when only
|
|
|
|
// a single metric is being pushed out, this ensure that low-frequency events don't
|
|
|
|
// skew th charts indefinitely.
|
|
|
|
func ResettingSample(sample Sample) Sample {
|
|
|
|
return &resettingSample{
|
|
|
|
Sample: sample,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// resettingSample is a simple wrapper around a sample that resets it upon the
|
|
|
|
// snapshot retrieval.
|
|
|
|
type resettingSample struct {
|
|
|
|
Sample
|
|
|
|
}
|
|
|
|
|
|
|
|
// Snapshot returns a read-only copy of the sample with the original reset.
|
metrics, cmd/geth: change init-process of metrics (#30814)
This PR modifies how the metrics library handles `Enabled`: previously,
the package `init` decided whether to serve real metrics or just
dummy-types.
This has several drawbacks:
- During pkg init, we need to determine whether metrics are enabled or
not. So we first hacked in a check if certain geth-specific
commandline-flags were enabled. Then we added a similar check for
geth-env-vars. Then we almost added a very elaborate check for
toml-config-file, plus toml parsing.
- Using "real" types and dummy types interchangeably means that
everything is hidden behind interfaces. This has a performance penalty,
and also it just adds a lot of code.
This PR removes the interface stuff, uses concrete types, and allows for
the setting of Enabled to happen later. It is still assumed that
`metrics.Enable()` is invoked early on.
The somewhat 'heavy' operations, such as ticking meters and exp-decay,
now checks the enable-flag to prevent resource leak.
The change may be large, but it's mostly pretty trivial, and from the
last time I gutted the metrics, I ensured that we have fairly good test
coverage.
---------
Co-authored-by: Felix Lange <fjl@twurst.com>
2024-12-10 06:27:29 -06:00
|
|
|
func (rs *resettingSample) Snapshot() *sampleSnapshot {
|
2021-03-26 09:14:12 -05:00
|
|
|
s := rs.Sample.Snapshot()
|
|
|
|
rs.Sample.Clear()
|
|
|
|
return s
|
|
|
|
}
|