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 config append #782

Merged
merged 2 commits into from
Mar 29, 2021
Merged

Conversation

LadyNamedLaura
Copy link

This should fix #759.
I start by reverting #707 and re-implementing a fix for it. the actual fix was actually quiet easy, just by adding a concat snippet at the end of each config block with a newline.

@LadyNamedLaura LadyNamedLaura force-pushed the fix-config-append branch 2 times, most recently from c425e2b to ed77327 Compare March 25, 2021 20:10
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Tested on me own infra and this works. Thanks!

@@ -73,6 +73,12 @@
puppet::config::main { $key: value => $value }
}

concat::fragment { 'puppet.conf_comment':
target => "${puppet_dir}/puppet.conf",
content => '# file managed by puppet',
Copy link
Member

Choose a reason for hiding this comment

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

This is a nice solution to the otherwise empty line.

@ekohl ekohl merged commit f1255d9 into theforeman:master Mar 29, 2021
@wbclark wbclark added the Bug label Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Appending values with puppet::config::entry is broken
4 participants