-
Notifications
You must be signed in to change notification settings - Fork 720
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
umask for data #1725
base: unstable
Are you sure you want to change the base?
umask for data #1725
Conversation
Signed-off-by: kronwerk <[email protected]>
Signed-off-by: kronwerk <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## unstable #1725 +/- ##
============================================
+ Coverage 71.10% 71.16% +0.05%
============================================
Files 123 123
Lines 65536 65536
============================================
+ Hits 46599 46637 +38
+ Misses 18937 18899 -38
|
I don't have much experience in administrating valkey deployments, but maybe using the server binary mask isn't also what we're expecting. |
yeah, that was another option - but it appears to be an overkill to create 1 (or may be even 2, for AOF and RDB separately) another config setting to achieve this flexibility. I (maybe I'm wrong?) can't find a situation there we gonna need different permissions for AOF and RDB - and Valkey binary and AOF|RDB also |
@madolson hi, could you please take a look at this PR? thx |
Signed-off-by: kronwerk <[email protected]>
data files (both AOF and RDB) are created now using hardcoded filemode (0644).
that can be inconvenient in some cases.
in this PR it's proposed to create that files using server.umask providing more flexibility for a user.