-
Notifications
You must be signed in to change notification settings - Fork 43
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
fixes #17504, #15619 - use generated puppet.conf when running Puppet #197
Conversation
'hiera_config' => hiera.config_path, | ||
'noop' => !!noop?, | ||
'profile' => !!profile?, | ||
'show_diff' => false, |
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.
based on jenkins finding, trailing comma does not work in Ruby 1.8
def write_config | ||
@logger.debug("Writing Puppet config file at #{@temp_file.path}") | ||
@temp_file.open | ||
@temp_file.truncate(0) |
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.
just out of curiosity, is this because of race condition even when using Tempfile
?
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.
It's in case the method is called twice so the file gets overwritten fully.
Except the 1.8 issue it looks good. |
c1eafe9
to
954f32f
Compare
PR updated with the trailing comma removed. Edit: and unpredictable ordering of the settings in puppet.conf under Ruby 1.8.7 fixed, now sorted by setting name. |
954f32f
to
af7aedc
Compare
Thanks @domcleal, merging. |
No description provided.