LGTM, thanks.
On Thu, Nov 7, 2013 at 3:52 PM, Klaus Aehlig <[email protected]> wrote: > Patch d0d7d7cf accidentally removed the offline-flag reset > when readding a node. Readd it. > > Signed-off-by: Klaus Aehlig <[email protected]> > --- > lib/cmdlib/node.py | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/lib/cmdlib/node.py b/lib/cmdlib/node.py > index e83049d..1d98f29 100644 > --- a/lib/cmdlib/node.py > +++ b/lib/cmdlib/node.py > @@ -317,6 +317,7 @@ class LUNodeAdd(LogicalUnit): > # later in the procedure; this also means that if the re-add > # fails, we are left with a non-offlined, broken node > if self.op.readd: > + self.new_node.offline = False > self.new_node.drained = False > self.LogInfo("Readding a node, the offline/drained flags were > reset") > # if we demote the node, we do cleanup later in the procedure > -- > 1.8.4.1 > > -- Thomas Thrainer | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
