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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

redis using sockets (v3.13.1) #316

Closed
mmattel opened this issue Mar 21, 2021 · 2 comments
Closed

redis using sockets (v3.13.1) #316

mmattel opened this issue Mar 21, 2021 · 2 comments

Comments

@mmattel
Copy link

mmattel commented Mar 21, 2021

Hello @mikaku 馃憢

it is possible to use a socket in the redis definition of monitorix 馃槂
list = /var/run/redis/redis-server.sock

You may want to add this in your docs.

Please also see that there is a shift in the text output starting with Avg that needs correction. It shifts dependent on the number of Cur.
image

@mikaku
Copy link
Owner

mikaku commented Apr 7, 2021

This commit includes support to recognize a socket file in list option. The file name must end with .sock.
Please download the latest redis.pm, test it and let me know if it works as expected.

@mikaku
Copy link
Owner

mikaku commented Apr 23, 2021

Closing due to no response for two weeks.

If you still need more information, please add a comment and the issue can be reopened.

@mikaku mikaku closed this as completed Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants