Issue #4922 has been updated by Markus Roberts.
Status changed from Needs more information to Accepted
Target version set to 2.6.x
I'm inclined to agree ("suboptimal", BTW, was intended as understatement). So
I think the key question in my mind (and, if it turns out I'm wrong about this
issue, the point I'm most likely to be missing) is:
Is there a reason for us to be rescuing 404s as nil rather than failing? In
other words, is there a case where we want to continue replacing a file after
we've failed to get the contents?
(Note that although I'm agreeing that this is a problem it is, so far as I can
see, very unlikely to occur in practice; with the present code the only way I
can see that you could generate such as situation would be with a complex &
mis-configured load balancing between puppetmasters which would also be
severely broken in other ways).
----------------------------------------
Bug #4922: Puppetd creates empty files, if it gets a 404
http://projects.puppetlabs.com/issues/4922
Author: Nico Schottelius
Status: Accepted
Priority: Normal
Assignee:
Category:
Target version: 2.6.x
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.
