Small simplification in MapLVsByNode
authorIustin Pop <iustin@google.com>
Mon, 19 Jan 2009 11:10:52 +0000 (11:10 +0000)
committerIustin Pop <iustin@google.com>
Mon, 19 Jan 2009 11:10:52 +0000 (11:10 +0000)
We don't need to pre-create the node entries in lvmap, since they will
be created at recursion time.

Reviewed-by: ultrotter

lib/objects.py

index b7a31db..26facb6 100644 (file)
@@ -587,12 +587,6 @@ class Instance(TaggableObject):
         lvmap[node].append(dev.logical_id[1])
 
       elif dev.dev_type in constants.LDS_DRBD:
-        if dev.logical_id[0] not in lvmap:
-          lvmap[dev.logical_id[0]] = []
-
-        if dev.logical_id[1] not in lvmap:
-          lvmap[dev.logical_id[1]] = []
-
         if dev.children:
           self.MapLVsByNode(lvmap, dev.children, dev.logical_id[0])
           self.MapLVsByNode(lvmap, dev.children, dev.logical_id[1])