[Cluster-devel] Re: [RFC PATCH] dlm: enhancing dlm_controld (pcmk) to be able to handle redundant rings

[Jiaju Zhang]

On Sat, Oct 24, 2009 at 4:51 AM, David Teigland <teigland at redhat.com> wrote:
> On Fri, Oct 23, 2009 at 12:18:09PM -0700, Steven Dake wrote:
>> On Fri, 2009-10-23 at 12:55 -0500, David Teigland wrote:
>> > On Fri, Oct 23, 2009 at 09:23:20PM +0800, Jiaju Zhang wrote:
>> > +    result = corosync_cfg_ring_status_get(handle,
>> > +                                       &interface_names,
>> > +                                       &interface_status,
>> > +                                       &interface_count);
>> > +    if (result != CS_OK) {
>> > +       log_error("Failed to get the ring status (error=%d)", result);
>> > +       (void)corosync_cfg_finalize(handle);
>> > +       return -1;
>> > +    }
>> > +
>> > +    (void)corosync_cfg_finalize(handle);
>> > +    return interface_count;
>> >
>> > What is interface_count if there's one ring up but rrp is configured?
>> > Could we read the totem/rrp config value from the objdb?
>
>
>> Dave,
>>
>> The rrp mode can be read from the confdb currently.
>
> Ah, yeah, that's what I meant.

Hello,

Attached is the improved patch, which read the rrp mode from the confdb.
Thanks a lot for your comments :)

Thanks,
Jiaju
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dlm-pcmk-config-protocol.diff
Type: text/x-diff
Size: 5130 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/cluster-devel/attachments/20091026/877ad8d8/attachment.bin>