Problem parsing redis url incoming driver

Bug #1691542 reported by Victor Eduardo Bazterra on 2017-05-17
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fix Released
Julien Danjou

Bug Description

Looking for more performance, I explored the very recent version of gnocchi in master branch

commit 25cc9d7a03e81199379c900b85d16b347a8f2387

and I believed I found a bug.

I wanted to have incoming measurements store using redis with sentinel. For this I used the following setup in /etc/gnocchi/gnocchi.conf

driver = redis
redis_url = redis://{fakeuser}:{password}@{sentinel1}:26379?sentinel=mymaster&sentinel_fallback={sentinel2}:26379&sentinel_fallback={sentinel3}:26379

Gnocchi API was not able to connect to redis with sentinel. However, it was able to run when connecting to non sentinel redis installation.

Debugging I found that in gnocchi/storage/common/ when processing CLIENT_LIST_ARGS the line

it takes the last element of the list instead the full list. I did the following modification to fix this issue:

This fixed my problem, I hope this help others.


Fix proposed to branch: master

Changed in gnocchi:
assignee: nobody → Julien Danjou (jdanjou)
status: New → In Progress

Submitter: Jenkins
Branch: master

commit 8dc03b9aca500889ef26c2044c48d3d40ee9ef84
Author: Julien Danjou <email address hidden>
Date: Wed May 17 23:31:28 2017 +0200

    redis: fix CLIENT_LIST_ARGS parsing

    The entire list must be taken, not only the last argument.

    Change-Id: I9b5b9336250cb5292cdd6c7ca4964bfe186575d5
    Closes-Bug: #1691542

Changed in gnocchi:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers