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.

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

Fix directory permissions regression #73

Merged
merged 1 commit into from
Jul 28, 2015

Conversation

sbcas
Copy link
Contributor

@sbcas sbcas commented Jun 26, 2015

Due to recipe ordering, /var/log/mysql does not get the correct permissions applied before the mariadb-server package gets installed, resulting in a scenario where MariaDB will not start.

@jjasghar
Copy link

👍 this is gating a downstream fix for the opetstack+chef builds. Please take a look at this PR asap :D

@SolverDude
Copy link

+1. I encountered this during personal openstack testing on CentOS, and this fix corrected my issue.

@jjasghar
Copy link

jjasghar commented Jul 6, 2015

Any chance we can get this merged?

@jjasghar
Copy link

@sinfomicien any chance we can get this merged? (yep, we have weekly meetings on this and i'll keep commenting every week till its done 🤘 )

@jklare
Copy link

jklare commented Jul 13, 2015

+1

@jjasghar
Copy link

Any chance this can get merged?

@jjasghar
Copy link

Found him on Twitter, tweeted at him too.

https://twitter.com/thewhitegeek

@jjasghar
Copy link

Another ping, can we get this merged please?

@kramvan1
Copy link
Contributor

👍

@sinfomicien
Copy link
Contributor

I just came from a long holiday! Sorry to be late... But to be honnest, it was long for you, but too short for me :) I take a look on the PR.

sinfomicien added a commit that referenced this pull request Jul 28, 2015
Fix directory permissions regression
@sinfomicien sinfomicien merged commit a22d1cb into sous-chefs:master Jul 28, 2015
@sinfomicien
Copy link
Contributor

Thx for your PR

@sbcas
Copy link
Contributor Author

sbcas commented Jul 28, 2015

Thanks for merging. Could we get a new version pushed to Supermarket?

@sinfomicien
Copy link
Contributor

I actually run all kitchen tests...After merging all PRs i could. And there's some regression bugs to fix. I think we can have a working version on supermarket by the end of tomorrow.

@lock
Copy link

lock bot commented Jul 24, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants