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

docs: explain how to add metrics to grafana #11875

Merged
merged 4 commits into from
Oct 19, 2024

Conversation

greged93
Copy link
Contributor

This PR adds documentation related to adding a new panel to Grafana and resolves #10794.

Copy link
Member

@Rjected Rjected left a comment

Choose a reason for hiding this comment

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

thanks! just left some comments

etc/README.md Outdated
Comment on lines 18 to 20
Thank you for providing that context. I understand now that this text is already part of a subsection. I'll revise it to
fit better within that structure while still improving clarity and readability.

Copy link
Member

Choose a reason for hiding this comment

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

Should remove this :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops 😗

etc/README.md Outdated
Comment on lines 65 to 64
9. Update dashboard file:
- Replace the content of the corresponding file in the [dashboards folder](grafana/dashboards) with the exported
JSON
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
9. Update dashboard file:
- Replace the content of the corresponding file in the [dashboards folder](grafana/dashboards) with the exported
JSON
9. Update dashboard file:
- Replace the content of the corresponding file in the [dashboards folder](./grafana/dashboards) with the exported
JSON

etc/README.md Outdated
Comment on lines 30 to 36
2. Build the Reth image:
```bash
docker build . -t reth:local
```
Modify the [docker-compose](docker-compose.yml) file to use your locally built image for the Reth service.

3. Run Docker Compose:
```bash
docker compose -f etc/docker-compose.yml -f etc/lighthouse.yml up -d
```
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should necessarily recommend using docker compose for this, let's just say that they should have a reth instance running, and a grafana instance running.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmmm these are docs written for a contributor no? personally found that docker was the easiest way for me to get everything up and running so that I could start adding a new panel

Copy link
Member

Choose a reason for hiding this comment

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

Adding these as steps make it seem a bit like it's required, but I guess it's worth making a note that we have the docker compose. I personally find it a bit hard to work with, and like to just run reth outside of docker normally

etc/README.md Outdated
6. Configure your metric panel:
- Set a panel title and description
- Select metric(s) from the `Metrics browser` or use the `PromQL` terminal
- Document your metric(s) by setting units, legends,...
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Document your metric(s) by setting units, legends,...
- Document your metric(s) by setting units, legends, etc.

etc/README.md Outdated
Comment on lines 71 to 74
#### Import Grafana dashboards

Running Grafana in Docker makes it possible to import existing dashboards, refer
to [docs on how to run only Grafana in Docker](/book/installation/docker.md#using-docker-compose#run-only-grafana-in-docker).
Copy link
Member

Choose a reason for hiding this comment

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

This is also possible without using docker, we should just add some notes on how to import the dashboard after it changes on main

etc/README.md Outdated

#### Adding a new metric to Grafana

To set up a new metric in the Reth Grafana dashboard:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
To set up a new metric in the Reth Grafana dashboard:
To set up a new metric in Reth and its Grafana dashboard:

@onbjerg onbjerg changed the title docs: adding grafana panels docs: add grafana panels Oct 18, 2024
@mattsse mattsse added the A-observability Related to tracing, metrics, logs and other observability tools label Oct 19, 2024
@greged93 greged93 force-pushed the greged93/docs/grafana branch 2 times, most recently from 8650897 to 46c975c Compare October 19, 2024 15:51
@onbjerg onbjerg changed the title docs: add grafana panels docs: explain how to add grafana panels Oct 19, 2024
@onbjerg onbjerg changed the title docs: explain how to add grafana panels docs: explain how to add metrics to grafana Oct 19, 2024
@onbjerg onbjerg requested a review from Rjected October 19, 2024 20:20
@onbjerg onbjerg added this pull request to the merge queue Oct 19, 2024
Merged via the queue into paradigmxyz:main with commit 6c026da Oct 19, 2024
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-observability Related to tracing, metrics, logs and other observability tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create guide on how to add and update grafana panels
4 participants