pcp
[Top] [All Lists]

RE: QA resource leaks

To: "'Nathan Scott'" <nathans@xxxxxxxxxx>, "'Frank Ch. Eigler'" <fche@xxxxxxxxxx>
Subject: RE: QA resource leaks
From: "Ken McDonell" <kenj@xxxxxxxxxxxxxxxx>
Date: Thu, 19 Mar 2015 17:43:13 +1100
Cc: "'pcp'" <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <1137434038.8265308.1426635422376.JavaMail.zimbra@xxxxxxxxxx>
References: <727434822.7795923.1426569757887.JavaMail.zimbra@xxxxxxxxxx> <1848657835.7797630.1426570540279.JavaMail.zimbra@xxxxxxxxxx> <20150317154425.GA29250@xxxxxxxxxx> <1137434038.8265308.1426635422376.JavaMail.zimbra@xxxxxxxxxx>
Thread-index: AQJlLwi/quks2St2ymBDuVYDLfeO7ADuM9aJAeyQaI8DGTcrT5vKIavg
Answer is in the commit message ....

    3. run pmmgr as user "pcp" and play assorted games with file/dir modes
       and $sudo use to make this all work (this is needed to allow the
       pmie/pmlogger to write their status in /var/lib/pcp/tmp/* so
       the pmcd pmda knows about them)

Taking $sudo out WILL break the test.

> -----Original Message-----
> From: Nathan Scott [mailto:nathans@xxxxxxxxxx]
> Sent: Wednesday, 18 March 2015 10:37 AM
> To: Frank Ch. Eigler; Ken McDonell
> Cc: pcp
> Subject: Re: QA resource leaks
> 
> 
> 
> ----- Original Message -----
> > > [...]
> > > Frank, test 666 is leaking pmmgr and valgrind processes ...
> > > [...]
> > > I have two of those for every test run over the last few days.  It
> > > looks like sudo's parent was killed, rather than the pmmgr child,
> > > since its been reparented to init.
> >
> > ISTM the sudo is not necessary or helpful here.  The following patch
> > takes that out, and doesn't affect results otherwise.
> 
> Ken, you added this sudo in commit 1db94e63528 - presumably removing it will
> re-introduce some other problem...?
> 
> cheers.
> 
> --
> Nathan

<Prev in Thread] Current Thread [Next in Thread>