Need storage tests for multipath devices.
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Checkbox Provider - Base |
Fix Released
|
High
|
Jeff Lane | ||
Bug Description
this is critical to Power8 certifications. We need a storage test similar to the storage_devices_* tests that works on multipath devices internally that are presented via /dev/mapper.
current storage testing only works against physical block devices and doing this against multi-pathing devices can cause data corruption of filesystem destruction as you could try running against sdg only to discover that sdg is also sda, your root filesystem.
So we need some sort of test to make use of a dev/mapper resource to determine the presence of multi-path devices (or more generically, dev/mapper devices, and test them while ignoring block devices.
So the behaviour should be, in order of precedence:
if /dev/mapper/mpathX
Run storage test against mpathX
else if /dev/sdX
Run storage test against sdX as we have in the past
for now, an either-or should be sufficient, there is a chance in the future we may need to expand this for multiple storage scenarios, but for now, the simple either/or is enough to do certification testing.
Related branches
- Paul Larson: Approve
- Jeff Lane : Needs Resubmitting
-
Diff: 152 lines (+89/-19)2 files modifiedbin/disk_stress_ng (+22/-7)
bin/storage_test (+67/-12)
- Sylvain Pineau: Approve
-
Diff: 71 lines (+7/-7)2 files modifiedjobs/disk.txt.in (+6/-6)
jobs/info.txt.in (+1/-1)
| Jeff Lane (bladernr) wrote | #1 |
| tags: | added: 16.04 hwcert-server |
| Mike Rushton (leftyfb) wrote | #2 |
| Changed in plainbox-provider-checkbox: | |
| status: | New → Confirmed |
| Changed in plainbox-provider-checkbox: | |
| status: | Confirmed → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Sylvain Pineau (sylvain-pineau) |
| milestone: | none → 0.35.0 |
| Changed in plainbox-provider-checkbox: | |
| milestone: | 0.35.0 → 0.36.0 |
| Changed in plainbox-provider-checkbox: | |
| status: | In Progress → Fix Committed |
| Jeff Lane (bladernr) wrote | #3 |
| Changed in plainbox-provider-checkbox: | |
| status: | Fix Committed → Confirmed |
| milestone: | 0.36.0 → future |
| Jeff Lane (bladernr) wrote | #4 |
| Changed in plainbox-provider-checkbox: | |
| assignee: | Sylvain Pineau (sylvain-pineau) → Jeff Lane (bladernr) |
| Changed in plainbox-provider-checkbox: | |
| milestone: | future → 0.36.0 |
| Changed in plainbox-provider-checkbox: | |
| status: | Confirmed → In Progress |
| Changed in plainbox-provider-checkbox: | |
| status: | In Progress → Fix Committed |
| Changed in plainbox-provider-checkbox: | |
| status: | Fix Committed → Fix Released |
this corresponds to https:/