RedisTimeSeries icon indicating copy to clipboard operation
RedisTimeSeries copied to clipboard

Suggestion - add a full path hint for redistimeseries.so

Open gacerioni opened this issue 2 years ago • 2 comments

I run Redis Stack in a StatefulSet, and I had to exec into the container to find . -name redistimeseries.so so I could prepare my configmap with something like:

    {{- if .Values.enableRedisTimeSeries }}
    loadmodule /opt/redis-stack/lib/redistimeseries.so
    {{- end }}   

So I decided to suggest a brief comment on top of the configuration, just to help the next person.

gacerioni avatar Dec 08 '23 15:12 gacerioni

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Dec 08 '23 15:12 CLAassistant

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ca23346) 89.02% compared to head (1c417a4) 89.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1546   +/-   ##
=======================================
  Coverage   89.02%   89.02%           
=======================================
  Files          35       35           
  Lines        5834     5834           
=======================================
  Hits         5194     5194           
  Misses        640      640           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Dec 08 '23 15:12 codecov[bot]