Bug 1303023 - missing result="" attribute on <result/> element in job results XML
Summary: missing result="" attribute on <result/> element in job results XML
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: reports
Version: 22
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: 22.1
Assignee: Dan Callaghan
QA Contact: matt jia
URL:
Whiteboard:
Depends On:
Blocks: 1303004
TreeView+ depends on / blocked
 
Reported: 2016-01-29 10:38 UTC by Ales Zelinka
Modified: 2016-02-01 04:30 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-02-01 04:30:32 UTC
Embargoed:

Attachments (Terms of Use)

Description Ales Zelinka 2016-01-29 10:38:21 UTC 
Description of problem:

result element no longer has result attribute. We rely on this in out tooling (tcms-results). This change breaks all beaker regression testing.


Details by xzhou:

> One attribute "result" has gone.
> 
> -<result id="184817448" path="/start" result="Pass" score="0.00">Install Started</result>
> +<result id="184817448" path="/start" score="0.00">Install Started</result>



Version-Release number of selected component (if applicable):
22.0
Comment 1 Dan Callaghan 2016-01-31 22:47:43 UTC 
So it turns out our test coverage for job results XML is not very thorough, which in hindsight is obviously quite a serious problem.

However I have compared the results XML output from release-21 branch vs. release-22 branch and this missing result="" attribute is the only issue I can see.
Comment 2 Dan Callaghan 2016-01-31 23:10:51 UTC 
http://gerrit.beaker-project.org/4648
Comment 3 Dan Callaghan 2016-01-31 23:11:15 UTC 
I am going to add some more complete test cases for the job results XML too but I don't want to hold up this fix for those.
Comment 4 Dan Callaghan 2016-02-01 01:17:37 UTC 
(In reply to Dan Callaghan from comment #3)
> I am going to add some more complete test cases for the job results XML too
> but I don't want to hold up this fix for those.

http://gerrit.beaker-project.org/4649
Comment 6 matt jia 2016-02-01 04:03:02 UTC 
Steps to verify:

1. look at one completed job results XML.

2. It contains the <result> element.
Comment 7 matt jia 2016-02-01 04:08:33 UTC 
(In reply to matt jia from comment #6)
> Steps to verify:
> 
> 1. look at one completed job results XML.
> 
> 2. It contains the <result> element.

To be more specially, the missing result attribute has been added back to the <result> element.
Comment 8 Dan Callaghan 2016-02-01 04:30:32 UTC 
Beaker 22.1 has been released.
Note You need to log in before you can comment on or make changes to this bug.