Issue #18333 has been updated by Robin Powell.
I'll note that (1) File recurse *checksums the things symlinks point to and (2) md5 checksumming is horrendously expensive. I have a pair of files at the end of symlinks that Puppet sets up that between them take ~200 seconds to check. Not cool. So a fix would be really nice. -Robin ---------------------------------------- Bug #18333: checksum => 'mtime' fails https://projects.puppetlabs.com/issues/18333#change-98677 * Author: Dustin Mitchell * Status: Investigating * Priority: Normal * Assignee: Josh Cooper * Category: * Target version: * Affected Puppet version: * Keywords: * Branch: ---------------------------------------- Per #13199, checksums other than md5 don't work with recurse => true, so I've disabled recursion and enumerated the files in my manifest. It still doesn't work. Here's the boiled-down test case: node "mobile-imaging-010.p10.releng.scl1.mozilla.com" { file { "/opt/bmm/www/squashfs/precise-panda-live-build.2012120602.squashfs": checksum => 'mtime', source => "puppet:///bmm/private/squashfs/precise-panda-live-build.2012120602.squashfs"; } } the result is err: /Stage[main]//Node[mobile-imaging-010.p10.releng.scl1.mozilla.com]/File[/opt/bmm/www/squashfs/precise-panda-live-build.2012120602.squashfs]/content: change from {mtime}Thu Jan 03 06:58:03 -0800 2013 to {md5}1006d8c8d337ca8d54a4c446b857f60c failed: Could not rename temporary file /opt/bmm/www/squashfs/precise-panda-live-build.2012120602.squashfs.puppettmp_3471 to /opt/bmm/www/squashfs/precise-panda-live-build.2012120602.squashfs: File written to disk did not match checksum; discarding changes ({mtime} vs {mtime}Thu Jan 03 07:15:47 -0800 2013) at /etc/puppet/environments/dmitchell/env/manifests/nodes.pp:74 at /etc/puppet/environments/dmitchell/env/manifests/nodes.pp:74 and, checking the Apache logs, it's requesting an md5 checksum: 10.12.137.33 - - [03/Jan/2013:07:15:19 -0800] "GET /dmitchell/file_metadata/bmm/private/squashfs/precise-panda-live-build.2012120602.squashfs? HTTP/1.1" 200 333 "-" "-" That's a pretty narrow test case. Is there any way that checksum => 'mtime' *does* work? -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
