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

[RFE] add ability to pull metrics from unix socket in redis pmda #2105

Open
jainnikhil30 opened this issue Nov 19, 2024 · 2 comments
Open

[RFE] add ability to pull metrics from unix socket in redis pmda #2105

jainnikhil30 opened this issue Nov 19, 2024 · 2 comments

Comments

@jainnikhil30
Copy link
Contributor

jainnikhil30 commented Nov 19, 2024

Currently redis pmda looks for a $host:$port combination for pulling the redis metrics. But it is possible that redis is using unix socket and not port like following:

redis 29103 1 0 04:05 ? 00:00:11 /usr/bin/redis-server unixsocket:/run/redis/redis.sock

With this redis pmda is not able to fetch the redis metrics. It should be able to fetch the metrics using redis.sock as well.

redis-cli -s /run/redis/redis.sock INFO
#Server
redis_version:6.2.16
redis_git_sha1:00000000
redis_git_dirty:0
redis_build_id:85a2e85bf3a77649
redis_mode:standalone
os:Linux 5.14.0-529.el9.x86_64 x86_64
@hhorak
Copy link
Contributor

hhorak commented Nov 20, 2024

@jainnikhil30 Can you share some more info about what is your use case to get the data from the socket directly?

@natoscott natoscott changed the title [RFE] add ability to pull metrics from socket in redis pmda [RFE] add ability to pull metrics from unix socket in redis pmda Jan 29, 2025
@natoscott
Copy link
Member

natoscott commented Jan 29, 2025

@hhorak this is a mode in which redis/valkey can be run where it communicates only locally (security, efficiency) - currently the PMDA makes assumptions and is unaware of this feature.

Longer term, it would be good to rewrite this PMDA as a new (python/C) valkey PMDA, and allow Redis metrics support as a backwards-compatibility option in the updated code.

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

3 participants