Re: func-inventory Patch

[Hany Fahim <hany.fahim@xxxxxxxxx>]

Yes, the 'break' causes it to break out of the for loop. I'm running this on CentOS 5.5, with the default python 2.4:

# certmaster-ca --list-signed | wc -l

20

# /usr/bin/func-inventory -v | grep 'system inventory' | wc -l

-- connection refused: server12

11

Now if I apply the patch which changes it to a 'continue' and run inventory again, I get all the hosts that are up:

# /usr/bin/func-inventory -v | grep system | wc -l

-- connection refused: server12

-- connection refused: server18

18

And indeed, I can verify that all hosts get their inventories updated with the patch. Hope this clears things up.

hany

On Wed, Mar 16, 2011 at 10:36 AM, seth vidal <skvidal@xxxxxxxxxxxxxxxxx> wrote:

On Wed, 2011-03-16 at 09:43 -0400, Hany Fahim wrote:
> Hey Seth,
>
>
> Just FYI, I'm using 0.27, but it also affected 0.25. It becomes
> apparent with many servers (I'm using it on 20 servers currently). The
> list gets generated in no particular order, and when #12 is down, it
> stops right there and doesn't complete.
>

I'm happy to apply your patch b/c it fixes the output of what hosts are
down but are you sure it stops processing them entirely? And doesn't
just stop outputting what it is doing?


-sv

_______________________________________________
Func-list mailing list
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list