diff options
author | Alexandru DAMIAN <alexandru.damian@intel.com> | 2015-03-12 12:44:05 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-03-21 00:00:23 +0000 |
commit | a273d8f1510a8a711e5f2b28b51f058cffd4bff7 (patch) | |
tree | cb421d9018506afef0b7bf54ce691aac2e00a2e0 /bitbake/lib/bb/ui | |
parent | c5e9654ac908537ad0b14e0804be312e7c6f5be8 (diff) | |
download | ast2050-yocto-poky-a273d8f1510a8a711e5f2b28b51f058cffd4bff7.zip ast2050-yocto-poky-a273d8f1510a8a711e5f2b28b51f058cffd4bff7.tar.gz |
bitbake: toasterui: improve info in the toaster_ui.log
We improve logging and signalling of errors in the
toaster_ui.log to facilitate debugging on remote systems.
(Bitbake rev: 3cd248f99b90367bd41aab81e255fc1912434890)
Signed-off-by: Alexandru DAMIAN <alexandru.damian@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/bb/ui')
-rw-r--r-- | bitbake/lib/bb/ui/buildinfohelper.py | 12 | ||||
-rw-r--r-- | bitbake/lib/bb/ui/toasterui.py | 8 |
2 files changed, 16 insertions, 4 deletions
diff --git a/bitbake/lib/bb/ui/buildinfohelper.py b/bitbake/lib/bb/ui/buildinfohelper.py index cf8c444..b511abe 100644 --- a/bitbake/lib/bb/ui/buildinfohelper.py +++ b/bitbake/lib/bb/ui/buildinfohelper.py @@ -33,6 +33,7 @@ from toaster.orm.models import Task_Dependency, Package_Dependency from toaster.orm.models import Recipe_Dependency from bb.msg import BBLogFormatter as format from django.db import models +from pprint import pformat import logging from django.db import transaction, connection @@ -282,7 +283,6 @@ class ORMWrapper(object): # we might have a race condition here, as the project layers may change between the build trigger and the actual build execution # but we can only match on the layer name, so the worst thing can happen is a mis-identification of the layer, not a total failure - from pprint import pformat # note that this is different buildrequest = BuildRequest.objects.get(pk = br_id) for brl in buildrequest.brlayer_set.all(): @@ -467,7 +467,7 @@ class ORMWrapper(object): Package_Dependency.objects.bulk_create(packagedeps_objs) if (len(errormsg) > 0): - raise Exception(errormsg) + logger.warn("buildinfohelper: target_package_info could not identify recipes: \n%s" % errormsg) def save_target_image_file_information(self, target_obj, file_name, file_size): target_image_file = Target_Image_File.objects.create( target = target_obj, @@ -885,9 +885,13 @@ class BuildInfoHelper(object): assert localfilepath.startswith("/") recipe_information = self._get_recipe_information_from_taskfile(taskfile) - recipe_object = Recipe.objects.get(layer_version = recipe_information['layer_version'], + try: + recipe_object = Recipe.objects.get(layer_version = recipe_information['layer_version'], file_path__endswith = recipe_information['file_path'], name = recipename) + except Recipe.DoesNotExist: + logger.error("Could not find recipe for recipe_information %s name %s" % (pformat(recipe_information), name)) + raise task_information = {} task_information['build'] = self.internal_state['build'] @@ -1127,7 +1131,7 @@ class BuildInfoHelper(object): Task_Dependency.objects.bulk_create(taskdeps_objects) if (len(errormsg) > 0): - raise Exception(errormsg) + logger.warn("buildinfohelper: dependency info not identify recipes: \n%s" % errormsg) def store_build_package_information(self, event): diff --git a/bitbake/lib/bb/ui/toasterui.py b/bitbake/lib/bb/ui/toasterui.py index df9f362..db119ce 100644 --- a/bitbake/lib/bb/ui/toasterui.py +++ b/bitbake/lib/bb/ui/toasterui.py @@ -327,9 +327,17 @@ def main(server, eventHandler, params ): except Exception as e: # print errors to log import traceback + from pprint import pformat exception_data = traceback.format_exc() logger.error("%s\n%s" % (e, exception_data)) + exc_type, exc_value, tb = sys.exc_info() + if tb is not None: + curr = tb + while curr is not None: + logger.warn("Error data dump %s\n%s\n" % (traceback.format_tb(curr,1), pformat(curr.tb_frame.f_locals))) + curr = curr.tb_next + # save them to database, if possible; if it fails, we already logged to console. try: buildinfohelper.store_log_exception("%s\n%s" % (str(e), exception_data)) |