Re: [patch][bug][ldirectord] a fallback server specified in a virtual server section without a port

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi simon,

From: Simon Horman <horms@xxxxxxxxxxxx>
|
> On Thu, Aug 05, 2010 at 01:51:25PM +0900, Sohgo Takeuchi wrote:
>> 
>> 
>> From: Sohgo Takeuchi <sohgo@xxxxxxxxxxxxxxxx>
>> |
>> > I found a bug in ldirectord and attach a patch to fix this
>> > problem to this E-mail.
>> > 
>> > The bug is that if a port is omitted in a "fallback" in a
>> > virtual section, the entry is never seen in the virtual server
>> > table even if all real servers are down.
>> > 
>> >     virtual=10.10.100.1:daytime
>> > 	real=10.10.100.2:daytime gate
>> > 	fallback=127.0.0.1
>> > 
>> > A debug message says like this.
>> > 
>> >     DEBUG2: Running system(/sbin/ipvsadm -a -t 10.10.100.1:13 -r 127.0.0.1: -g -w 1)
>> >     Running system(/sbin/ipvsadm -a -t 10.10.100.1:13 -r 127.0.0.1: -g -w 1)
>> >     illegal real server address[:port] specified
>> >     DEBUG2: system(/sbin/ipvsadm -a -t 10.10.100.1:13 -r 127.0.0.1: -g -w 1) failed:
>> >     system(/sbin/ipvsadm -a -t 10.10.100.1:13 -r 127.0.0.1: -g -w 1) failed:
>> > 
>> > 
>> > The problem in the source code is that when a port of "fallback"
>> > is omitted, the port is derived from a port specified in a
>> > "virtual" service (from a behavior of the parse_fallback
>> > function), but the port is used before it is defined.
>> > 
>> > I tested ldirectord on Ubuntu 10.04 with perl 5.10.1.
>> > 
>> > with best regards,
>> 
>> Does anyone have an interest in this bug?
> 
> Sorry, I missed your previous post.
> I have pushed your patch into the agents tree.

Thank you very much!

--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Filesystem Devel]     [Linux NFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]     [X.Org]

  Powered by Linux