[yocto] [PATCH 2/2] buildslave-janitor: Add in try/except handling to make calls robust against failure

2014-03-19 Thread Richard Purdie
There is a suspicion that the command can fail causing the janitor trash removal process to exit. This adds in some exception handling to deal with this. It include a timeout to stop it entering nasty loops. This combined with the previous patch should let us figure out what error conditions its

Re: [yocto] [PATCH 2/2] buildslave-janitor: Add in try/except handling to make calls robust against failure

2014-03-19 Thread Burton, Ross
Both patches look good, Reviewed-by: Ross Burton ross.bur...@intel.com Ross On 19 March 2014 14:03, Richard Purdie richard.pur...@linuxfoundation.org wrote: There is a suspicion that the command can fail causing the janitor trash removal process to exit. This adds in some exception handling to

Re: [yocto] [PATCH 2/2] buildslave-janitor: Add in try/except handling to make calls robust against failure

2014-03-19 Thread Flanagan, Elizabeth
Agreed. Pulling them. Thanks. -b On Wed, Mar 19, 2014 at 7:28 AM, Burton, Ross ross.bur...@intel.com wrote: Both patches look good, Reviewed-by: Ross Burton ross.bur...@intel.com Ross On 19 March 2014 14:03, Richard Purdie richard.pur...@linuxfoundation.org wrote: There is a suspicion