Slapbuilder improvment for former changes

option one_disk and no_usb are no longer needed when using virtual
In case of virtual option remove kvm module activation

slapos/builder/__init__.py

@@ -122,6 +122,10 @@ class Parser(OptionParser):
       self.error("%s isn't valid. The first argument must be the raw " \
                  "slapos file" % system_path)
 
+    if options.virtual:
+      options.no_usb=True
+      options.one_disk=True
+
     device_path = os.path.abspath(device_path)
     if not options.no_usb:
       mode = os.stat(device_path)[ST_MODE]
@@ -285,6 +289,12 @@ def run(config):
             'script/%s' % script).read())
           os.chmod(path, 0755)
           
+      # Removing line in slapos script activating kvm in virtual 
+      if config.virtual:
+        path = os.path.join(mount_dir_path, 'etc', 'slapos','slapos')
+        _call(['sed','-i',"$d",path],dry_run=dry_run)
+        _call(['sed','-i',"$d",path],dry_run=dry_run)
+        
       # Adding slapos_firstboot in case of MultiDisk usage    
       if not config.one_disk :
       	for script in ['slapos_firstboot']:
@@ -307,7 +317,7 @@ def run(config):
         _call(['umount', mount_dir_path], dry_run=dry_run)
       else:
         print "Umount Virtual Machine"
-        _call(["vmware-mount","-x"],dry_run=dry_run)
+        _call(["vmware-mount","-K",config.system_path],dry_run=dry_run)
   finally:
     # Always delete temporary directory
     #print "No deleting"
@@ -372,4 +382,3 @@ def main():
 
     sys.exit(return_code)
 
-

slapos/builder/script/slapos

@@ -117,12 +117,10 @@ echo -n "Banging..."
 /opt/slapos/bin/bang -m "Rebooted" /etc/slapos/slapos.cfg
 echo "done."
 
-# Set kvm up
-modprobe kvm_intel
-chmod 666 /dev/kvm
-
 # Enable noop scheduler for disk which have SLAPOS labeled partition
 disk=`blkid -L SLAPOS | sed -r -e 's/(\/dev\/|[0-9]*$)//g'`
 echo noop > /sys/block/$disk/queue/scheduler
 
-
+# Set kvm up
+modprobe kvm_intel
+chmod 666 /dev/kvm
\ No newline at end of file