[Linux-ha-dev] Re: [mgmt][patch1]Add boolean_op in mgmt

[Lars Marowsky-Bree]

On 2007-08-29T16:56:06, Yan Gao <ygao at novell.com> wrote:

Hi Yan, thanks for posting the patches! I'll try to go through them and
over feedback (based on the code, not on testing them yet):

> mgmt patch 1:
> http://coolpix.a-s-k.hk/~gaoyan/heartbeat-0713-location-boolean.patch
> 
> Signed-off-by: Yan Gao <ygao at novell.com>
> 
> Description: 
> Boolean_op for location rules is added in mgmt. 
> This fixed bug #1616
> http://old.linux-foundation.org/developer_bugzilla/show_bug.cgi?id=1616
> 
> Pls review and test it.

This one looks OK.

It still doesn't allow nested rules, but it is a step forward.

Thanks!



-- 
Teamlead Kernel, SuSE Labs, Research and Development
SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg)
"Experience is the name everyone gives to their mistakes." -- Oscar Wilde