[Pacemaker] resource stop timeout broken in 1.0 branch tip

[Andrew Beekhof]

On Tue, Oct 12, 2010 at 9:34 AM, Keisuke MORI <keisuke.mori+ha at gmail.com> wrote:
> 2010/10/9 Andrew Beekhof <andrew at beekhof.net>:
>> On Fri, Oct 8, 2010 at 4:17 PM, Lars Ellenberg
>> <lars.ellenberg at linbit.com> wrote:
>>> On Wed, Oct 06, 2010 at 06:29:12PM +0900, Keisuke MORI wrote:
>>>> 2010/10/6 Andrew Beekhof <andrew at beekhof.net>:
>>>> >> Is there more changesets
>>>> >> that need to be backported regarding to this issues?
>>>> >
>>>> > There is now that Andreas brought the problem to my attention :-)
>>>> >   http://hg.clusterlabs.org/pacemaker/1.1/rev/e097c70226fe
>>>> >
>>>> >> If not, I think that the Andreas' patch should be applied to 1.0.
>>>> >> It seems to me that the patch is sane as it would restore the old
>>>> >> behavior for the stop operation with having the resource attributes as
>>>> >> the first patch intended.
>>>> >
>>>> > See the comment in the above patch. Andreas' original patch wouldn't
>>>> > have worked if the resource definition changed.
>>>>
>>>> I see, I will backport this to 1.0 too.
>
> Done.
> http://hg.clusterlabs.org/pacemaker/stable-1.0/rev/0d019d9e9c61
>
>
>>>
>>> May I take the oportunity to point you to
>>> http://hg.clusterlabs.org/pacemaker/1.1/rev/3f8df3dfb328
>>
>> ACK, no objection to this being backported :-)
>
> Also done, along with a minor compilation fix.
> http://hg.clusterlabs.org/pacemaker/stable-1.0/rev/70438ddd4351
> http://hg.clusterlabs.org/pacemaker/stable-1.0/rev/0a40fd0cb9f2

Great work!  I hope to be able to start 1.0 testing later this week.