Commit 10cd4a29 authored by Guido Trotter's avatar Guido Trotter
Browse files

Remove NoHooksLU.BuildHooksEnv

Since NoHooksLU defines HPATH as None, BuildHooksEnv will never be called (as
the LogicalUnit.BuildHooksEnv docstring correctly states). Removing the
function altogether, to avoid having dead code lying around, and to make sure
the non-implemented one in LogicalUnit will be called if someone mistakenly
tries to call it, so it's immediately clear that something is wrong.

Reviewed-by: iustinp
parent 8a3fe350
......@@ -164,14 +164,6 @@ class NoHooksLU(LogicalUnit):
HPATH = None
HTYPE = None
def BuildHooksEnv(self):
"""Build hooks env.
This is a no-op, since we don't run hooks.
return {}, [], []
def _AddHostToEtcHosts(hostname):
"""Wrapper around utils.SetEtcHostsEntry.
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