Commit 8a3fe350 authored by Guido Trotter's avatar Guido Trotter
Browse files

LogicalUnit.BuildHooksEnv, update docstring

The LogicalUnit.BuildHooksEnv docstring used to say that the node list should
not include the master node. This is obviously not the case checking the
relevant code, and double-checking with iustin he confirmed it just document
some obsolete behaviour. This patch removes those remarks making the docstring
shorter, easier, and, last but not least, correct! ;)

Reviewed-by: iustinp
parent 7b7857ee
......@@ -145,11 +145,7 @@ class LogicalUnit(object):
added by the hooks runner. If the LU doesn't define any
environment, an empty dict (and not None) should be returned.
As for the node lists, the master should not be included in the
them, as it will be added by the hooks runner in case this LU
requires a cluster to run on (otherwise we don't have a node
list). No nodes should be returned as an empty list (and not
No nodes should be returned as an empty list (and not None).
Note that if the HPATH for a LU class is None, this function will
not be called.
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