On Tue, May 14, 2013 at 10:03:59PM +0200, Lars Ellenberg wrote:
> On Tue, May 14, 2013 at 09:59:50PM +0200, Lars Ellenberg wrote:
> > On Mon, May 13, 2013 at 01:53:11PM +0200, Michael Schwartzkopff wrote:
> > > Hi,
> > >
> > > crm tells me it is version 1.2.4
> > > pacemaker tell me it is verison 1.1.9
> > >
> > > So it should work since incompatibilities are resolved in crm higher that
> > > version 1.2.1. Anywas crm tells me nonsense:
> > >
> > > # crm
> > > crm(live)# node
> > > crm(live)node# standby node1
> > > ERROR: bad lifetime: node1
> >
> > Your node is not named node1.
> > check: crm node list
> >
> > Maybe a typo, maybe some case-is-significant nonsense,
> > maybe you just forgot to use the fqdn.
> > maybe the check for "is this a known node name" is (now) broken?
> >
> >
> > standby with just one argument checks if that argument
> > happens to be a known node name,
> > and assumes that if it is not,
> > it "has to be" a lifetime,
> > and the current node is used as node name...
> >
> > Maybe we should invert that logic, and instead compare the single
> > argument against allowed lifetime values (reboot, forever), and assume
> > it is supposed to be a node name otherwise?
> >
> > Then the error would become
> > ERROR: unknown node name: node1
> >
> > Which is probably more useful most of the time.
> >
> > Dejan?
>
> Something like this maybe:
>
> diff --git a/modules/ui.py.in b/modules/ui.py.in
> --- a/modules/ui.py.in
> +++ b/modules/ui.py.in
> @@ -1185,7 +1185,7 @@ class NodeMgmt(UserInterface):
> if not args:
> node = vars.this_node
> if len(args) == 1:
> - if not args[0] in listnodes():
> + if args[0] in ("reboot", "forever"):
Yes, I wanted to look at it again. Another complication is that
the lifetime can be just about anything in that date ISO format.
Thanks,
Dejan
> node = vars.this_node
> lifetime = args[0]
> else:
>
> --
> : Lars Ellenberg
> : LINBIT | Your Way to High Availability
> : DRBD/HA support and consulting http://www.linbit.com
>
> _______________________________________________
> Pacemaker mailing list: Pacemaker@oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker
>
> Project Home: http://www.clusterlabs.org
> Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
> Bugs: http://bugs.clusterlabs.org
_______________________________________________
Pacemaker mailing list: Pacemaker@oss.clusterlabs.org
http://oss.clusterlabs.org/mailman/listinfo/pacemaker
Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://bugs.clusterlabs.org
