Issue #18110 has been updated by Erik Dalén.
It seems like line 605 in puppet/type/file.rb (on 3.1.0):
self.fail "Could not back up; will not replace" unless perform_backup
actually does the backups (and failing if they failed). And then later on the
check if we used force or not is done (on line 613..620):
if self[:force] == :true
debug "Removing existing directory for replacement with #{should}"
FileUtils.rmtree(self[:path])
else
notice "Not removing directory; use 'force' to override"
return
end
----------------------------------------
Bug #18110: Recursive filebucket backup when not needed
https://projects.puppetlabs.com/issues/18110#change-84098
Author: Erik Dalén
Status: Needs More Information
Priority: Normal
Assignee: Erik Dalén
Category: filebucket
Target version:
Affected Puppet version: 3.0.1
Keywords: file link filebucket recursive backup force false
Branch:
When I do something like this (fictive example):
<pre>
filebucket { 'puppet':
path => false,
server => $::servername,
}
file { '/tmp':
ensure => link,
target => '/var/tmp/',
force => false,
backup => puppet,
}
</pre>
Then puppet will start a recursive backup on /tmp and then after that figure
out that it won't replace the directory with a link without force being true.
It shouldn't do any filebucketing at all here as it is not deleting or changing
any files.
--
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.
