Issue #4922 has been updated by Brice Figureau.
Nico Schottelius wrote: > The reason why I opened the ticket is to show that the bug was not solved at > the origin, but only the symptom. I definitely hope that 2.6.1 does not have > that bug anymore, BUT even if, it seems that according to #4319, the fix is > only on the > server side. No the bug was client side. The client is sending a different url for metadata and content. It gets from the metadata that the file exist, but when fetching its content it was getting a 404. > Thus to test this (should probably be included as a test case in puppet), you > would do: > > - return 200 on metadata > - return 404 on content > - See whether client recognizes the problem and does not create empty files > > If you keep the "fix" as it is, the problem may reoccur later again. While I do agree that the file content downloading should be more defensive and not write any files in case of 404, I think the bug is fixed. ---------------------------------------- Bug #4922: Puppetd creates empty files, if it gets a 404 http://projects.puppetlabs.com/issues/4922 Author: Nico Schottelius Status: Needs more information Priority: Normal Assignee: Category: Target version: Affected version: 2.6.0 Keywords: Branch: bFull report can be found at http://www.nico.schottelius.org/blog/puppet-empties-new-and-existing-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 post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
