Skip to content

feat: add push, fetch, and sync latency metrics - #362

Draft
anandh8x wants to merge 1 commit into
Gitlawb:mainfrom
anandh8x:agent/metrics-latency-histograms
Draft

feat: add push, fetch, and sync latency metrics#362
anandh8x wants to merge 1 commit into
Gitlawb:mainfrom
anandh8x:agent/metrics-latency-histograms

Conversation

@anandh8x

Copy link
Copy Markdown
Contributor

Summary

  • Add Prometheus histograms for successful push and fetch durations.
  • Track the origin ref-update timestamp through both peer transports and the sync queue.
  • Record sync lag only after the mirror is registered and the queue item completes.

Validation

  • cargo fmt --all -- --check
  • cargo check --locked --workspace --all-targets
  • cargo clippy --locked -p gitlawb-node --bin gitlawb-node -- -D warnings
  • Focused metrics, sync, notification, and migration tests
  • cargo test --locked -p gitlawb-node (825 passed; two pre-existing concurrency test failures remain outside this change)

@github-actions github-actions Bot added the needs-issue PR has no linked issue label Aug 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for the contribution. A couple of things will help us review this faster:

  • Link the issue this addresses (Closes #123). For protocol changes, open an issue first.

See CONTRIBUTING.md. Update the PR and these notes will clear automatically.

@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d99e82ab-a88a-4d52-87ab-a16e23ee60b0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@beardthelion beardthelion added crate:node gitlawb-node — the serving node and REST API subsystem:peers Peer announce, discovery, and registry subsystem:replication Mirror, replica, and cross-node sync labels Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crate:node gitlawb-node — the serving node and REST API needs-issue PR has no linked issue subsystem:peers Peer announce, discovery, and registry subsystem:replication Mirror, replica, and cross-node sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants