lp:~roadmr/checkbox/more-parser-tries

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:checkbox at revision 2305

Daniel Manrique (community): Needs Resubmitting on 2013-08-16

Zygmunt Krynicki (community): Approve on 2013-08-16

Diff: 5435 lines (+5148/-38)

5 files modified

checkbox-old/checkbox/parsers/tests/test_submission.py (+2/-2)
checkbox-old/checkbox/parsers/tests/test_udevadm.py (+80/-19)
checkbox-old/checkbox/parsers/tests/udevadm_data/LENOVO_T420.txt (+5034/-0)
checkbox-old/checkbox/parsers/udevadm.py (+30/-17)
checkbox-old/debian/changelog (+2/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1211521: udevadm parser picking wrong identifiers for devices

High

Fix Released

Link a bug report

Related blueprints

2309. By Daniel Manrique on 2013-08-16

Updates to udevadm tests to match some slight changes from using raw_path to obtain device IDs (a few extra disk device controllers appeared)

2308. By Daniel Manrique on 2013-08-16

checkbox:parsers:udevadm: Use the device's raw path internally for matching and determining identifiers.

Using the "cleaned-up", public path attribute resulted in assigning the
wrong IDs to devices, for instance a network interface would inherit the
PCI IDs of its parent bus.

2307. By Daniel Manrique on 2013-08-16

Added tests to verify that a set of devices matches certain expectations.
We look for bus, category, product name, vendor and product id matches.
This ensures that returned data matches what we expect of a device
(LP: #1211521)

2306. By Daniel Manrique on 2013-08-15

Added data file for Lenovo T420

2305. By Daniel Manrique on 2013-08-15

Added __repr__ method to UdevadmDevice, helps in debugging

2304. By Daniel Manrique on 2013-08-15

"Added a couple of checks so that tests looking for video files in $CHECKBOX_SHARE fail gracefully if the files don't exist. [r=zkrynicki][bug=1202946][author=roadmr]"

2303. By Daniel Manrique on 2013-08-15

"Adds a __version__ attribute to the checkbox.parsers.submission module, at the request of the PES Infrastructure team, to be used to determine when to reparse a submission. [r=zkrynicki][bug=1211825][author=roadmr]"

2302. By Daniel Manrique on 2013-08-15

"Explicitly sets the log dir so logs are stored in a predictable location, for the two stress tests that use pm_test script. [r=zkrynicki][bug=1205194][author=roadmr]"

2301. By Daniel Manrique on 2013-08-14

"scripts/internet_test: Added a passing threshold parameter for packet loss, added auto-adjustment of ping count and deadline so that they don't conflict with each other (LP: #1211144)
 [r=bladernr][bug=1211144][author=roadmr]"

2300. By Brendan Donegan on 2013-08-13

"automatic merge by tarmac [r=sylvain-pineau,cypressyew][bug=][author=brendan-donegan]"