Jump to conversation
Unresolved conversations (2)
@maqi maqi Jun 5, 2023
this is about the whole system usage? I think we don't care much about it?
Outdated
sn_logging/src/metrics.rs
RolandSherwin happybeing
@maqi maqi Jun 5, 2023
just a remind that this feature need to be enabled for the public testnet run
sn_cli/Cargo.toml
RolandSherwin happybeing
Resolved conversations (5)
@maqi maqi Jun 5, 2023
I think we only interested in our own process's usage, but here is refreshing the whole system's ?
sn_logging/src/metrics.rs
happybeing
@maqi maqi Jun 5, 2023
if any of the above 2 is `None`, then actually no point to have this metric instance?
sn_logging/src/metrics.rs
RolandSherwin
@maqi maqi Jun 5, 2023
is this somehow overlapped with the fields within `Metrics` ? maybe just make ```rust struct Metrics { network_metrics: NetworkMetrics, process_metrics: ProcessMetrics, } ```
sn_logging/src/metrics.rs
RolandSherwin
@maqi maqi Jun 5, 2023
number of messages received & transmitted as well?
sn_logging/src/metrics.rs
RolandSherwin maqi
@maqi maqi Jun 5, 2023
will the 5s interval being too frequent ? especially with a public testnet ?
sn_logging/src/metrics.rs
RolandSherwin