[Pacemaker] crmsh resource update issue

[Dejan Muhamedagic]

On Fri, Sep 28, 2012 at 01:53:52PM +0300, Vladislav Bogdanov wrote:
> 27.09.2012 20:03, Dejan Muhamedagic wrote:
> > On Thu, Sep 27, 2012 at 05:02:24PM +0200, Dejan Muhamedagic wrote:
> >> Hi Vladimir,
> >>
> >> On Thu, Sep 27, 2012 at 10:22:43AM +0300, Vladislav Bogdanov wrote:
> >>> Hi Dejan, list,
> >>>
> >>> It looks like shell 1.2.0 (I use
> >>> b58a3398bf11621fe7811380f00245dac52d34c6 with patch you sent recently)
> >>> incorrectly replaces the whole cib with just resources section, so all
> >>> node state sections are dropped.
> >>
> >> Node state: do you mean the <status> section? That's bad. I'll
> >> see what happened.
> > 
> > This should fix it:
> > 
> > http://hg.savannah.gnu.org/hgweb/crmsh/rev/86b59c2c2b49
> > 
> > Please let me know if it did.
> 
> It did the trick, thank you!

Great!

> As I do not use vim, I needed to apply following patch (broken by
> 17f058d97f17):
> -------
> --- a/modules/cibconfig.py.orig 2012-09-28 09:45:07.000000000 +0000
> +++ b/modules/cibconfig.py      2012-09-28 10:38:13.624100578 +0000
> @@ -344,6 +344,7 @@ class CibObjectSetCli(CibObjectSet):
>          '''Extra processing of the string to be editted'''
>          if user_prefs.editor.startswith("vi"):
>              return "%s\n#vim:set syntax=pcmk\n" % s
> +        return s

OK. Silly me. Sorry for the non-vi users ;-) Thanks for the
patch!

Dejan

>      def process(self, cli_list, update = False):
>          '''
>          Create new objects or update existing ones.
> -------
> 
> Vladislav
> 
> 
> _______________________________________________
> Pacemaker mailing list: Pacemaker at 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