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

Add grafana panel(s) for NFT TTR #2003

Closed
alanshaw opened this issue Jun 20, 2022 · 4 comments
Closed

Add grafana panel(s) for NFT TTR #2003

alanshaw opened this issue Jun 20, 2022 · 4 comments
Assignees
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization pi/SLA

Comments

@alanshaw
Copy link
Contributor

Similar to https://protocollabs.grafana.net/d/fmj7-9Znz/web3-storage-stats?orgId=1&viewPanel=25

@alanshaw alanshaw added kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization labels Jun 20, 2022
@dchoi27 dchoi27 added the pi/SLA label Jun 20, 2022
@gobengo
Copy link
Contributor

gobengo commented Jun 21, 2022

@alanshaw I wrestled with this a not understanding why there were differences in grafana UI of the metric you point to above compared to the ones I'm sending.

I noticed that the metric you're using there is a gauge https://github.com/web3-storage/web3.storage/blob/78ef3c2a7d0c810a1dc816072e88acbfacd5e742/.github/workflows/cron-test.yml#L39
Whereas in #1943 we used a histogram.

@gobengo
Copy link
Contributor

gobengo commented Jun 21, 2022

@alanshaw There are two panels in this dashboard i made

I think you should be able to see it?
There aren't many datapoints yet, but the panels/queries should be right for when more data comes in.

I used these docs: https://grafana.com/blog/2020/06/23/how-to-visualize-prometheus-histograms-in-grafana/

If you want a plot exactly like the one you linked to, I believe I'll need to add another metric of type gauge.

@lanzafame
Copy link

@gobengo With histograms, three metrics are actually created, the metric_buckets, metric_count, and metric_sum. To create the panel that @alanshaw is after you should be able to use the metric_sum metric.

@gobengo
Copy link
Contributor

gobengo commented Jun 27, 2022

Data now flowing in after realizing what was wrong (nft.storage needed PUSHGATEWAY_BASIC_AUTH secret, which I added).

https://protocollabs.grafana.net/d/9wQvbG37k/bengo-nft-ttr?orgId=1&from=now-12h&to=now

lmk if these aren't sufficient @alanshaw, or if you want me to add them to any specific other dashboards (or make available as reusable panels) and we'll reopen/fast-follow

@gobengo gobengo closed this as completed Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A net-new feature or improvement to an existing feature need/triage Needs initial labeling and prioritization pi/SLA
Projects
None yet
Development

No branches or pull requests

4 participants