The new fwts_test script will break the firmware/fwts_$1.log attachment job
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Fix Released
|
Low
|
Sylvain Pineau | ||
Provider for Plainbox - Canonical Certification (Legacy) |
Fix Released
|
Medium
|
Po-Hsu Lin |
Bug Description
We have a pair of fwts tests in our job definition file, the "firmware/fwts_$1", which will be generated by the "firmware/fwts" local job, and its attachment job "firmware/
These two jobs will be generated base on the fwts_test --list output.
However, with the recent change in fwts, the following test cases will have a trailing "_hwe" in the filename
HWE_TESTS = ['mtrr',
This filename change will cause the attachment jobs fail.
Affected test plans (with both firmware/fwts and firmware/fwts_logs in the test plan):
certification-
client-
client-
client-
client-
client-
CDTS:
ihv-firmware.pxu
ihv-firmware-
(certification-
But luckily,
it seems that only the CDTS provider do have the separated test cases for fwts, therefore this bug do have some impact on it.
For the test plans in certification-
Related branches
- Maciej Kisielewski (community): Approve
-
Diff: 13 lines (+2/-1)1 file modifiedproviders/plainbox-provider-checkbox/bin/fwts_test (+2/-1)
Changed in plainbox-provider-checkbox: | |
assignee: | Po-Hsu Lin (cypressyew) → Sylvain Pineau (sylvain-pineau) |
Changed in cdts: | |
assignee: | nobody → Sylvain Pineau (sylvain-pineau) |
Changed in plainbox-provider-canonical-certification: | |
assignee: | nobody → Sylvain Pineau (sylvain-pineau) |
Changed in cdts: | |
status: | New → In Progress |
Changed in plainbox-provider-canonical-certification: | |
status: | New → In Progress |
importance: | Undecided → Medium |
Changed in plainbox-provider-checkbox: | |
status: | In Progress → Fix Committed |
Changed in cdts: | |
status: | In Progress → Invalid |
status: | Invalid → Fix Committed |
Changed in plainbox-provider-canonical-certification: | |
milestone: | none → 0.19 |
Changed in plainbox-provider-checkbox: | |
status: | Fix Committed → Fix Released |
Changed in plainbox-provider-canonical-certification: | |
status: | Fix Committed → Fix Released |
Changed in cdts: | |
status: | Fix Committed → Fix Released |
BTW, I think we should rename this kind of local jobs, maybe rename them with a "generator-" prefix, so that it will become easier to read in the test plan, also, we could run all the local jobs by just running "generator-.* job".