backend: remove a couple of useless mkdir calls
authorGuido Trotter <ultrotter@google.com>
Wed, 5 May 2010 14:36:25 +0000 (16:36 +0200)
committerGuido Trotter <ultrotter@google.com>
Fri, 7 May 2010 11:34:33 +0000 (13:34 +0200)
Those directories must exist for the node daemon to run (it's in the
node daemon's list of ensured directories) and those functions are only
called by the node daemon, so there's no point in those checks+mkdir
calls.

Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: Balazs Lecz <leczb@google.com>

lib/backend.py

index 761628c..81f82bc 100644 (file)
@@ -2030,8 +2030,6 @@ def ExportSnapshot(disk, dest_node, instance, cluster_name, idx, debug):
   export_script = inst_os.export_script
 
   logfile = _InstanceLogName("export", inst_os.name, instance.name)
-  if not os.path.exists(constants.LOG_OS_DIR):
-    os.mkdir(constants.LOG_OS_DIR, 0750)
 
   real_disk = _OpenRealBD(disk)
 
@@ -2184,8 +2182,6 @@ def ImportOSIntoInstance(instance, src_node, src_images, cluster_name, debug):
   import_script = inst_os.import_script
 
   logfile = _InstanceLogName("import", instance.os, instance.name)
-  if not os.path.exists(constants.LOG_OS_DIR):
-    os.mkdir(constants.LOG_OS_DIR, 0750)
 
   comprcmd = "gunzip"
   impcmd = utils.BuildShellCmd("(cd %s; %s >%s 2>&1)", inst_os.path,