[TODO]: Review Redis config #3855
Labels
area/configuration (file)
area/documentation
kind/improvement
Improve an existing feature, configuration file or the documentation
meta/help wanted
The OP requests help from others - chime in! :D
stale-bot/ignore
Indicates that this issue / PR shall not be closed by our stale-checking CI
Description
I saw these past comments of mine while looking into separate issue about our logging support, figured it might be worth raising a TODO issue if anyone has time.
For Debian Bookworm / DMS v14, I take it there is no issues with the Redis v7 upgrade, and no complaints about persistence config have come up since the review feedback.
Feel free to close this issue if it doesn't seem relevant, I'm not likely to have time to invest towards it myself. This is more for maintainer documentation / discovery reasons 😅
Presently dependent upon config shipped by default / Debian package, and modifying that:
docker-mailserver/target/scripts/startup/setup.d/security/rspamd.sh
Lines 117 to 145 in 23705e6
docker-mailserver/target/supervisor/conf.d/supervisor-app.conf
Lines 108 to 115 in 23705e6
Concerns / Advice provided previously:
ENABLE_REDIS
& add persistence for Redis #3143 (review)For maintenance, it'd probably be worthwhile to document the configuration changes. Some of the changes aren't obvious why they're done (as per review feedback below), where the only context available is
git blame
(until some eventual refactoring adds friction there 😬 )References
Screenshots because I'm lazy.
Debian Bookworm upgrade (Redis v7)
The 2nd linked comment also expresses a concern for Redis v7 upgrade with Debian Bookworm:
There was also another review comment there on the config contributed, with an intent for it to be reviewed for Debian Bookworm update (I don't think that was done yet?):
Persistence
Related was a concern about how persistence would be configured and changes in that support with v7 of Redis (additionally noting a difference in Debian shipped config vs upstream Redis):
Out of the two persistence links to Redis docs originally provided, only one is still valid. The other appears to now be located here.
Log config
Separate concern was expressed regarding log config (which may be more relevant [when Vector is adopted into DMS])#3561)):
From upstream Redis config docs (should be the default for us due to removing the config line, thus stdout to file through supervisord):
NOTE:
/var/log/{redis,rspamd}
exist as directories. Presumably left-over from Debian package installs? 🤷♂️The text was updated successfully, but these errors were encountered: