Puppet

Apparently, adding a ‘\’ to the end of each line makes it function properly.

However, this is not a good solution.

I have the same or similar problem, but since i use here-documents, ‘\’ at end of line is not an option.

I have a define, which basically does a /bin/cat of an here-document to a file. It has always worked until puppet-2.7.3 included, but fails with 2.7.4 and 2.7.5. It uses parameter variables, but i can see in debug mode that they are correctly known and expanded.

Here is the define:

define filesection($file, $sectionname, $lines='', $ensure = 'present') {
   case $ensure {
      present: {
         exec { "exec add ${sectionname} $file":
            path => ["/bin", "/usr/bin" ],
            command => "/bin/cat <<EOF >>'${file}'

## PUPPET BEGIN ${sectionname}
${lines}
## PUPPET END ${sectionname}
EOF",
            unless => "grep -qFx '## PUPPET BEGIN ${sectionname}' '${file}'"
         }
      }
# (...)
   }
}

Here is the calling line:

filesection { 'bashrc-shopt-expand_aliases':
    file        => '/etc/bashrc',
    sectionname => 'shopt-expand_aliases',
    lines       => 'shopt -s expand_aliases',
}

And here is the behaviour of different Puppet versions:

2.7.3:

notice: /Stage[os]/Bashrc::Shopt::Expand_aliases/Filesection[bashrc-shopt-expand_aliases]/Exec[exec add shopt-expand_aliases /etc/bashrc]/returns: executed successfully

2.7.4 and 2.7.5 rpms:

debug: Exec[exec add shopt-expand_aliases /etc/bashrc](provider=posix): Executing check 'grep -qFx '## PUPPET BEGIN shopt-expand_aliases' '/etc/bashrc''
debug: Executing 'grep -qFx '## PUPPET BEGIN shopt-expand_aliases' '/etc/bashrc''
debug: Exec[exec add shopt-expand_aliases /etc/bashrc](provider=posix): Executing '/bin/cat <<EOF >>'/etc/bashrc'

## PUPPET BEGIN shopt-expand_aliases
shopt -s expand_aliases
## PUPPET END shopt-expand_aliases
EOF'
debug: Executing '/bin/cat <<EOF >>'/etc/bashrc'

## PUPPET BEGIN shopt-expand_aliases
shopt -s expand_aliases
## PUPPET END shopt-expand_aliases
EOF'
err: /Stage[os]/Bashrc::Shopt::Expand_aliases/Filesection[bashrc-shopt-expand_aliases]/Exec[exec add shopt-expand_aliases /etc/bashrc]/returns: change from notrun to 0 failed: /bin/cat <<EOF >>'/etc/bashrc'

## PUPPET BEGIN shopt-expand_aliases
shopt -s expand_aliases
## PUPPET END shopt-expand_aliases
EOF returned 1 instead of one of [0] at /etc/puppet/environments/production/manifests/classes/filesection.pp:15

Thus breaking my sed 2liners completely. Thanks for that.

This should fix things:

diff --git i/lib/puppet/util.rb w/lib/puppet/util.rb
index 80ce963..a264a0a 100644
--- i/lib/puppet/util.rb
+++ w/lib/puppet/util.rb
@@ -244,7 +244,12 @@ module Util

   def execute_posix(command, arguments, stdin, stdout, stderr)
     child_pid = Kernel.fork do
-      command = Array(command)
+      # We can't just call Array(command), and rely on it returning
+      # things like ['foo'], when passed ['foo'], because
+      # Array(command) will call command.to_a internally, which when
+      # given a string can end up doing Very Bad Things(TM), such as
+      # turning "/tmp/foo;\r\n /bin/echo" into ["/tmp/foo;\r\n", " /bin/echo"]
+      command = [command].flatten
       Process.setsid
       begin
         $stdin.reopen(stdin)