Bug #55451 for Net-Telnet-Gearman: First function on workers() is in wrong field

Wed Mar 10 17:02:46 2010 DOUGW [...] cpan.org - Ticket created

The first function registered to a worker shows up in the client_id field of the Net::Telnet::Gearman::Worker object. The first two ":"'s are sometimes adjacent. So this code in Net::Telnet::Gearman.pm: my ( $fd, $ip, $cid, @functions ) = split /[\s:]+/, $line; should be: my ( $fd, $ip, $cid, $funcs ) = split /(?:\s*:\s*)+/, $line; my @functions = split " ", $funcs;

Wed Mar 10 17:11:44 2010 DOUGW [...] cpan.org - Correspondence added

On Wed Mar 10 17:02:46 2010, DOUGW wrote: Show quoted text

> should be:

Correction: my ( $fd, $ip, $cid, $funcs ) = split /\s*:\s*/, $line; my @functions = split " ", $funcs;

Wed Mar 10 17:11:45 2010 DOUGW [...] cpan.org - Status changed from 'new' to 'open'

Wed Mar 10 18:47:24 2010 DOUGW [...] cpan.org - Subject changed from (no value) to 'First function on workers() is in wrong field'

Thu Mar 11 01:25:09 2010 johannes [...] plunien.de - Taken

Thu Mar 11 01:31:20 2010 johannes [...] plunien.de - Correspondence added

Hey, thanks for your message. Unfortunately your change does not work for me. Can you please provide some raw output of your gearmand telnet response. E.g. do: telnet gearmand-host 4730 workers ^] i get something like this: 123 1.2.3.4 - : resize_image 456 192.168.0.1 - : convert2png convert2jpeg convert2gif I'm using gearmand v0.11 (the c implementation). Thanks, plu

Thu Mar 11 11:04:47 2010 DOUGW [...] cpan.org - Correspondence added

On Thu Mar 11 01:31:20 2010, PLU wrote: Show quoted text

> thanks for your message. Unfortunately your change does not work for me. > Can you please provide some raw output of your gearmand telnet response.

I'm using gearman 0.12 The output on my localhost messes up Net::Telnet::Gearman even more: workers 12 ::1c00:0:ec14:9900%149960024 - : 11 ::1c00:0:ec14:9900%149960024 - : reverse list:3418 dereg:3418 which brings up another issue...functions (and client ids?) with colons in the name. (I was experimenting with registering a function that would list or unregister functions with a worker). Some sample output from the production server (also 0.12) is: 42 :: - : get 41 :: - : get

Thu Mar 11 14:13:16 2010 johannes [...] plunien.de - Correspondence added

On Thu Mar 11 11:04:47 2010, DOUGW wrote: Show quoted text

> On Thu Mar 11 01:31:20 2010, PLU wrote: >

> > thanks for your message. Unfortunately your change does not work for me. > > Can you please provide some raw output of your gearmand telnet response.

> > workers > 12 ::1c00:0:ec14:9900%149960024 - : > 11 ::1c00:0:ec14:9900%149960024 - : reverse list:3418 dereg:3418 > 42 :: - : get > 41 :: - : get

great, that helped me a lot, thanks! Can you please try this new dist: http://www.pqpq.de/~plu/Net-Telnet-Gearman-0.02000.tar.gz Please let me know if this fixes your problem so i can upload it to the CPAN. Cheers, plu

Thu Mar 11 17:10:16 2010 DOUGW [...] cpan.org - Correspondence added

I now get via Data::Dumper for a Net::Telnet::Gearman::Worker, on localhost: $VAR1 = bless( { 'functions' => [ 'reverse', 'list:17266', 'dereg:17266' ], 'client_id' => '-', 'file_descriptor' => '12', 'ip_address' => '::1c00:0:ec14:9900%149960024' }, 'Net::Telnet::Gearman::Worker' ); and in prod: $VAR1 = bless( { 'functions' => [ 'ns_store', 'list:5042', 'dereg:5042' ], 'client_id' => '-', 'file_descriptor' => '49', 'ip_address' => '::' }, 'Net::Telnet::Gearman::Worker' ); I get now that the ip address is in IPv6 format, but what is the "%149960024" in the first example? Should that be in the client_id? Or is this correct?

Thu Mar 11 18:11:00 2010 DOUGW [...] cpan.org - Correspondence added

On Thu Mar 11 17:10:16 2010, DOUGW wrote: Show quoted text

> I get now that the ip address is in IPv6 format, but what is the > "%149960024" in the first example? Should that be in the client_id? > Or is this correct?

I just reviewed IPv6 format, and the "%" is part of it (the "scoped-id")...so perfect!

Fri Mar 12 00:57:02 2010 johannes [...] plunien.de - Correspondence added

On Thu Mar 11 18:11:00 2010, DOUGW wrote: Show quoted text

> On Thu Mar 11 17:10:16 2010, DOUGW wrote: >

> > I get now that the ip address is in IPv6 format, but what is the > > "%149960024" in the first example? Should that be in the client_id? > > Or is this correct?

> > I just reviewed IPv6 format, and the "%" is part of it (the > "scoped-id")...so perfect! >

Great, thanks! So i'm going to upload it to CPAN right now. Cheers, plu

Fri Mar 12 01:00:04 2010 johannes [...] plunien.de - Status changed from 'open' to 'resolved'