It is not an error sign for a configuration to have detached disks, since they
can be handled from now on by the attach/detach operations.

Signed-off-by: Alex Pyrgiotis <[email protected]>

diff --git a/lib/config/__init__.py b/lib/config/__init__.py
index c666786..beb91f0 100644
--- a/lib/config/__init__.py
+++ b/lib/config/__init__.py
@@ -934,8 +934,6 @@ class ConfigWriter(object):
     @param result: list containing diagnostic messages
 
     """
-    instance_disk_uuids = [d for insts in data.instances.values()
-                           for d in insts.disks]
     for disk_uuid in data.disks:
       disk = data.disks[disk_uuid]
       result.extend(["disk %s error: %s" % (disk.uuid, msg)
@@ -943,9 +941,6 @@ class ConfigWriter(object):
       if disk.uuid != disk_uuid:
         result.append("disk '%s' is indexed by wrong UUID '%s'" %
                       (disk.name, disk_uuid))
-      if disk.uuid not in instance_disk_uuids:
-        result.append("disk '%s' is not attached to any instance" %
-                      disk.uuid)
 
   def _UnlockedVerifyConfig(self):
     """Verify function.
-- 
1.7.10.4

Reply via email to