-
Notifications
You must be signed in to change notification settings - Fork 114
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
Updated the log directory permission #2376
Conversation
✔️ Deploy Preview for chef-supermarket ready! 🔨 Explore the source changes: 99698a5 🔍 Inspect the deploy log: https://app.netlify.com/sites/chef-supermarket/deploys/61d5704ad6dac20008a23899 😎 Browse the preview: https://deploy-preview-2376--chef-supermarket.netlify.app |
Simplecov Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to document this
We may also need to change other recipes https://github.com/chef/supermarket/blob/main/omnibus/cookbooks/omnibus-supermarket/recipes/redis.rb#L31 |
There's already a PR for the same purpose. Have we included all the relevant changes out of the other PR? |
fdf30a4
to
1a64e30
Compare
@chef/docs-team Please take a look 👀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One change and a question.
The setting in the two files is slightly different ('700' vs '0700')-- is this intentional?
Signed-off-by: saghoshprogress <saghosh@progress.com>
Signed-off-by: Dheeraj Singh Dubey <dhsingh@progress.com>
Signed-off-by: Dheeraj Singh Dubey <dhsingh@progress.com>
1a64e30
to
99698a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Signed-off-by: saghoshprogress saghosh@progress.com
Added option to modify the log directory permission, default will remain the same.
Issues Resolved
Issues
Check List
https://buildkite.com/chef/chef-supermarket-main-omnibus-adhoc/builds/264#1c7d3371-3a9e-4019-a442-831b6b23c7a2