Issue #18333 has been updated by eric sorenson.

The problem appears to be that `lib/puppet/indirector/file_server.rb` is 
ultimately called for both actual content serving as well as metadata serving, 
but does not make use of the query parameters on the request.
----------------------------------------
Bug #18333: checksum => 'mtime' fails
https://projects.puppetlabs.com/issues/18333#change-84090

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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to