Keep the /host directory when copying over the files

Bug #241124 reported by Agostino Russo on 2008-06-18
4
Affects Status Importance Assigned to Milestone
lvpm
High
Geza Kovacs

Bug Description

The /host directory is required when the virtualdisk is resized, or /host (and hence /boot) will not be accessible. Patch provided.

=== modified file 'lvpm/usr/bin/lvpm-804'
--- lvpm/usr/bin/lvpm-804 2008-05-22 05:23:49 +0000
+++ lvpm/usr/bin/lvpm-804 2008-06-18 23:37:04 +0000
@@ -215,7 +215,7 @@
 killall zenity
 zenity --progress --percentage 15 --text="Copying files over (This can take over an hour, please wait)" &

-rsync -av --exclude '/sys/*' --exclude '/proc/*' --exclude '/host' --exclude '/mnt/*' --exclude '/media/*/*' --exclude '/tmp/*' --exclude '/home/*/.gvfs' --exclude '/root/.gvfs' / /media/tdm
+rsync -av --exclude '/sys/*' --exclude '/proc/*' --exclude '/host/*' --exclude '/mnt/*' --exclude '/media/*/*' --exclude '/tmp/*' --exclude '/home/*/.gvfs' --exclude '/root/.gvfs' / /media/tdm

 killall zenity
 zenity --progress --percentage 80 --text="Regenerating GRUB boot menu" &
@@ -583,7 +583,7 @@
 mkdir /media/extra
 mount -o loop,sync /host/ubuntu/disks/new.disk /media/extra
 zenity --progress --percentage 45 --text="Copying over files" &
-rsync -av --exclude '/sys/*' --exclude '/proc/*' --exclude '/host' --exclude '/mnt/*' --exclude '/media/*/*' --exclude '/tmp/*' --exclude '/home/*/.gvfs' --exclude '/root/.gvfs' / /media/extra
+rsync -av --exclude '/sys/*' --exclude '/proc/*' --exclude '/host/*' --exclude '/mnt/*' --exclude '/media/*/*' --exclude '/tmp/*' --exclude '/home/*/.gvfs' --exclude '/root/.gvfs' / /media/extra
 umount /media/extra
 rmdir /media/extra
 echo "

Geza Kovacs (gezakovacs) wrote :

Much thanks for pointing that out. I've applied your patch, and will release a new version with it as soon as I've tested it. I actually don't believe the modification is necessary for the first line (transferring to a dedicated partition), since the host mountpoint is changed to /media/drv0, but I've applied it nevertheless; an empty, unused folder isn't a big deal if it could potentially fix some corner-case issue.

Changed in lvpm:
assignee: nobody → gezakovacs
importance: Undecided → High
status: New → Fix Committed
Geza Kovacs (gezakovacs) wrote :

The patched version seems to be working fine, so I've released it on the usual spot as sourceforge, as revision 96.

As usual, the latest version should be directly accessible via the redirector at http://lubi.sourceforge.net/lvpm-latest.deb

Marking as Fix Released.

Changed in lvpm:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers