Issue 4641 - Append performance fix to trac cgi file

modules/trac/manifests/init.pp

Index: modules/trac/manifests/init.pp
===================================================================
--- a/modules/trac/manifests/init.pp
+++ b/modules/trac/manifests/init.pp
@@ -233,7 +233,7 @@
         Exec['install_TracXMLRPC'],
         Exec['install_TracHTTPAuth']],
     }
-  
+
     exec {"deploy_$name":
       command => "trac-admin /home/trac/$environment \
         deploy /home/trac/htdocs-$name \
@@ -245,7 +245,22 @@
         Exec["update_env_$name"],
         Package["tofrodos"]],
     }
-  
+
+    file {"/home/trac/htdocs-$name/cgi-bin/trac_performance_fix.snippet":

[mathias, 2016/11/17 07:43:31]

Why not using a target path like
/usr/local/lib/python2.7/dist-packages/trac_monkey_patches.py or similar? That
would allow for patching using a simple include statement, which can be inserted
with a file_line resource (from Puppetlabs stdlib).

[f.nicolaisen, 2016/11/17 08:23:46]

Nice. Yeah, we can try doing that instead of exec/sed. I don't think file_line
can read contents from files though, so the snippet will have to be a value
inside this file again.

[mathias, 2016/11/17 10:18:33]

It should be possible, something like this one:

    file_line {"patch $name trac.fcgi":
      after => "^# Author:",
      line => "import trac_monkey_patches",
      path => "/home/trac/htdocs-$name/cgi-bin/trac.fcgi",
    }

[f.nicolaisen, 2016/11/17 10:54:31]

OK, I'll try. Thanks!

+      ensure => present,
+      source => 'puppet:///modules/trac/trac_performance_fix.snippet',
+      owner => trac,
+      require => Exec["deploy_$name"],
+    }
+
+    exec {"append_cgi_config_to_$name":

[mathias, 2016/11/17 07:43:31]

It isn't really appending any more.

[f.nicolaisen, 2016/11/17 08:23:46]

True that, will fix.

+      command        => "sed -i '/# Author: Jonas Borgström <jonas@edgewall.com>/r trac_performance_fix.snippet' trac.fcgi",
+      cwd            => "/home/trac/htdocs-$name/cgi-bin/",
+      user           => trac,
+      path           => '/usr/bin:/usr/sbin:/bin:/usr/local/bin',
+      require        => File["/home/trac/htdocs-$name/cgi-bin/trac_performance_fix.snippet"],

[Wladimir Palant, 2016/11/16 17:02:52]

This should also require Exec["deploy_$name"], it cannot run earlier than
deployment.

[f.nicolaisen, 2016/11/16 17:04:55]

The File has that require, so this is transitively already the case. Should I
add the same require here for the sake of clarity?

[Wladimir Palant, 2016/11/16 18:57:32]

Ah, I didn't notice this.

+    }
+
     file {"/home/trac/htdocs-$name/htdocs/common/logo.png":
       ensure => present,
       source => $logo,