Re: [Ganglia-developers] mod_python

2008-01-18 Thread Brad Nicholes
I can check it in. Brad >>> On 1/18/2008 at 12:42 PM, in message <[EMAIL PROTECTED]>, Matthias Blankenhaus <[EMAIL PROTECTED]> wrote: > > On Fri, 18 Jan 2008, Brad Nicholes wrote: > >> From what I have read about the popen2 interface, I think you are right. > It was either that or havin

Re: [Ganglia-developers] mod_python

2008-01-18 Thread Matthias Blankenhaus
On Fri, 18 Jan 2008, Brad Nicholes wrote: > From what I have read about the popen2 interface, I think you are right. > It was either that or having to manually close the stdin. > I don't think I haved write access to the tree. Could you pl. check it in or does Bernard takes care of this ?

Re: [Ganglia-developers] mod_python

2008-01-18 Thread Brad Nicholes
From what I have read about the popen2 interface, I think you are right. It was either that or having to manually close the stdin. Brad >>> On 1/18/2008 at 11:34 AM, in message <[EMAIL PROTECTED]>, Matthias Blankenhaus <[EMAIL PROTECTED]> wrote: > > On Fri, 18 Jan 2008, Brad Nicholes wrote:

Re: [Ganglia-developers] mod_python

2008-01-18 Thread Matthias Blankenhaus
On Fri, 18 Jan 2008, Brad Nicholes wrote: > I am seeing the same thing. It goes away if I use the subprocess module. > There must be some problem in the way that popen2 is invoked that is leaving > a defunct process. This fixes the problem: diff -u tcpconn.py tcpconn.py.new --- tcpconn.

Re: [Ganglia-developers] mod_python

2008-01-18 Thread Brad Nicholes
I am seeing the same thing. It goes away if I use the subprocess module. There must be some problem in the way that popen2 is invoked that is leaving a defunct process. Brad >>> On 1/17/2008 at 7:37 PM, in message <[EMAIL PROTECTED]>, "Bernard Li" <[EMAIL PROTECTED]> wrote: > Hi Brad: > >

Re: [Ganglia-developers] Patch to fix value figures in per-host metrics

2008-01-18 Thread Carlo Marcelo Arenas Belon
On Thu, Jan 17, 2008 at 01:45:20PM -0500, Jesse Becker wrote: > Another small patch. This one fixes an error introduced in r919, that > breaks the proper display of current metric values in the charts. > Specifically the "(now 0.23)", and similar text. Committed revision 926. Carlo

Re: [Ganglia-developers] Click-to-enlarge patch

2008-01-18 Thread Carlo Marcelo Arenas Belon
On Thu, Jan 17, 2008 at 09:48:11AM -0500, Jesse Becker wrote: > On Jan 17, 2008 12:44 AM, Carlo Marcelo Arenas Belon > > > > svn diff -c-921 | patch -p0 && svn merge -c810 > > This actually reverted three lines too many, and broke the enlarging > of optional graphs. Which shouldn't be the case