[PATCH 0/3] Pass BRBE to buildinfohelper as soon as it's set

[PATCH 0/3] Pass BRBE to buildinfohelper as soon as it's set

[Ed Bartosh]

Hi,

This patchset is a replacement of 3 patches that were rejected yesterday
by upstream.

It fixes 'hanging' builds in UI when build is failed on early stages (recipe
parsing or processing configuration).

The following changes since commit befaceb2873ff8c8cc791d48f0324442c15cbcdc:

  toaster: return back 'New project' button (2015-12-08 13:10:33 +0200)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ed/toaster/trigger-metadata-event
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/trigger-metadata-event

Ed Bartosh (3):
  toaster: implement BitbakeController.triggerEvent
  toaster: trigger SetBRBE event
  toasterui: process SetBRBE event

 bitbake/lib/bb/ui/toasterui.py                          | 2 ++
 bitbake/lib/toaster/bldcontrol/bbcontroller.py          | 3 +++
 bitbake/lib/toaster/bldcontrol/localhostbecontroller.py | 3 +++
 3 files changed, 8 insertions(+)

--
Regards,
Ed

[PATCH 1/3] toaster: implement BitbakeController.triggerEvent

[Ed Bartosh]

This API will be used to pass data from Toaster to
toaster_ui and buildinfohelper.

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 bitbake/lib/toaster/bldcontrol/bbcontroller.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/bitbake/lib/toaster/bldcontrol/bbcontroller.py b/bitbake/lib/toaster/bldcontrol/bbcontroller.py
index 8b99318..781ff73 100644
--- a/bitbake/lib/toaster/bldcontrol/bbcontroller.py
+++ b/bitbake/lib/toaster/bldcontrol/bbcontroller.py
@@ -55,6 +55,9 @@ class BitbakeController(object):
     def getVariable(self, name):
         return self._runCommand(["getVariable", name])
 
+    def triggerEvent(self, event):
+        return self._runCommand(["triggerEvent", event])
+
     def build(self, targets, task = None):
         if task is None:
             task = "build"
-- 
2.1.4

[PATCH 2/3] toaster: trigger SetBRBE event

[Ed Bartosh]

Triggered MetadataEvent "SetBRBE" when TOASTER_BRBE variable
is set on bitbake server. This should make buildinfohelper
aware of the build request id, which is used to properly
report build status and failures back to Toaster.

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 bitbake/lib/toaster/bldcontrol/localhostbecontroller.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/bitbake/lib/toaster/bldcontrol/localhostbecontroller.py b/bitbake/lib/toaster/bldcontrol/localhostbecontroller.py
index 92d9ac5..00228e9 100644
--- a/bitbake/lib/toaster/bldcontrol/localhostbecontroller.py
+++ b/bitbake/lib/toaster/bldcontrol/localhostbecontroller.py
@@ -287,6 +287,9 @@ class LocalhostBEController(BuildEnvironmentController):
         # set variables
         for var in variables:
             bbctrl.setVariable(var.name, var.value)
+            if var.name == 'TOASTER_BRBE':
+                bbctrl.triggerEvent('bb.event.MetadataEvent("SetBRBE", "%s")' \
+                                     % var.value)
 
         # Add 'toaster' and 'buildhistory' to INHERIT variable
         inherit = {item.strip() for item in bbctrl.getVariable('INHERIT').split()}
-- 
2.1.4

[PATCH 3/3] toasterui: process SetBRBE event

[Ed Bartosh]

Currently BRBE is known to toasterui only when build is
started. It's passed to it with BuildStarted event. This is
too late as if build fails earilier than build starts toasterui
can not inform Toaster about the failure.

Set BRBE as soon as it's provided by Toaster.
This should make toasterui to be able to inform Toaster
about early build failures, e.g. failures during recipe parsing.

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 bitbake/lib/bb/ui/toasterui.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/bitbake/lib/bb/ui/toasterui.py b/bitbake/lib/bb/ui/toasterui.py
index 435cc2b..0265df7 100644
--- a/bitbake/lib/bb/ui/toasterui.py
+++ b/bitbake/lib/bb/ui/toasterui.py
@@ -363,6 +363,8 @@ def main(server, eventHandler, params):
                     buildinfohelper.update_artifact_image_file(event)
                 elif event.type == "LicenseManifestPath":
                     buildinfohelper.store_license_manifest_path(event)
+                elif event.type == "SetBRBE":
+                    buildinfohelper.brbe = buildinfohelper._get_data_from_event(event)
                 else:
                     logger.error("Unprocessed MetadataEvent %s ", str(event))
                 continue
-- 
2.1.4

Re: [PATCH 0/3] Pass BRBE to buildinfohelper as soon as it's set

[Richard Purdie]

On Tue, 2015-12-08 at 17:23 +0200, Ed Bartosh wrote:
> Hi,
> 
> This patchset is a replacement of 3 patches that were rejected yesterday
> by upstream.
> 
> It fixes 'hanging' builds in UI when build is failed on early stages (recipe
> parsing or processing configuration).
> 
> The following changes since commit befaceb2873ff8c8cc791d48f0324442c15cbcdc:
> 
>   toaster: return back 'New project' button (2015-12-08 13:10:33 +0200)
> 
> are available in the git repository at:
> 
>   git://git.yoctoproject.org/poky-contrib ed/toaster/trigger-metadata-event
>   http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/trigger-metadata-event
> 
> Ed Bartosh (3):
>   toaster: implement BitbakeController.triggerEvent
>   toaster: trigger SetBRBE event
>   toasterui: process SetBRBE event

This looks much better, thanks!

Cheers,

Richard

Re: [PATCH 0/3] Pass BRBE to buildinfohelper as soon as it's set

[Michael Wood]

On 08/12/15 16:12, Richard Purdie wrote:
> On Tue, 2015-12-08 at 17:23 +0200, Ed Bartosh wrote:
>> Hi,
>>
>> This patchset is a replacement of 3 patches that were rejected yesterday
>> by upstream.
>>
>> It fixes 'hanging' builds in UI when build is failed on early stages (recipe
>> parsing or processing configuration).
>>
>> The following changes since commit befaceb2873ff8c8cc791d48f0324442c15cbcdc:
>>
>>    toaster: return back 'New project' button (2015-12-08 13:10:33 +0200)
>>
>> are available in the git repository at:
>>
>>    git://git.yoctoproject.org/poky-contrib ed/toaster/trigger-metadata-event
>>    http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/trigger-metadata-event
>>
>> Ed Bartosh (3):
>>    toaster: implement BitbakeController.triggerEvent
>>    toaster: trigger SetBRBE event
>>    toasterui: process SetBRBE event
> This looks much better, thanks!
>
> Cheers,
>
> Richard
>

Submitted to bitbake and committed to toaster-next

Thanks,

Michael

[PATCH 0/3] Pass BRBE to buildinfohelper as soon as it's set

[Michael Wood]

This fixes 'hanging' builds in UI when build is failed on early stages (recipe
parsing or processing configuration).

See also https://lists.yoctoproject.org/pipermail/toaster/2015-December/003464.html

http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/toaster/trigger-metadata-event


Ed Bartosh (3):
  toaster: implement BitbakeController.triggerEvent
  toaster: trigger SetBRBE event
  toasterui: process SetBRBE event

 lib/bb/ui/toasterui.py                          | 2 ++
 lib/toaster/bldcontrol/bbcontroller.py          | 3 +++
 lib/toaster/bldcontrol/localhostbecontroller.py | 3 +++
 3 files changed, 8 insertions(+)

-- 
2.1.4