Re: [PATCH 1/1] report-error.bbclass: Added file syncronization.

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 876 bytes --]

On Fri, Jun 26, 2015 at 04:17:01PM +0100, Burton, Ross wrote:
> On 26 June 2015 at 06:56, <mariano.lopez@linux.intel.com> wrote:
> 
> > errorreport_handler would fail if several errors are
> > triggered at the same time because of two proccess
> > writting to the same file. This patch add the required
> > syncronization to handle concurrent process.
> >
> 
> The code in the autobuilder that sends error reports doesn't care for the
> filenames, just that they're under error-reports/.  Instead of locking
> around a single file - which surely will replace errors with lost reports -
> just write each report to a unique filename.

This is about multiple TaskFailed events being processed at the same
time, isn't it? So it's still part of the same report and unrelated to
uploading to web app.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]