Commit b4de68a9 authored by Iustin Pop's avatar Iustin Pop
Browse files

Style fixes for trunk

This small patch fixes:
  - wrong indentation in two places
  - use of 'os' variable that hides global scope os module

Reviewed-by: imsnah
parent b6e82a65
......@@ -1303,14 +1303,14 @@ class LUDiagnoseOS(NoHooksLU):
for node_name, nr in rlist.iteritems():
if not nr:
for os in nr:
if not in all_os:
for os_obj in nr:
if not in all_os:
# build a list of nodes for this os containing empty lists
# for each node in node_list
all_os[] = {}
all_os[] = {}
for nname in node_list:
all_os[][nname] = []
all_os[][nname] = []
return all_os
def Exec(self, feedback_fn):
......@@ -3326,8 +3326,8 @@ class LUCreateInstance(LogicalUnit):
if self.op.file_storage_dir and os.path.isabs(self.op.file_storage_dir):
raise errors.OpPrereqError("File storage directory not a relative"
" path")
raise errors.OpPrereqError("File storage directory not a relative"
" path")
#### allocator run
if [self.op.iallocator, self.op.pnode].count(None) != 1:
......@@ -4499,7 +4499,7 @@ class LUExportInstance(LogicalUnit):
# shutdown the instance, but not the disks
if not rpc.call_instance_shutdown(src_node, instance):
raise errors.OpExecError("Could not shutdown instance %s on node %s" %
(, src_node))
(, src_node))
vgname = self.cfg.GetVGName()
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment