Skip to content

chore: add storage pool backup metrics#3357

Closed
halfprice wants to merge 2 commits into
mainfrom
zhewu/backup_pool_metrics
Closed

chore: add storage pool backup metrics#3357
halfprice wants to merge 2 commits into
mainfrom
zhewu/backup_pool_metrics

Conversation

@halfprice

Copy link
Copy Markdown
Collaborator

Description

Add four operational gauges so we can see backup-side pool activity without
ad-hoc SQL:

  • walrus_storage_pools{status="live"|"expired"} — pool count by lifetime
  • walrus_pooled_blob_refs — total junction-table size
  • walrus_pool_referenced_blobs — distinct blob_ids alive via pools

Test plan

How did you test the new or updated feature?


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that
a user might notice and any actions they must take to implement updates. (Add release notes after the colon for each item)

  • Storage node:
  • Aggregator:
  • Publisher:
  • CLI:

halfprice and others added 2 commits May 7, 2026 22:20
The orchestrator already publishes blob_states and total_bytes_archived
on a 30s tick. Extend the same metrics query with four new aggregations
so operators can see pool activity at a glance:

  storage_pools{status="live"}     — pools whose end_epoch is in the future
  storage_pools{status="expired"}  — pools past their end_epoch (some still
                                     within the GC retention margin, others
                                     pending the pool-expiry sweep)
  pooled_blob_refs                 — total junction-table size; correlates
                                     with pending pool-cleanup work
  pool_referenced_blobs            — distinct blob_ids currently kept alive
                                     by at least one pool ref

All four scan small auxiliary tables or use simple predicates, so they
add negligible cost to the existing metrics query.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@iM4M-nugrh iM4M-nugrh 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.

commit

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open 14 days with no activity. It will be closed in 7 days unless you remove the stale label, add the do-not-close label, or comment on it.

@github-actions github-actions Bot added the stale label Jun 1, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

This PR was closed because it has not seen any activity since being marked as stale.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants