Issue #23247 has been reported by Tony Kolstee.

----------------------------------------
Bug #23247: Temporary file not cleaned up when download of file content fails
https://projects.puppetlabs.com/issues/23247

* Author: Tony Kolstee
* Status: Unreviewed
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
* Affected Puppet version: 3.2.4
* Keywords: 
* Branch: 
----------------------------------------
file.rb comments indicate that a temporary file is always used when deploying a 
new file of nonzero length.

Presumably the temporary file should be deleted in the event that the file 
operation fails, but in at least one circumstance the empty temporary file is 
being left in place.

I recently ran into a situation where a typo in the auth.conf on one of my 
puppet masters was causing clients to see ERROR 400 sporadically when trying to 
download file metadata and/or content. This was leaving an empty temporary file 
on the client. In my particular case, the file being deployed was in facts.d 
and was interacting with a facter bug (#22622) which causes facter to die when 
an empty executable is present in this directory. This interaction was 
effectively disabling both puppet client and facter until the empty temporary 
file was manually deleted.

I know this behavior is present in puppet-3.2.4, but am not aware whether it 
exists in other versions.


-- 
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.

Reply via email to