-
Notifications
You must be signed in to change notification settings - Fork 161
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
NSQ mixin #719
NSQ mixin #719
Conversation
add topic metrics
repeat topics panels
- set of two dashboards - nsqgrafonnet timeseries panel used
af24785
to
9dd545d
Compare
685bbb9
to
0875135
Compare
a3ece3b
to
e503dc8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of small nits about formatting, but nothing that should block this if you do not agree with my comments.
Excellent work. Thank you!
local prometheus = grafana.prometheus; | ||
|
||
{ | ||
new(title, description=null):: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great, I'll likely use it in some of my future mixins.
Do we anticipate building more of these for other newer visualizations?
Do we think that it makes sense to offer this upstream to grafonnet, or as a separate extension library?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even though there concerns about grafonnet lib(grafana/grafonnet-lib#332), I think we should use some time to update grafonnet upstream.
This PR introduces NSQ mixin.
NSQ exposes metrics via statsd, so Grafana agent/statsd_exporter required so get metrics into Prometheus.
This NSQ mixin interesting features:
Set of two dashboards:
NSQ Topics - view metrics grouped by topics and channels
NSQ Instances - view metrics grouped by nsqd instances
Dashboard links to quickly jump from one dashboard(view) to another.
Data links to contextually jump from instance view to topics view and vice versa
Alerts for topics and channel depth.
In order to test this mixin with live data, please use sample app in dev environment.