Issue #8119 has been updated by Daniel Pittman. Assignee changed from James Turnbull to Patrick Carlisle
Patrick Carlisle wrote: > Why was this retargeted at Telly? I'm ready to merge, but waiting on that > since it was originally targeted at 2.7.x and that still makes sense to me. I don't recall, and foolishly didn't record whatever conversation led to that decision. I don't see any problem that prevents this applying inside the 2.7 series, assuming that the code is implementation only and there are no user visible effects of it. (Other than, say, less races ;) ---------------------------------------- Bug #8119: PATCH - Write to a temporary file and move when storing reports. https://projects.puppetlabs.com/issues/8119 Author: Ricky Zhou Status: In Topic Branch Pending Review Priority: Normal Assignee: Patrick Carlisle Category: reports Target version: Telly Affected Puppet version: Keywords: Branch: When writing reports, there is a window in between opening and writing to the report file when the report file exists as an empty file. This makes writing report processors a little annoying as they have to deal with this case. This patch writes the report into a temporary file then renames it to the report file, which eliminates this window. http://ricky.fedorapeople.org/puppet/0001-Write-to-a-temporary-file-and-move-when-storing-repo.patch -- 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.
