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.

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

feat: new export for stuck packets #24

Merged
merged 13 commits into from
Oct 31, 2023
Merged

Conversation

jlehtimaki
Copy link
Contributor

This PR will add new IBC export for IBC channel which will include stuck/pending packets metric.
Also renaming/combining IBCClientsCollector to IBCCollector to use only one collector for all IBC related metrics.

This is used to have metrics about IBC channel, now just have stuck packets
@jlehtimaki jlehtimaki requested review from kayano and shahbazn October 26, 2023 10:32
this splits one total metric so that it will produce three metrics, src,dst and total
@jlehtimaki jlehtimaki force-pushed the netx-78-expor-stuck-packets branch from da69e26 to fac6c90 Compare October 27, 2023 08:04
pkg/ibc/ibc.go Show resolved Hide resolved
pkg/collector/collector.go Outdated Show resolved Hide resolved
pkg/ibc/ibc.go Outdated Show resolved Hide resolved
pkg/ibc/ibc.go Outdated Show resolved Hide resolved
pkg/ibc/ibc.go Outdated Show resolved Hide resolved
pkg/collector/collector.go Outdated Show resolved Hide resolved
pkg/ibc/ibc.go Outdated Show resolved Hide resolved
pkg/ibc/ibc.go Show resolved Hide resolved
@jlehtimaki jlehtimaki force-pushed the netx-78-expor-stuck-packets branch from 248ac90 to 839bdaa Compare October 30, 2023 14:35
@jlehtimaki jlehtimaki requested a review from kayano October 31, 2023 09:11
@jlehtimaki jlehtimaki merged commit 2bba835 into main Oct 31, 2023
3 checks passed
@jlehtimaki jlehtimaki deleted the netx-78-expor-stuck-packets branch October 31, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants