Last modified: 2014-06-03 05:28:45 UTC
Created attachment 15525 [details] Puppet log For completeness, I'll post a few things, including the full error and what roles I have enabled. Short version is: You cannot specify more than one of content, source, target at /tmp/vagrant-puppet-3/modules-0/apache/manifests/conf.pp:59 What's weird is that seems to be old code from https://gerrit.wikimedia.org/r/#/c/78401/ , but the provisioning error only occurs if simple_performant is enabled.
Created attachment 15526 [details] Roles enabled when provisioning fails
I'd go for the silliest suspects first: like, let's change the name of either: apache::mod { 'expires': } [...] apache::conf { 'expires':
Nope, wishful thinking didn't help, that's not it; I get the error anyway on labs-vagrant.
Well we do have two apache::conf as no other role: include role::thumb_on_404 hence apache::conf { 'thumb.php on 404': ... + apache::conf { 'expires': ... which then go to $config_file = inline_template('<%= @title.gsub(/\W/, "-") %>') file { "${config_dir}/${config_file}${config_extension}": Do we just have to make ${config_file} different for the two?
No, that's not a problem, since the filename is auto-generated form the title (e.g. 'expires' is one of the titles here). Actually, the error text is accurate. The problem is two-fold: 1. It's referring to a non-existent 'puppet:///files/expires' file (this file was not added with the commit). 2. Either content (text to put in the file) or source (filename to copy to file) is needed, not both. Content is already there. I'm testing the fix right now (it's taking a little while on the localization rebuild), but it's gotten much farther than before.
Change 136961 had a related patch set uploaded by Mattflaschen: Remove unneeded source attribute (content used instead) https://gerrit.wikimedia.org/r/136961
Works fine, although I found bug 66061.
Change 136961 merged by jenkins-bot: Remove unneeded source attribute (content used instead) https://gerrit.wikimedia.org/r/136961