curtin-dev team mailing list archive

[Dan Bungert <mp+469319@xxxxxxxxxxxxxxxxxx>]

The proposal to merge ~dbungert/curtin:lp-2061073 into curtin:master has been updated.

Description changed to:

In the crash logs of LP: #2061073, we can see that after we wipefs the 'disk' (raid), some data we were trying to read about the partitions is missing.  Two things appear incorrect:

* the sysfs reader knows that file entries might be missing, but attempts to proceed anyhow, so most or all the time we have those missing files we're going to crash like in this bug.  Handle that more smoothly by logging the scenario
* prior to that, quick_zero() attempts to do the recursive wipe by wiping the disk, then the partitions, but that means we're racing the kernel recognizing that these partitions have vanished so attempting to read info about the about-to-disappear partitions is likely unreliable.  Wipe partitions first, then disk.

Also, a new integration test has been added attempting to produce this case.

For more details, see:
https://code.launchpad.net/~dbungert/curtin/+git/curtin/+merge/469319
-- 
Your team curtin developers is requested to review the proposed merge of ~dbungert/curtin:lp-2061073 into curtin:master.