Jarod.w has uploaded a new change for review.

Change subject: add -d parameter when umouning in mk_sysprep_floppy
......................................................................

add -d parameter when umouning in mk_sysprep_floppy

After umounting mntpoint successfully, it doesn't free automatically
the loop device. Here, we add '-d' parameter to free the device to
prevent it using up all loop devices.

The issue only happened in the special case that node is built using
livecd-tools(version: >= 16.8-1) and rpms from centos63. Because:
1. Centos63 disables auto-destruction feature and still depends on
  /etc/mtab to free the loop device.
2. Livecd-tools links /etc/mtab to /proc/self/mounts, so umount can't
  get suggestion to free the loop device from /etc/mtab.

Change-Id: I70e802b9062e586b10937c700f9a481bf1fe51b0
Signed-off-by: jarod.w <[email protected]>
---
M vdsm/mk_sysprep_floppy.in
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/70/8770/1

diff --git a/vdsm/mk_sysprep_floppy.in b/vdsm/mk_sysprep_floppy.in
index 5c491f0..8981702 100644
--- a/vdsm/mk_sysprep_floppy.in
+++ b/vdsm/mk_sysprep_floppy.in
@@ -37,6 +37,6 @@
 
 /bin/cat > $mntpoint/sysprep.inf
 
-umount $mntpoint
+umount -d $mntpoint
 rmdir $mntpoint
 


--
To view, visit http://gerrit.ovirt.org/8770
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I70e802b9062e586b10937c700f9a481bf1fe51b0
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Jarod.w <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to