Remove self.op.file_storage_dir isabs check
authorGuido Trotter <ultrotter@google.com>
Fri, 17 Jun 2011 09:16:58 +0000 (09:16 +0000)
committerGuido Trotter <ultrotter@google.com>
Fri, 17 Jun 2011 11:25:59 +0000 (14:25 +0300)
As the manpage says, and the code does, self.op.file_storage_dir is an
additional relative path under the cluster file storage dir. As such it
should not be absolute.

Signed-off-by: Guido Trotter <ultrotter@google.com>
Reviewed-by: RenĂ© Nussbaumer <rn@google.com>

lib/cmdlib.py

index 0451126..08a8d09 100644 (file)
@@ -7096,10 +7096,6 @@ class LUInstanceCreate(LogicalUnit):
       raise errors.OpPrereqError("Invalid file driver name '%s'" %
                                  self.op.file_driver, errors.ECODE_INVAL)
 
-    if self.op.file_storage_dir and os.path.isabs(self.op.file_storage_dir):
-      raise errors.OpPrereqError("File storage directory path not absolute",
-                                 errors.ECODE_INVAL)
-
     ### Node/iallocator related checks
     _CheckIAllocatorOrNode(self, "iallocator", "pnode")