Skip to content

feat: Add local sample setup for Prometheus + Grafana monitoring#182

Open
OldManYells wants to merge 2 commits into
retromcorg:masterfrom
OldManYells:monitoring
Open

feat: Add local sample setup for Prometheus + Grafana monitoring#182
OldManYells wants to merge 2 commits into
retromcorg:masterfrom
OldManYells:monitoring

Conversation

@OldManYells

Copy link
Copy Markdown
Contributor

📌 Pull Request

Description

Small example of setting up Prometheus and Grafana monitoring for Poseidon.

Went a bit further and decided to setup the exporting directly by the server itself instead of using an exporter like JMX. Gives us a bit more control. This is setup using docker for local testing but can easily be extended to other deployment models

Details are available in monitoring/README.md

Sample included dasboard
image

@Garsooon Garsooon self-requested a review April 5, 2026 10:40

@Garsooon Garsooon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docker works fine after tinkering with the code I comentted on and at that works really well, good work! However my comments will need to be addressed before I can confortably merge it.

- GF_SECURITY_ADMIN_USER=admin
- GF_SECURITY_ADMIN_PASSWORD=admin
- GF_USERS_ALLOW_SIGN_UP=false
ports:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting consistent errors here when trying to start up the run script after getting docker up. Is there a reason you used 3000:3000 and not 3001:3000?

@@ -0,0 +1,39 @@
#!/bin/sh

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be an issue on my end, but set -eu doesnt seem to work (on windows). set -e works just fine though.

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