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

Redis does not start #118

Closed
waldner opened this issue Sep 27, 2019 · 4 comments
Closed

Redis does not start #118

waldner opened this issue Sep 27, 2019 · 4 comments
Labels

Comments

@waldner
Copy link

waldner commented Sep 27, 2019

Image: graphiteapp/graphite-statsd:1.1.5-12

Setting REDIS_TAGDB=true attemps to use redis for tagDB, but it fails because of the following error:

*** FATAL CONFIG FILE ERROR ***
Reading the configuration file, at line 8
>>> 'logfile /var/log/redis/redis-server.log'
Can't open the log file: No such file or directory

periodically repeated in the log, as runsv tries to restart it (I suppose). Since /var/log is host-mounted in my setup, I can create the directory by hand and then it works, but I believe it should not be necessary.

@deniszh
Copy link
Member

deniszh commented Oct 11, 2019

Hi @waldner,

Sorry, can't reproduce.

% docker run -e "REDIS_TAGDB=true" \
 --name graphite\
 --restart=always\
 -p 80:80\
 -p 2003-2004:2003-2004\
 -p 2023-2024:2023-2024\
 -p 8125:8125/udp\
 -p 8126:8126\
 graphiteapp/graphite-statsd
Started runsvdir, PID is 40
wait for processes to start....
/usr/sbin/crond 4.5 dillon's cron daemon, started with loglevel notice
...
Superuser created successfully.
ok: run: nginx: (pid 51) 3s
run: carbon: (pid 57) 5s; run: log: (pid 56) 5s
run: carbon-aggregator: (pid 67) 5s; run: log: (pid 65) 5s
down: carbon-relay: 1s, normally up, want up; run: log: (pid 58) 5s
down: collectd: 1s, normally up, want up
run: cron: (pid 54) 5s
run: graphite: (pid 61) 5s
run: nginx: (pid 51) 5s
run: redis: (pid 66) 5s
...

At the same I see data inside redis:

% docker exec -it 616f00db971e redis-cli --stat
------- data ------ --------------------- load -------------------- - child -
keys       mem      clients blocked requests            connections
182        1.02M    5       0       768 (+0)            9
182        1.02M    5       0       769 (+1)            9
182        1.02M    5       0       770 (+1)            9
182        1.02M    5       0       779 (+9)            9
182        1.02M    5       0       780 (+1)            9
182        1.02M    5       0       781 (+1)            9
182        1.02M    5       0       782 (+1)            9
182        1.02M    5       0       783 (+1)            9
182        1.02M    5       0       784 (+1)            9

@deniszh deniszh closed this as completed Oct 11, 2019
@deniszh deniszh reopened this Oct 11, 2019
@deniszh
Copy link
Member

deniszh commented Oct 11, 2019

Ah, sorry, I read it wrong.

@deniszh
Copy link
Member

deniszh commented Oct 11, 2019

Yes, /var/log/redis/ should be created automatically.

@deniszh deniszh added the bug label Oct 11, 2019
@deniszh
Copy link
Member

deniszh commented Oct 24, 2019

Fixed in 1.1.6-1

@deniszh deniszh closed this as completed Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants