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.

Sign up for GitHub

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

Jump to bottom

Monitoring Module #102

Merged
merged 19 commits into from
Oct 28, 2024
Merged

Monitoring Module #102

merged 19 commits into from
Oct 28, 2024

Conversation

brainrake
Copy link
Contributor

@brainrake brainrake commented Sep 10, 2024 •
edited
Loading

  • cardano.monitoring nixos module
  • prometheus exporter configuration
  • prometheus server
  • prometheus scraper configuration
  • grafana server
  • networking and firewall configuration
  • test
  • example
  • documentation
  • more doc
  • cloud deployment documentation: split from load balancer documentation
  • template: rename (multi-host, full, or something)
  • fix ports used for firewall, use service configs
  • prometheus exporters
    • node (host)
    • nginx
    • cardano-node
    • postgres
    • ogmios
    • kupo
    • blockfrost
    • db-sync
    • oura
  • prometheus scrapers
    • node (host)
    • nginx
    • postgres
    • cardano-node
    • ogmios
    • kupo
    • blockfrost
    • db-sync
    • oura
  • grafana dashboards
    • node (host)
    • nginx
    • postgres
    • cardano-node
    • ogmios
    • kupo
    • blockfrost
    • db-sync
    • oura
  • update nixpkgs to fix vm issue
  • fix doc rendering

as i couldn't find working grafana dashboards for all the services, we decided on the 2024-10-01 meeting to build a basic dashboard for cardano-node in #109 and consider dashboards for the other services out of scope

closes #88

@brainrake brainrake changed the title init monitoring module Monitoring Module Sep 10, 2024
@brainrake brainrake self-assigned this Sep 10, 2024
@brainrake brainrake force-pushed the marton/monitoring branch 2 times, most recently from 5846def to b657f17 Compare October 22, 2024 14:51
@brainrake brainrake mentioned this pull request Oct 22, 2024
Closed
avnik
avnik requested changes Oct 28, 2024
View reviewed changes
Copy link
Contributor

@avnik avnik left a comment

Choose a reason for hiding this comment

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

Looks sane for me. As I see is also have some Loadbalancer related changes.
Could you please address a magic numbers which I denote in code?

modules/monitoring.nix Outdated Show resolved Hide resolved
avnik
avnik approved these changes Oct 28, 2024
View reviewed changes
Copy link
Contributor

@avnik avnik left a comment

Choose a reason for hiding this comment

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

LGTM

@brainrake brainrake merged commit fdddfe2 into main Oct 28, 2024
2 checks passed
@brainrake brainrake deleted the marton/monitoring branch October 28, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@avnik avnik avnik approved these changes

Assignees

@brainrake brainrake

Labels
None yet
Projects
None yet
Milestone
5. Processing modules pt. 4 and closeout
Development

Successfully merging this pull request may close these issues.

Monitoring module
2 participants
@brainrake @avnik