Skip to content

Some update - #53

Open
dewabisma wants to merge 1 commit into
mainfrom
beast/mainnet-monitoring
Open

Some update#53
dewabisma wants to merge 1 commit into
mainfrom
beast/mainnet-monitoring

Conversation

@dewabisma

Copy link
Copy Markdown
Contributor

No description provided.

@dewabisma
dewabisma requested a review from n13 September 9, 2026 04:52

@n13 n13 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewer model: GPT 5.6 Sol

Verdict: REQUEST_CHANGES

One blocking operational issue remains:

  • [P1] Keep the RPC scrape targets on resolvable hostnames — prometheus/prometheus.yml:837 and :872. All four new targets (rpc1/rpc2 × qm/hm under *-mainnet.quantus.com) currently have no A or AAAA answer from either 1.1.1.1 or 8.8.8.8, and an HTTPS probe fails with curl error 6. In contrast, all four hostnames this patch replaces still resolve and return the expected Cloudflare Access 401. Deploying this config now makes both rpcnode-chain and rpcnode-node unable to scrape either RPC node, so the mainnet RPC dashboards and data-based alerts lose their real signal. Please provision and externally verify the four exact DNS/tunnel hostnames before switching monitoring, or retain the resolving targets until that dependency is live.

Validation performed at head 07907dc828a7b1062a4af7b3b47ae41e125a12d3:

  • promtool v3.14.0 check config prometheus/prometheus.yml passed.
  • All changed JSON and YAML parsed successfully; git diff --check passed.
  • Prettier passed for every changed Grafana JSON/YAML file. The repository-wide formatter still reports only three unchanged files.
  • All 18 new bootnode/RPC metric names were checked in public DNS; the 14 bootnode names resolve, while the four RPC names above do not.

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