-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Fixed prepending of root_dir override to the other paths #38707
Conversation
ping @Ch3LL |
This fix is for |
Any thoughts on this? |
Sorry about the delay there, @alexbleotu. It's merged now. :] Thank you! |
(And yes, this will be merged forward into develop in the next few days.) |
Thanks @cachedout |
@alexbleotu I have had to revert this because many tests broke and it's blocking the release. We need to re-open this and take another crack at it. |
@cachedout sorry about that, will take another look tomorrow. The tests
were probably disabled on my system cause I had no failures. Unfortunately,
tests didn't run automatically on the branch for whatever reason.
…On 17 Jan 2017 23:18, "Mike Place" ***@***.***> wrote:
@alexbleotu <https://github.com/alexbleotu> I have had to revert this
because many tests broke and it's blocking the release. We need to re-open
this and take another crack at it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#38707 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AQOKgee0ivY4z5ubwLKv4EMNbLdVThuAks5rTUxegaJpZM4LhwDh>
.
|
@alexbleotu Yes, that was my fault. We had a Jenkins outage and I was in a rush with the release. Apologies. Thanks for taking another swing at this one. :] |
What does this PR do?
Fixed prepending of root_dir override to the other paths
What issues does this PR fix or reference?
#38663
Previous Behavior
root_dir
in config was added to defaultroot dir
, if default overrides existed in_syspaths.py
New Behavior
Just the
root_dir
in the config is used if defined, otherwise the defaultroot_dir
Tests written?
No (existing tests work with new code)
Please review Salt's Contributing Guide for best practices.