Skip to content

Commit

Permalink
Align metric function name with metric's name (informalsystems#1607)
Browse files Browse the repository at this point in the history
- fn `ibc_client_update` is the only fn which has a slightly different
name with the metrics it emits
- renaming it to its metrics name improves the readability of the code
  • Loading branch information
erain9 authored Nov 24, 2021
1 parent cb70bfb commit 5864ea6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion relayer/src/worker/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ impl<ChainA: ChainHandle, ChainB: ChainHandle> ClientWorker<ChainA, ChainB> {
match client.refresh() {
Ok(Some(_)) => {
telemetry!(
ibc_client_update,
ibc_client_updates,
&self.client.dst_chain_id,
&self.client.dst_client_id,
1
Expand Down
2 changes: 1 addition & 1 deletion telemetry/src/state.rs
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ impl TelemetryState {
}

/// Update the number of client updates per client
pub fn ibc_client_update(&self, chain: &ChainId, client: &ClientId, count: u64) {
pub fn ibc_client_updates(&self, chain: &ChainId, client: &ClientId, count: u64) {
let labels = &[
KeyValue::new("chain", chain.to_string()),
KeyValue::new("client", client.to_string()),
Expand Down

0 comments on commit 5864ea6

Please sign in to comment.