Bug 868026 - failure to install task RPM is not properly reflected in task results
Summary: failure to install task RPM is not properly reflected in task results
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: beah
Version: 0.9
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: 0.11
Assignee: Amit Saha
QA Contact: Nick Coghlan
URL:
Whiteboard: Misc
: 877869 (view as bug list)
Depends On:
Blocks: 593663
TreeView+ depends on / blocked
 
Reported: 2012-10-18 22:34 UTC by Dan Callaghan
Modified: 2018-02-06 00:41 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-01-17 04:34:23 UTC
Embargoed:


Attachments (Terms of Use)

Description Dan Callaghan 2012-10-18 22:34:15 UTC
If the rpm database becomes corrupted somehow, such that beah cannot install task RPMs, the task result in Beaker appears as "Completed New" which is not really sensible. The harness should at least report the task as "Completed Failed" or better yet, simply abort the whole recipe.

Comment 2 Marian Csontos 2012-10-19 09:09:10 UTC
The task should have been aborted, which used to work fine.
What does the job-results show?

Comment 4 Dan Callaghan 2012-11-26 01:32:17 UTC
*** Bug 877869 has been marked as a duplicate of this bug. ***

Comment 9 Amit Saha 2012-12-12 11:54:08 UTC
As the task logs of the original bug report show, although the abort event was generated by beah, it wasn't handled correctly. Discussing with Marian Csontos, we inferred that the problem was possibly in the beaker backend - beakerlc.py. If the async_proc (on end event) sets the task to finished before
proc_evt_abort gets to it, it will return without aborting the
task. This is what possibly happened here. Hence, checking for the completed status of the task should prevent this from happening.

I couldn't ofcourse reproduce the problem when I run the job for a couple of times.

On Gerrit: http://gerrit.beaker-project.org/#/c/1563/

Comment 12 Nick Coghlan 2013-01-10 05:27:38 UTC
We were never able to reproduce this - the change was made by analysis. Testing indicates that this change at least did not introduce any regressions.

Comment 13 Dan Callaghan 2013-01-17 04:34:23 UTC
Beaker 0.11.0 has been released.


Note You need to log in before you can comment on or make changes to this bug.