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

[Feature Request] Include Redis RTT in statsd metrics that are emitted #475

Closed
yvjessestephens opened this issue Apr 29, 2024 · 3 comments
Closed

Comments

@yvjessestephens
Copy link

  • Which Faktory package and version?
    Faktory 1.7.0, Enterprise

Some Redis metrics are already available (see Ent-Metrics):

  • Redis network connections (ops.redis.connections)
  • Redis RAM usage, in bytes (ops.redis.memory)

I'm interested in being able to capture the RTT so that we can alert if that metric exceeds its baseline by very much and investigate what might have changed to cause that (e.g. the SaaS Redis instance got moved to a different zone), and to track that metric over time for historical performance.

@mperham mperham closed this as completed Apr 29, 2024
@yvjessestephens
Copy link
Author

yvjessestephens commented May 1, 2024

@mperham , now that you have marked this as completed, in what version could I expect this feature to be available?

@mperham
Copy link
Collaborator

mperham commented May 1, 2024

I don't have an ETA on 1.9.0, I was trying to land #474 first but it's blocked until I have a way to reproduce and verify the PR improves things. I might need to adjust my release plan.

@mperham
Copy link
Collaborator

mperham commented May 20, 2024

this is now available in 1.9

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

No branches or pull requests

2 participants