Re: [Pacemaker] crmsh resource update issue

2012-10-04 Thread Borislav Borisov
On Thu, Oct 4, 2012 at 2:22 PM, Dejan Muhamedagic deja...@fastmail.fmwrote: On Wed, Oct 03, 2012 at 09:17:17PM +0300, Vladislav Bogdanov wrote: 03.10.2012 20:11, Dejan Muhamedagic wrote: On Wed, Oct 03, 2012 at 11:57:47AM +0300, Vladislav Bogdanov wrote: 03.10.2012 11:48, Dejan

Re: [Pacemaker] crmsh resource update issue

2012-10-04 Thread David Vossel
- Original Message - From: Borislav Borisov borislav.v.bori...@gmail.com To: The Pacemaker cluster resource manager pacemaker@oss.clusterlabs.org Sent: Thursday, October 4, 2012 7:52:59 AM Subject: Re: [Pacemaker] crmsh resource update issue On Thu, Oct 4, 2012 at 2:22 PM

Re: [Pacemaker] crmsh resource update issue

2012-10-04 Thread Dejan Muhamedagic
: [Pacemaker] crmsh resource update issue On Thu, Oct 4, 2012 at 2:22 PM, Dejan Muhamedagic deja...@fastmail.fm wrote: On Wed, Oct 03, 2012 at 09:17:17PM +0300, Vladislav Bogdanov wrote: 03.10.2012 20:11, Dejan Muhamedagic wrote: On Wed, Oct 03, 2012 at 11:57

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Vladislav Bogdanov
03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew Beekhof wrote: ... Do you mean get the LSB and/or systemd metadata from the new lrmd? That should work already. Is there an utility which implements new lrm API? Answering to myself. lrmd_test It allows to do

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Dejan Muhamedagic
On Wed, Oct 03, 2012 at 09:21:41AM +0300, Vladislav Bogdanov wrote: 03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew Beekhof wrote: ... Do you mean get the LSB and/or systemd metadata from the new lrmd? That should work already. Is there an utility which

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Dejan Muhamedagic
On Wed, Oct 03, 2012 at 11:57:47AM +0300, Vladislav Bogdanov wrote: 03.10.2012 11:48, Dejan Muhamedagic wrote: On Wed, Oct 03, 2012 at 09:21:41AM +0300, Vladislav Bogdanov wrote: 03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew Beekhof wrote: ... Do you mean get

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Vladislav Bogdanov
03.10.2012 20:11, Dejan Muhamedagic wrote: On Wed, Oct 03, 2012 at 11:57:47AM +0300, Vladislav Bogdanov wrote: 03.10.2012 11:48, Dejan Muhamedagic wrote: On Wed, Oct 03, 2012 at 09:21:41AM +0300, Vladislav Bogdanov wrote: 03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Andrew Beekhof
On Wed, Oct 3, 2012 at 4:21 PM, Vladislav Bogdanov bub...@hoster-ok.com wrote: 03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew Beekhof wrote: ... Do you mean get the LSB and/or systemd metadata from the new lrmd? That should work already. Is there an utility which

Re: [Pacemaker] crmsh resource update issue

2012-10-03 Thread Andrew Beekhof
On Thu, Oct 4, 2012 at 7:45 AM, Andrew Beekhof and...@beekhof.net wrote: On Wed, Oct 3, 2012 at 4:21 PM, Vladislav Bogdanov bub...@hoster-ok.com wrote: 03.10.2012 08:57, Vladislav Bogdanov wrote: 03.10.2012 01:53, Andrew Beekhof wrote: ... Do you mean get the LSB and/or systemd metadata

Re: [Pacemaker] crmsh resource update issue

2012-10-02 Thread Vladislav Bogdanov
Hi Dejan, 28.09.2012 14:07, Dejan Muhamedagic wrote: ... OK. Silly me. Sorry for the non-vi users ;-) Thanks for the patch! Dejan One more patch to fix lrmadmin disfunction with 1.1.8. It is a little bit intrusive, but I'm definitely not a python coder :) You can just take it as a

Re: [Pacemaker] crmsh resource update issue

2012-10-02 Thread Vladislav Bogdanov
02.10.2012 15:41, Vladislav Bogdanov wrote: ... for req_op in self.required_ops: if req_op not in n_ops: -n_ops[req_op] = {} +if not (self.ra_class == stonith and op in (start, stop)): s/ op/ req_op/ +n_ops[req_op]

Re: [Pacemaker] crmsh resource update issue

2012-10-02 Thread Dejan Muhamedagic
Hi, On Tue, Oct 02, 2012 at 03:41:32PM +0300, Vladislav Bogdanov wrote: Hi Dejan, 28.09.2012 14:07, Dejan Muhamedagic wrote: ... OK. Silly me. Sorry for the non-vi users ;-) Thanks for the patch! Dejan One more patch to fix lrmadmin disfunction with 1.1.8. But this fixes

Re: [Pacemaker] crmsh resource update issue

2012-10-02 Thread Andrew Beekhof
On 03/10/2012, at 6:26 AM, Vladislav Bogdanov bub...@hoster-ok.com wrote: 02.10.2012 17:40, Dejan Muhamedagic wrote: ... One more patch to fix lrmadmin disfunction with 1.1.8. But this fixes support for RH fence-agents, if I'm not wrong. Yes btw. It is a little bit intrusive, but

Re: [Pacemaker] crmsh resource update issue

2012-09-28 Thread Vladislav Bogdanov
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

Re: [Pacemaker] crmsh resource update issue

2012-09-28 Thread 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

[Pacemaker] crmsh resource update issue

2012-09-27 Thread Vladislav Bogdanov
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. Logs and code analysis show that crm calls 'cibadmin -p -R'

Re: [Pacemaker] crmsh resource update issue

2012-09-27 Thread Dejan Muhamedagic
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

Re: [Pacemaker] crmsh resource update issue

2012-09-27 Thread Dejan Muhamedagic
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