Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monitoring: improve reported ATProto subscribeRepos send delay #1644

Open
snarfed opened this issue Dec 21, 2024 · 0 comments
Open

Monitoring: improve reported ATProto subscribeRepos send delay #1644

snarfed opened this issue Dec 21, 2024 · 0 comments
Labels

Comments

@snarfed
Copy link
Owner

snarfed commented Dec 21, 2024

We use the task_delay log-based metric to monitor whether we're behind on emitting records via subscribeRepos to the relay and other clients. We need to fix two things here:

  • Right now, we include timestamps from clients reading historical events in the rollback window, which we shouldn't, since it makes the actual reported delay wrong
  • This metric stops updating if we stop emitting events. We should regularly check and report the difference between our latest stored seq and the latest event we've emitted
@snarfed snarfed added the infra label Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant