-
Notifications
You must be signed in to change notification settings - Fork 13
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
Make it possible to configure the server #3
Comments
Hi @mjpieters, thanks for reaching out! There have been some voices in the past indicating that snap provides a more standard configuration interface which may be leveraged. Unfortunately, I'm not familiar with snap and don't currently have the bandwidth to look into that - but any help with that will be appreciated. |
I've noticed and switched to a PPA providing newer Redis versions instead. However, if you are looking for an example, https://github.com/markshuttle/redis-snap uses |
Yeah I've looked into the configure hook exposed to snaps and I think it'd be mad to use the interface exposed by snap craft since you'd need to check each setting individually. This works alright when you only expose 4 settings but redis has 150 configuration keys, so yeah I don't know. |
@JonasKruckenberg What would be the de-facto standard way for snap services to handle that? I would assume they'd just have a config file in a well known location, isn't that the case? |
Well since the default configuration is so poorly supported most snap I've come across opted to use "out of band configuration" via some other method. This is mostly via settings in their app because most snaps are visual and users don't wanna mess with config files. |
This'd also allow admin to copy paste their existing config files which would be a nice feature too |
Can the
redis.server
service please look for aredis.conf
file, perhaps in$SNAP_COMMON/etc/
?All the bettir if the default configuration file for the Redis version packaged were to be copied there on install.
The text was updated successfully, but these errors were encountered: