plainbox: xml-exporter is not exporting comments
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox |
Fix Released
|
High
|
Daniel Manrique | ||
Bug Description
Plainbox Version: 0.2+bzr2082+
A job failed due to an unmet requirement:
- audio/alsa_
comments: job cannot be started: resource expression "package.name == 'python-gst0.10'" evaluates to false
inhibitors:
* resource expression "package.name == 'python-gst0.10'" evaluates to false
Python-gst0.10 is not present on raring so the failure to run is correct. The problem is that the comment field in the XML report is not set in this case. Look at this fragment:
<question name="audio/
<answer type="multiple_
<
<value type="str"
<value type="str"
<value type="str"
<value type="str"
<
<
Plainbox knows what caused the skip, so it should put that in the comment field, and somehow isn't.
I'll attach the raw XML to this report.
Related branches
- Zygmunt Krynicki (community): Approve
-
Diff: 118 lines (+38/-24)3 files modifiedplainbox/plainbox/impl/applogic.py (+33/-0)
plainbox/plainbox/impl/commands/sru.py (+4/-24)
plainbox/plainbox/impl/exporter/xml.py (+1/-0)
| Daniel Manrique (roadmr) wrote | #2 |
| Changed in checkbox: | |
| assignee: | nobody → Daniel Manrique (roadmr) |
| status: | New → In Progress |
| importance: | Undecided → High |
| Daniel Manrique (roadmr) wrote | #3 |
| Zygmunt Krynicki (zyga) wrote | #4 |
| summary: |
- plainbox: Jobs skipped due to unmet requirements don't set the comment - field to explain the skip + plainbox: xml-exporter is not exporting comments |
| Changed in checkbox: | |
| milestone: | none → 2013-05-23 |
| status: | In Progress → Fix Committed |
| Changed in checkbox: | |
| status: | Fix Committed → Fix Released |
Another job with the same problem:
- mediacard/
comments: job cannot be started: resource expression "device.category == 'CARDREADER'" evaluates to false
inhibitors:
* resource expression "device.category == 'CARDREADER'" evaluates to false
To reiterate, the job failures themselves will be investigated in separate bugs, this one is about why plainbox is not putting that info in the comment field.