The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #264

file checksum option always uses md5-sums

Added by Redmine Admin almost 8 years ago. Updated over 4 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Luke Kanies% Done:

0%

Category:fileserving
Target version:-
Affected Puppet version:0.25.4 Branch:
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

From the Mailinglist: Subject: Re: [Puppet-users] strange mtime/md5sum behaviour and constantly

    changing files with links
> as I have some huge files to distribute and md5-sum-checking takes SOME time
> for them, I'm trying to switch to mtime/timestamps, but got no luck. Regularly
> the mtime seems to change and additionally the md5sums generate themselves
> anyway.

I'm pretty sure that the file copying subsystem always uses checksums
regardless of what you ask.  This is a bug.  I thought I'd developed
it to support any of them, but realized recently I had not.

Message-Id: 5136CF0F-446C-44FB-9CC6-1A2776F60374@madstop.com

thx for fixing,

Greetings,

        /fux

Related issues

Related to Puppet - Bug #1564: Saving File#checksum to state.yaml broken Closed 09/07/2008

History

#1 Updated by Redmine Admin almost 8 years ago

Checksumming with md5-files doesn’t seem the only problem here, the checksum-type “mtime” seems to have a “bug”:

On every sync, the mtime of the target file gets changed, if puppet compares the source-file (mtime some years ago) with the target-file (some 5 minutes ago), it has to resync that file. mtime only seems to make sense, if the corresponding time gets set by puppet after the sync, otherwise we shouldn’t support mtime as checksum-type, I guess.

Another problem derived from this bugreport is the one luke mentioned as “pretty shure it is a bug”. In source:/trunk/lib/puppet/type/pfile/checksum.rb@1760#L240 the corresponding checksum-type should be used, I guess, not a default. But for now I’m to tired to create a patch, or debug any futher, sorry :)

here is my “testcase”, with which I debugged the problem:

$ cat site.pp
file { "/home/thorsten/copy": 
        source => "/home/thorsten/huge",
        checksum => "mtime"
}

#2 Updated by Redmine Admin about 6 years ago

  • Status changed from 1 to Accepted

#3 Updated by Luke Kanies over 4 years ago

  • Status changed from Accepted to Closed

Nearly positive this works correctly now, or at least will in rowlf.

Also available in: Atom PDF