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.

Sign up for GitHub

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

Jump to bottom

fix zookeeper logs issue #2022

Merged
merged 2 commits into from
Mar 31, 2017
Merged

fix zookeeper logs issue #2022

merged 2 commits into from
Mar 31, 2017

Conversation

riuvshin
Copy link

What does this PR do?

to fix permissions issue we have to create folder with 777 permissions before volume mount.

What issues does this PR fix or reference?

#2021

Changelog

fix zookeeper logs issue

@riuvshin riuvshin added this to the 5.7.0 milestone Mar 31, 2017
skryzhny
skryzhny requested changes Mar 31, 2017
View reviewed changes
Copy link
Contributor

@skryzhny skryzhny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Making directory world writeable is a quick hack, need to change owner of it to right one, and keep default permissions like 755 or 775 or even 750

@riuvshin riuvshin merged commit 816613d into master Mar 31, 2017
@riuvshin riuvshin deleted the fix_zookeeper_logs branch March 31, 2017 13:45
@JamesDrummond JamesDrummond mentioned this pull request Apr 9, 2017
Closed
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skryzhny skryzhny skryzhny approved these changes

@garagatyi garagatyi garagatyi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
5.7.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@riuvshin @skryzhny @garagatyi