Puppet: System Administration Automated

Support

Ticket #511: fix-511.patch

File fix-511.patch, 1.8 kB (added by DavidS, 1 year ago)

adds a test and always converts the timestamp to utc

  • a/lib/puppet/type/pfile/checksum.rb

    old new  
    161161                if ! FileTest.file?(@resource[:path]) 
    162162                    @resource.debug "Cannot MD5 sum %s; using mtime" % 
    163163                        [@resource.stat.ftype] 
    164                     sum = @resource.stat.mtime.to_s 
     164                    sum = @resource.stat.mtime.utc.to_s 
    165165                else 
    166166                    begin 
    167167                        File.open(@resource[:path]) { |file| 
     
    186186                    end 
    187187                end 
    188188            when :timestamp, :mtime: 
    189                 sum = @resource.stat.mtime.to_s 
    190                 #sum = File.stat(@resource[:path]).mtime.to_s 
     189                sum = @resource.stat.mtime.utc.to_s 
     190                #sum = File.stat(@resource[:path]).mtime.utc.to_s 
    191191            when :time: 
    192                 sum = @resource.stat.ctime.to_s 
    193                 #sum = File.stat(@resource[:path]).ctime.to_s 
     192                sum = @resource.stat.ctime.utc.to_s 
     193                #sum = File.stat(@resource[:path]).ctime.utc.to_s 
    194194            else 
    195195                raise Puppet::Error, "Invalid sum type %s" % checktype 
    196196            end 
  • a/test/ral/types/file.rb

    old new  
    434434                assert(file.property(:checksum).insync?(sum), 
    435435                    "checksum is not in sync") 
    436436 
     437                assert(/ UTC /.match(sum), 
     438                        "checksum not in UTC timezone, but is '%s'" % sum ) if type =~ /time/  
     439 
    437440                sleep 1.1 if type =~ /time/ 
    438441                assert_nothing_raised() { 
    439442                    File.unlink(path)