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

Add new metrics requested by operators #2182

Merged
merged 31 commits into from
May 17, 2022
Merged

Add new metrics requested by operators #2182

Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
b2bf490
Add metric for number of WS reconnections
romac May 3, 2022
bbd4197
Reformatting
romac May 3, 2022
fe54a90
Add metric for number of IBC events received over WebSocket
romac May 3, 2022
5f35129
Add missing doc comments
romac May 3, 2022
08fe032
Add metric for number of messages submitted to a chain
romac May 3, 2022
c885d1e
Add facility for querying an account's balance via the bank module
romac May 4, 2022
a4db8e5
Code reorganization
romac May 4, 2022
0d15e95
Move `query_balance` to `ChainEndpoint`
romac May 4, 2022
2afafd4
Add `wallet_balance` metric
romac May 4, 2022
71471a4
Add `query_balance` to the chain handle
romac May 4, 2022
6892327
Update parameter name
romac May 5, 2022
f516c9d
Add wallet worker to monitor wallet balance and populate correspondin…
romac May 10, 2022
71f5420
Update bank proto
romac May 11, 2022
76adc3a
Implement `tx_latency` metric
romac May 11, 2022
263eb5d
Use `moka::sync::Cache` to capture in-flight txs in the telemetry sta…
romac May 12, 2022
670a187
Use short UUID
romac May 12, 2022
a6eb59f
Rename `tx_latency` to `tx_latency_confirmed` and add `tx_latency_sub…
romac May 12, 2022
787431e
Remove useless clone
romac May 12, 2022
7be0dc1
Add more labels to `tx_latency_*` metrics
romac May 12, 2022
e525576
Fix for rebase on master
romac May 12, 2022
b51bb46
Update guide with new metrics
romac May 13, 2022
60933be
Add changelog entry
romac May 13, 2022
66ad0c2
Do not add tracking id as label to tx_latency metrics to avoid blowin…
romac May 13, 2022
800f73b
Use custom aggregator to properly report wallet balance metric
romac May 16, 2022
c2b52ea
Update guide metrics
romac May 16, 2022
907d41a
Merge branch 'master' into romac/2112-telemetry
romac May 16, 2022
a07d836
Document `TrackingId`
romac May 17, 2022
a35a79a
Rename constructors of `TrackingId`
romac May 17, 2022
2d3a34d
Turn comments into doc comments
romac May 17, 2022
ef9db39
Merge branch 'master' into romac/2112-telemetry
romac May 17, 2022
684d1c3
Rename `chain::tx` module to `chain::tracking`
romac May 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Do not add tracking id as label to tx_latency metrics to avoid blowin… 
…g up the Prometheus exporter
  • Loading branch information
@romac
romac committed May 13, 2022
commit 66ad0c2a4dd809b8faa40d2cfe2c3b8f65a5fba7
4 changes: 2 additions & 2 deletions telemetry/src/state.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ impl TelemetryState {
let latency = start.elapsed().as_millis() as u64;

let labels = &[
KeyValue::new("tracking_id", tracking_id),
// KeyValue::new("tracking_id", tracking_id),
KeyValue::new("chain", chain_id.to_string()),
KeyValue::new("counterparty", counterparty_chain_id.to_string()),
KeyValue::new("channel", channel_id.to_string()),
Expand All @@ -266,7 +266,7 @@ impl TelemetryState {
let latency = start.elapsed().as_millis() as u64;

let labels = &[
KeyValue::new("tracking_id", tracking_id),
// KeyValue::new("tracking_id", tracking_id),
KeyValue::new("chain", chain_id.to_string()),
KeyValue::new("counterparty", counterparty_chain_id.to_string()),
KeyValue::new("channel", channel_id.to_string()),
Expand Down