Skip to content

Document CRN-specific scoring rules - #82

Merged
aliel merged 1 commit into
mainfrom
feature/crn-scoring-rules
Aug 26, 2026
Merged

Document CRN-specific scoring rules#82
aliel merged 1 commit into
mainfrom
feature/crn-scoring-rules

Conversation

@aliel

@aliel aliel commented Aug 26, 2026

Copy link
Copy Markdown
Member

Adds an Additional rules for Compute Resource Nodes section to the scoring page (docs/nodes/resources/scoring/index.md), documenting the CRN-only checks that were missing:

  • Stable IP addresses — static IPv4/IPv6 required; 2 or more IPv4 or IPv6-range changes over a rolling 30-day window are flagged. IPv6 is keyed to the node's VM address pool (IPV6_ADDRESS_POOL), not the access address. The window is rolling, so the score recovers as changes age out.
  • One scored node per machine — when several CRNs share an IPv4 or IPv6 pool, only the verified-or-earliest node keeps its score; the rest are set to 0. NAT/reverse-proxy caveat included.
  • Liveness (active / inactive / dead) — 24h proof-of-CRN; a dead node's score is set to 0.
  • Diagnostic codes table (1001–1007).

An info note states the rules are rolled out progressively, with warning codes published before a rule affects the score.

Notes for reviewers:

  • The 30-day IP-stability window matches the intended deployment setting (the scoring default is currently 365 days; deployment sets IP_STABILITY_WINDOW).
  • These rules ship on the updated scoring service; timing the publish with that rollout is recommended.

Add an "Additional rules for Compute Resource Nodes" section to the
scoring page covering the CRN-only checks: IP stability (static IPv4/IPv6
and a limit of 2 address changes over a rolling 30-day window, with IPv6
keyed to the VM address pool), one scored node per machine (duplicate
IPv4/IPv6-pool handling), and liveness status (active/inactive/dead).
Include the diagnostic code table and a note that the rules are rolled
out progressively with warning codes before they affect the score.
@github-actions

Copy link
Copy Markdown

Deployed on:

Preview for this pull request, updated on every commit and removed once the PR is closed.

@foxpatch-aleph foxpatch-aleph 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.

This PR adds a well-structured, clearly written section documenting CRN-specific scoring rules. The diagnostic code table is complete and maps correctly to the rules described in the text. Internal links and anchors are valid, terminology (IPV6_ADDRESS_POOL, /status/config) is consistent with the installation guides, and the VitePress container syntax matches the rest of the docs. The content accurately distinguishes the 30-day rolling IP-stability window from the 2-year performance window, explains the liveness thresholds, and includes appropriate operator-facing warnings about NAT/shared-IP scenarios. No correctness, security, or quality issues found.

docs/nodes/resources/scoring/index.md (line 146): Minor: diagnostic code 1007 ("Not currently proving it is a CRN") corresponds to the "inactive" status, but the Liveness section only states that "dead" nodes get a score of 0. It might help to clarify whether 1007 is purely informational or if it can affect the score during the progressive rollout — an operator seeing this code may wonder if their score is impacted. The top-level info note about warnings covers this implicitly, but an explicit note would reduce support questions.

@aliel
aliel merged commit d068dd0 into main Aug 26, 2026
1 check passed
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