Merge branch 'stable-2.16' into stable-2.17
[ganeti-github.git] / lib / client / gnt_cluster.py
index 83dfca2..9d135ad 100644 (file)
 # W0614: Unused import %s from wildcard import (since we need cli)
 # C0103: Invalid name gnt-cluster
 
-from cStringIO import StringIO
+import itertools
 import os
 import time
-import OpenSSL
 import tempfile
-import itertools
+
+from cStringIO import StringIO
+
+import OpenSSL
 
 from ganeti.cli import *
 from ganeti import bootstrap
@@ -203,7 +205,7 @@ def InitCluster(opts, args):
   # check the disk template types here, as we cannot rely on the type check done
   # by the opcode parameter types
   diskparams_keys = set(diskparams.keys())
-  if not (diskparams_keys <= constants.DISK_TEMPLATES):
+  if diskparams_keys > constants.DISK_TEMPLATES:
     unknown = utils.NiceSort(diskparams_keys - constants.DISK_TEMPLATES)
     ToStderr("Disk templates unknown: %s" % utils.CommaJoin(unknown))
     return 1
@@ -305,10 +307,7 @@ def InitCluster(opts, args):
 
   default_ialloc_params = opts.default_iallocator_params
 
-  if opts.enabled_user_shutdown:
-    enabled_user_shutdown = True
-  else:
-    enabled_user_shutdown = False
+  enabled_user_shutdown = bool(opts.enabled_user_shutdown)
 
   if opts.ssh_key_type:
     ssh_key_type = opts.ssh_key_type
@@ -848,14 +847,15 @@ def VerifyDisks(opts, args):
         if all_missing:
           ToStdout("Instance %s cannot be verified as it lives on"
                    " broken nodes", iname)
-        else:
-          ToStdout("Instance %s has missing logical volumes:", iname)
-          ival.sort()
-          for node, vol in ival:
-            if node in bad_nodes:
-              ToStdout("\tbroken node %s /dev/%s", node, vol)
-            else:
-              ToStdout("\t%s /dev/%s", node, vol)
+          continue
+
+        ToStdout("Instance %s has missing logical volumes:", iname)
+        ival.sort()
+        for node, vol in ival:
+          if node in bad_nodes:
+            ToStdout("\tbroken node %s /dev/%s", node, vol)
+          else:
+            ToStdout("\t%s /dev/%s", node, vol)
 
       ToStdout("You need to replace or recreate disks for all the above"
                " instances if this message persists after fixing broken nodes.")