Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-10 Thread Tim Serong
On 5/10/2011 at 08:22 PM, Holger Teutsch wrote: > On Tue, 2011-05-10 at 08:24 +0200, Andrew Beekhof wrote: > > On Mon, May 9, 2011 at 8:44 PM, Holger Teutsch > > wrote: > > > On Wed, 2011-04-27 at 13:25 +0200, Andrew Beekhof wrote: > > >> On Sun, Apr 24, 2011 at 4:31 PM, Holger Teutsch >

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-10 Thread Holger Teutsch
On Tue, 2011-05-10 at 08:24 +0200, Andrew Beekhof wrote: > On Mon, May 9, 2011 at 8:44 PM, Holger Teutsch wrote: > > On Wed, 2011-04-27 at 13:25 +0200, Andrew Beekhof wrote: > >> On Sun, Apr 24, 2011 at 4:31 PM, Holger Teutsch > >> wrote: > > ... > >> > Remaining diffs seem to be not related to

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Andrew Beekhof
On Mon, May 9, 2011 at 8:44 PM, Holger Teutsch wrote: > On Wed, 2011-04-27 at 13:25 +0200, Andrew Beekhof wrote: >> On Sun, Apr 24, 2011 at 4:31 PM, Holger Teutsch >> wrote: > ... >> > Remaining diffs seem to be not related to my changes. >> >> Unlikely I'm afraid.  We run the regression tests a

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Holger Teutsch
On Wed, 2011-04-27 at 13:25 +0200, Andrew Beekhof wrote: > On Sun, Apr 24, 2011 at 4:31 PM, Holger Teutsch wrote: ... > > Remaining diffs seem to be not related to my changes. > > Unlikely I'm afraid. We run the regression tests after every commit > and complain loudly if they fail. > What is th

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Holger Teutsch
I had 1.1 but Dejan asked my to rebase my patches on devel. So long story short: devel now works after upgrading to the rev you mentioned and I got back to working on my patches. Thanx Holger On Mon, 2011-05-09 at 10:58 +0200, Andrew Beekhof wrote: > I thought you said you were running 1.1? > >

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Andrew Beekhof
I thought you said you were running 1.1? May 5 17:09:33 devel1 pacemakerd: [5929]: info: read_config: Reading configure for stack: corosync This message is specific to the devel branch. Update to get the following fix and you should be fine: http://hg.clusterlabs.org/pacemaker/devel/rev/84e

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Andrew Beekhof
On Mon, May 9, 2011 at 9:58 AM, Holger Teutsch wrote: > On Fri, 2011-05-06 at 16:15 +0200, Andrew Beekhof wrote: >> On Fri, May 6, 2011 at 12:28 PM, Holger Teutsch >> wrote: >> > On Fri, 2011-05-06 at 11:03 +0200, Andrew Beekhof wrote: >> >> On Fri, May 6, 2011 at 9:53 AM, Andrew Beekhof wrote:

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-09 Thread Holger Teutsch
On Fri, 2011-05-06 at 16:15 +0200, Andrew Beekhof wrote: > On Fri, May 6, 2011 at 12:28 PM, Holger Teutsch wrote: > > On Fri, 2011-05-06 at 11:03 +0200, Andrew Beekhof wrote: > >> On Fri, May 6, 2011 at 9:53 AM, Andrew Beekhof wrote: > >> > On Thu, May 5, 2011 at 5:43 PM, Holger Teutsch > >> >

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-06 Thread Andrew Beekhof
On Fri, May 6, 2011 at 12:28 PM, Holger Teutsch wrote: > On Fri, 2011-05-06 at 11:03 +0200, Andrew Beekhof wrote: >> On Fri, May 6, 2011 at 9:53 AM, Andrew Beekhof wrote: >> > On Thu, May 5, 2011 at 5:43 PM, Holger Teutsch >> > wrote: >> >> On Fri, 2011-04-29 at 09:41 +0200, Andrew Beekhof wrot

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-06 Thread Holger Teutsch
On Fri, 2011-05-06 at 11:03 +0200, Andrew Beekhof wrote: > On Fri, May 6, 2011 at 9:53 AM, Andrew Beekhof wrote: > > On Thu, May 5, 2011 at 5:43 PM, Holger Teutsch > > wrote: > >> On Fri, 2011-04-29 at 09:41 +0200, Andrew Beekhof wrote: > >>> >> > >>> > Unfortunately the devel code does not run

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-06 Thread Andrew Beekhof
On Fri, May 6, 2011 at 9:53 AM, Andrew Beekhof wrote: > On Thu, May 5, 2011 at 5:43 PM, Holger Teutsch wrote: >> On Fri, 2011-04-29 at 09:41 +0200, Andrew Beekhof wrote: >>> >> >>> > Unfortunately the devel code does not run at all in my environment so I >>> > have to fix this first. >>> >>> Oh?

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-06 Thread Andrew Beekhof
On Thu, May 5, 2011 at 5:43 PM, Holger Teutsch wrote: > On Fri, 2011-04-29 at 09:41 +0200, Andrew Beekhof wrote: >> >> >> > Unfortunately the devel code does not run at all in my environment so I >> > have to fix this first. >> >> Oh?  I ran CTS on it the other day and it was fine here. >> > > I i

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-05-05 Thread Holger Teutsch
On Fri, 2011-04-29 at 09:41 +0200, Andrew Beekhof wrote: > >> > > Unfortunately the devel code does not run at all in my environment so I > > have to fix this first. > > Oh? I ran CTS on it the other day and it was fine here. > I installed pacemaker-devel on top of a compilation of pacemaker-1.

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-29 Thread Andrew Beekhof
On Thu, Apr 28, 2011 at 9:53 AM, Holger Teutsch wrote: > Hi Dejan, > > On Tue, 2011-04-26 at 17:58 +0200, Dejan Muhamedagic wrote: >> Hi Holger, >> >> On Sun, Apr 24, 2011 at 04:31:33PM +0200, Holger Teutsch wrote: >> > On Mon, 2011-04-11 at 20:50 +0200, Andrew Beekhof wrote: >> > > why? >> > >  

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-28 Thread Holger Teutsch
Hi Dejan, On Tue, 2011-04-26 at 17:58 +0200, Dejan Muhamedagic wrote: > Hi Holger, > > On Sun, Apr 24, 2011 at 04:31:33PM +0200, Holger Teutsch wrote: > > On Mon, 2011-04-11 at 20:50 +0200, Andrew Beekhof wrote: > > > why? > > > CMD_ERR("Resource %s not moved:" > > > " spe

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-27 Thread Andrew Beekhof
On Sun, Apr 24, 2011 at 4:31 PM, Holger Teutsch wrote: > On Mon, 2011-04-11 at 20:50 +0200, Andrew Beekhof wrote: >> why? >>         CMD_ERR("Resource %s not moved:" >>                 " specifying --master is not supported for >> --move-from\n", rsc_id); >> > it did not look sensible to me but I

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-26 Thread Dejan Muhamedagic
Hi Holger, On Sun, Apr 24, 2011 at 04:31:33PM +0200, Holger Teutsch wrote: > On Mon, 2011-04-11 at 20:50 +0200, Andrew Beekhof wrote: > > why? > > CMD_ERR("Resource %s not moved:" > > " specifying --master is not supported for > > --move-from\n", rsc_id); > > > it did not

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-24 Thread Holger Teutsch
On Mon, 2011-04-11 at 20:50 +0200, Andrew Beekhof wrote: > why? > CMD_ERR("Resource %s not moved:" > " specifying --master is not supported for > --move-from\n", rsc_id); > it did not look sensible to me but I can't recall the exact reasons 8-) It's now implemented. > also

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-18 Thread Dejan Muhamedagic
Hi Holger, On Fri, Apr 08, 2011 at 03:21:31PM +0200, Holger Teutsch wrote: > On Thu, 2011-04-07 at 12:33 +0200, Dejan Muhamedagic wrote: > > > New syntax: > > > --- > > > > > > crm_resource --move-from --resource myresource --node mynode > > >-> all resource variants: check whether ac

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-11 Thread Andrew Beekhof
why? CMD_ERR("Resource %s not moved:" " specifying --master is not supported for --move-from\n", rsc_id); also the legacy handling is a little off - do a make install and run tools/regression.sh and you'll see what i mean. other than that the crm_resource part looks pretty

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-08 Thread Holger Teutsch
On Thu, 2011-04-07 at 12:33 +0200, Dejan Muhamedagic wrote: > > New syntax: > > --- > > > > crm_resource --move-from --resource myresource --node mynode > >-> all resource variants: check whether active on mynode, then create > > standby constraint > > > > crm_resource --move-from --

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-07 Thread Dejan Muhamedagic
Hi Holger, On Thu, Apr 07, 2011 at 10:03:49AM +0200, Holger Teutsch wrote: > On Thu, 2011-04-07 at 08:57 +0200, Andrew Beekhof wrote: > > On Wed, Apr 6, 2011 at 5:48 PM, Holger Teutsch > > wrote: > > > On Wed, 2011-04-06 at 15:38 +0200, Dejan Muhamedagic wrote: > > >> On Wed, Apr 06, 2011 at 01:

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-07 Thread Holger Teutsch
On Thu, 2011-04-07 at 08:57 +0200, Andrew Beekhof wrote: > On Wed, Apr 6, 2011 at 5:48 PM, Holger Teutsch wrote: > > On Wed, 2011-04-06 at 15:38 +0200, Dejan Muhamedagic wrote: > >> On Wed, Apr 06, 2011 at 01:00:36PM +0200, Andrew Beekhof wrote: > >> > On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhame

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-07 Thread Andrew Beekhof
On Wed, Apr 6, 2011 at 3:38 PM, Dejan Muhamedagic wrote: > On Wed, Apr 06, 2011 at 01:00:36PM +0200, Andrew Beekhof wrote: >> On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhamedagic >> wrote: >> > Ah, right, sorry, wanted to ask about the difference between >> > move-off and move. The description loo

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-07 Thread Andrew Beekhof
On Wed, Apr 6, 2011 at 5:48 PM, Holger Teutsch wrote: > On Wed, 2011-04-06 at 15:38 +0200, Dejan Muhamedagic wrote: >> On Wed, Apr 06, 2011 at 01:00:36PM +0200, Andrew Beekhof wrote: >> > On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhamedagic >> > wrote: >> > > Ah, right, sorry, wanted to ask about

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-06 Thread Holger Teutsch
On Wed, 2011-04-06 at 15:38 +0200, Dejan Muhamedagic wrote: > On Wed, Apr 06, 2011 at 01:00:36PM +0200, Andrew Beekhof wrote: > > On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhamedagic > > wrote: > > > Ah, right, sorry, wanted to ask about the difference between > > > move-off and move. The descripti

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-06 Thread Dejan Muhamedagic
On Wed, Apr 06, 2011 at 01:00:36PM +0200, Andrew Beekhof wrote: > On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhamedagic > wrote: > > Ah, right, sorry, wanted to ask about the difference between > > move-off and move. The description looks the same as for move. Is > > it that in this case it is for c

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-06 Thread Andrew Beekhof
On Tue, Apr 5, 2011 at 12:27 PM, Dejan Muhamedagic wrote: > Ah, right, sorry, wanted to ask about the difference between > move-off and move. The description looks the same as for move. Is > it that in this case it is for clones so crm_resource needs an > extra node parameter? You wrote in the doc

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Holger Teutsch
Hi Dejan, On Tue, 2011-04-05 at 13:40 +0200, Dejan Muhamedagic wrote: > Hi Holger, > > On Tue, Apr 05, 2011 at 01:19:56PM +0200, Holger Teutsch wrote: > > Hi Dejan, > > > > On Tue, 2011-04-05 at 12:27 +0200, Dejan Muhamedagic wrote: > > > On Tue, Apr 05, 2011 at 12:10:48PM +0200, Holger Teutsch

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Dejan Muhamedagic
Hi Holger, On Tue, Apr 05, 2011 at 01:19:56PM +0200, Holger Teutsch wrote: > Hi Dejan, > > On Tue, 2011-04-05 at 12:27 +0200, Dejan Muhamedagic wrote: > > On Tue, Apr 05, 2011 at 12:10:48PM +0200, Holger Teutsch wrote: > > > Hi Dejan, > > > > > > On Tue, 2011-04-05 at 11:48 +0200, Dejan Muhameda

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Holger Teutsch
Hi Dejan, On Tue, 2011-04-05 at 12:27 +0200, Dejan Muhamedagic wrote: > On Tue, Apr 05, 2011 at 12:10:48PM +0200, Holger Teutsch wrote: > > Hi Dejan, > > > > On Tue, 2011-04-05 at 11:48 +0200, Dejan Muhamedagic wrote: > > > Hi Holger, > > > > > > On Mon, Apr 04, 2011 at 09:31:02PM +0200, Holger

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Dejan Muhamedagic
On Tue, Apr 05, 2011 at 12:10:48PM +0200, Holger Teutsch wrote: > Hi Dejan, > > On Tue, 2011-04-05 at 11:48 +0200, Dejan Muhamedagic wrote: > > Hi Holger, > > > > On Mon, Apr 04, 2011 at 09:31:02PM +0200, Holger Teutsch wrote: > > > On Mon, 2011-04-04 at 15:24 +0200, Andrew Beekhof wrote: > > [..

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Holger Teutsch
Hi Dejan, On Tue, 2011-04-05 at 11:48 +0200, Dejan Muhamedagic wrote: > Hi Holger, > > On Mon, Apr 04, 2011 at 09:31:02PM +0200, Holger Teutsch wrote: > > On Mon, 2011-04-04 at 15:24 +0200, Andrew Beekhof wrote: > [...] > > > > crm_resource --move-off --resource myClone --node C > >-> I want

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Dejan Muhamedagic
Hi Holger, On Mon, Apr 04, 2011 at 09:31:02PM +0200, Holger Teutsch wrote: > On Mon, 2011-04-04 at 15:24 +0200, Andrew Beekhof wrote: [...] > > crm_resource --move-off --resource myClone --node C >-> I want the instance moved off C, regardless where it is moved on What is the difference betw

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-05 Thread Holger Teutsch
On Mon, 2011-04-04 at 21:31 +0200, Holger Teutsch wrote: > On Mon, 2011-04-04 at 15:24 +0200, Andrew Beekhof wrote: > > On Mon, Apr 4, 2011 at 2:43 PM, Holger Teutsch > > wrote: > > > On Mon, 2011-04-04 at 11:05 +0200, Andrew Beekhof wrote: > > >> On Sat, Mar 19, 2011 at 11:55 AM, Holger Teutsch

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-04 Thread Holger Teutsch
On Mon, 2011-04-04 at 15:24 +0200, Andrew Beekhof wrote: > On Mon, Apr 4, 2011 at 2:43 PM, Holger Teutsch wrote: > > On Mon, 2011-04-04 at 11:05 +0200, Andrew Beekhof wrote: > >> On Sat, Mar 19, 2011 at 11:55 AM, Holger Teutsch > >> wrote: > >> > Hi Dejan, > >> > > >> > On Fri, 2011-03-18 at 14:

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-04 Thread Andrew Beekhof
On Mon, Apr 4, 2011 at 2:43 PM, Holger Teutsch wrote: > On Mon, 2011-04-04 at 11:05 +0200, Andrew Beekhof wrote: >> On Sat, Mar 19, 2011 at 11:55 AM, Holger Teutsch >> wrote: >> > Hi Dejan, >> > >> > On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: >> >> Hi, >> >> >> >> On Fri, Mar 18

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-04 Thread Holger Teutsch
On Mon, 2011-04-04 at 11:05 +0200, Andrew Beekhof wrote: > On Sat, Mar 19, 2011 at 11:55 AM, Holger Teutsch > wrote: > > Hi Dejan, > > > > On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: > >> Hi, > >> > >> On Fri, Mar 18, 2011 at 12:21:40PM +0100, Holger Teutsch wrote: > >> > Hi, > >>

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-04-04 Thread Andrew Beekhof
On Sat, Mar 19, 2011 at 11:55 AM, Holger Teutsch wrote: > Hi Dejan, > > On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: >> Hi, >> >> On Fri, Mar 18, 2011 at 12:21:40PM +0100, Holger Teutsch wrote: >> > Hi, >> > I would like to submit 2 patches of an initial implementation for >> > disc

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-03-21 Thread Holger Teutsch
Hi Dejan, On Mon, 2011-03-21 at 16:11 +0100, Dejan Muhamedagic wrote: > Hi Holger, > > On Sat, Mar 19, 2011 at 11:55:57AM +0100, Holger Teutsch wrote: > > Hi Dejan, > > > > On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: > > > Hi, > > > > > > On Fri, Mar 18, 2011 at 12:21:40PM +0100

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-03-21 Thread Dejan Muhamedagic
Hi Holger, On Sat, Mar 19, 2011 at 11:55:57AM +0100, Holger Teutsch wrote: > Hi Dejan, > > On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: > > Hi, > > > > On Fri, Mar 18, 2011 at 12:21:40PM +0100, Holger Teutsch wrote: > > > Hi, > > > I would like to submit 2 patches of an initial im

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-03-19 Thread Holger Teutsch
Hi Dejan, On Fri, 2011-03-18 at 14:24 +0100, Dejan Muhamedagic wrote: > Hi, > > On Fri, Mar 18, 2011 at 12:21:40PM +0100, Holger Teutsch wrote: > > Hi, > > I would like to submit 2 patches of an initial implementation for > > discussion. .. > > To recall: > > > > crm_resource --move resource > >

Re: [Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-03-18 Thread Dejan Muhamedagic
Hi, On Fri, Mar 18, 2011 at 12:21:40PM +0100, Holger Teutsch wrote: > Hi, > I would like to submit 2 patches of an initial implementation for > discussion. > > Patch 1 implements migration of the Master role of an m/s resource to > another node in crm_resource > Patch 2 adds support for the shell

[Pacemaker] [PATCH]Bug 2567 - crm resource migrate should support an optional "role" parameter

2011-03-18 Thread Holger Teutsch
Hi, I would like to submit 2 patches of an initial implementation for discussion. Patch 1 implements migration of the Master role of an m/s resource to another node in crm_resource Patch 2 adds support for the shell. crm_resource does this with options "crm_resource --move --resource ms_test --m