Skip to content

Metasnapshot metrics fix and dashboard#321

Merged
alexbozhenko merged 10 commits into
nats-io:mainfrom
alexbozhenko:metasnapshot_duration
Feb 6, 2026
Merged

Metasnapshot metrics fix and dashboard#321
alexbozhenko merged 10 commits into
nats-io:mainfrom
alexbozhenko:metasnapshot_duration

Conversation

@alexbozhenko
Copy link
Copy Markdown
Member

@alexbozhenko alexbozhenko commented Feb 5, 2026

Follow-up to #315

  1. Metrics are present only in jsz, but not in statsz endpoint. Per conversation with the server team, we do not want meta shnapshot metrics in every statsz ping. So Fix the metrics source
  2. Use recommended base units
  3. Add dashboard with annotations of actual snapshot events, so effects of changes in meta_compact_size (see Add meta_compact_size option to further control JetStream meta group compaction/snapshotting nats-server#7521) can be easily tracked:
image

@alexbozhenko alexbozhenko requested a review from wallyqs February 5, 2026 12:42
container_name: prometheus
restart: "no"
ports:
- "9090:9090"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is to debug raw values in prometheus

Comment thread Dockerfile
@@ -1,4 +1,4 @@
FROM golang:1.24-alpine3.21 AS build
FROM golang:1.25-alpine3.23 AS build
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

These version bumps are intentional, otherwise it wouldn't build local image, since version in go.mod is newer

@alexbozhenko alexbozhenko changed the title Metasnapshot duration Metasnapshot metrics fix and dashboard Feb 5, 2026
Copy link
Copy Markdown
Member

@wallyqs wallyqs left a comment

Choose a reason for hiding this comment

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

LGTM

@alexbozhenko alexbozhenko merged commit 669a5b7 into nats-io:main Feb 6, 2026
2 checks 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