9293 2016-03-17 17:27:38 +0000 bitbake: Add support for trigger event on errors like exceptions 2016-07-20 19:35:21 +0000 1 1 5 Infrastructure AutoBuilder autobuilder 2.2 x86 Multiple VERIFIED FIXED High enhancement 2.2 M1 1 anibal.limon anibal.limon benjamin.esquivel infras.ab.watcher Infras.watcher jose.perez.carranza joshuagloe pidge poky.bs.watcher poky.watcher libertad.gonzalez.de.la.cruz --- --- No (bug/feature does not impact docs) oldest_to_newest 60109 0 anibal.limon 2016-03-17 17:27:38 +0000 We need a mechanism for trigger events in error scenarios like exceptions, currently we have bb.build.TaskFailed event when tasks fails. It could be done adding event trigger when call bb.error(). The performance needs to be evaluated to don't slow build times. 60492 1 joshuagloe 2016-03-29 15:59:58 +0000 Is it truly necessary to integrate this into the core of bitbake? Perhaps we would be better served by providing a wrapper which catches the non-zero exit codes and performs error reporting. This way we make the feature (and any negative impact) opt-in and furthermore don't complicate the core bitbake code base unnecessarily. 60493 2 anibal.limon 2016-03-29 16:09:30 +0000 (In reply to comment #1) > Is it truly necessary to integrate this into the core of bitbake? Perhaps we > would be better served by providing a wrapper which catches the non-zero > exit codes and performs error reporting. > > This way we make the feature (and any negative impact) opt-in and > furthermore don't complicate the core bitbake code base unnecessarily. This bug was filed by me for enable support for send generic errors to error-report web, currently the error-report class looks into task errors triggered by TaskFailed event. Certainly we should take into account the performance metrics and the complexity of the implementation. I'll try to implement this approach but if it becomes too difficult to implement/maintain/performance then i'll change the implementation to review the retcode and if error parse output/logs but this requires another changes in the AB infrastructure also to get rid/move to script error-report class. 62521 3 anibal.limon 2016-05-30 16:01:41 +0000 We decided that is better to have a process to monitor stdout of bitbake process and looks for exception/errors instead of modify bitbake code base for do it. The current implementation is at level of Autobuilder and is almost finish, see [1]. [1] http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/log/?h=contrib/alimon/devel 63235 4 anibal.limon 2016-06-21 23:12:16 +0000 This support was implemented inside of Yocto Autobuilder that's the reason for move to Autobuilder and now is on review at, https://lists.yoctoproject.org/pipermail/yocto/2016-June/030742.html Beth ack'ed the first part of changes so only need to review the v2. 63402 5 anibal.limon 2016-06-28 14:56:03 +0000 The patches are integrated at, http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=beafae04c9633e4330be588ab0490927f701127a http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=57beefdf33491730155e1a2184895cc82ef95c0a http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=e1e7c3b86720786ae4f60da008f68d4cf5d58559 The remaining work is the deployment into the AB cluster. 64249 6 libertad.gonzalez.de.la.cruz 2016-07-20 19:35:21 +0000 From the nightly autobuilder you can see that in https://autobuilder.yoctoproject.org/main/builders/nightly-oe-selftest/builds/602 the error report is collected and sent.