-
Notifications
You must be signed in to change notification settings - Fork 69
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
Support Grafana via prom_ex
#1518
base: main
Are you sure you want to change the base?
Conversation
0e877c8
to
9d4afc7
Compare
This fixes a bunch of `force_ssl` issues since this one endpoint can be http only, plus its easy not to expose it to the world
I like the idea of this a lot, do we know how to have prometheus scrape ECS tasks directly? Last time I had a chance to work with prometheus it didn't handle dynamic pods/tasks/containers well (at least with whatever set up the company had haha) |
Sorry, I'm not sure on the best way to solve the prometheus scape part of the equation. Fly.io makes it super super easy by providing you a prometheus out of the box. We have this up and running with NervesCloud, including the correct taging of apps, instances, and hosts. |
270a8a6
to
2eb3082
Compare
Using Grafana has become more of the norm, and this is the start of supporting all the Grafanas
Both the Web and Devices Endpoints are monitored, which gives us some initial insight into whats going on within the app.
I've implemented the
/metrics
route via a newMetricsEndpoint
, as this allows us to expose it on its own port which isn't exposed, without SSL, which Fly can monitor via its Prometheus service.I think these dashboards require a little more work until I'm happy with the data, and insight, they provide.