Comment 1 for bug 69217

Revision history for this message
Phillip Susi (psusi) wrote :

I believe that this breaks the intended functionality of the loop. The purpose of the loop is to keep scanning devices until the ones containing the volume group appear, then enable the volume group. The line you added performs essentially the same scan only once, and if it is not found the first time, carries on.

In other words, you would accomplish the same thing by simply removing the while loop and letting the body execute only one time, which is how this script looked in edgy. It was changed in dapper because sometimes it takes some time for the disks to be detected and we don't want the script to decide it can't find the vg when it just needs to wait a while.