Issue #8119 has been updated by Ricky Zhou.
Yeah - the use case is that we're using a script in a cronjob to parse reports - the script just reads the newest report file in each directory. However, if the script happens to run at the same time as puppet is being run, it'll sometimes see an empty file because puppet hasn't had a chance to write it yet. Writing to a temporary file and moving it in place would ensure this can't happen. ---------------------------------------- Bug #8119: PATCH - Write to a temporary file and move when storing reports. https://projects.puppetlabs.com/issues/8119 Author: Ricky Zhou Status: Needs More Information Priority: Normal Assignee: Category: reports Target version: 2.7.x 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.
