<div dir="ltr"><div dir="ltr">Github issue is a fact.<div><a href="https://github.com/collectd/collectd/issues/2993">https://github.com/collectd/collectd/issues/2993</a></div><div><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">Best Regards,<br>Peter</div></div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 13, 2018 at 3:28 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Nov 13, 2018 at 03:17:33PM +0200, Peter Dimitrov wrote:<br>
> I was extending the virt plugin.<br>
> It already collects similar data (about VMs) using Libvirt's API, but lacks<br>
> disk usage information.<br>
> <br>
> I went through some hoops to link libguestfs correctly to collectd.<br>
> Is it okay to just include fork(), waitpid() example? It does reproduce the<br>
> issue.<br>
<br>
Yes, I think a small example/reproducer is a fine idea.<br>
<br>
Rich.<br>
<br>
> Best Regards,<br>
> Peter<br>
> <br>
> <br>
> On Tue, Nov 13, 2018 at 1:34 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank">rjones@redhat.com</a>><br>
> wrote:<br>
> <br>
> > On Tue, Nov 13, 2018 at 12:00:19PM +0100, Florian Forster wrote:<br>
> > > Hi,<br>
> > ><br>
> > > thank you very much for reporting this! Sounds like a bug in the exec<br>
> > plugin –<br>
> > > it never ceases to amaze me how many issues a single plugin can have ;)<br>
> > ><br>
> > > > > > This means that any plugin that does the usual pattern of:<br>
> > > > > ><br>
> > > > > >   pid = fork ();<br>
> > ><br>
> > > Note that the exec plugin is the *only* plugin that does this. All other<br>
> > > plugins are forbidden to fork(), popen() or create new processes in any<br>
> > other<br>
> > > way. The only plugin doing that, the exec plugin, has had enough issues<br>
> > over<br>
> > > the years for me to feel justified in that decision. ;-)<br>
> > ><br>
> > > As mentioned before, a Github issue would be appreciated so we can<br>
> > properly<br>
> > > track this problem.<br>
> ><br>
> > I don't know if Peter is using the exec plugin or is trying to write<br>
> > an ordinary plugin.  However the library he is using (libguestfs)<br>
> > certainly does fork subprocess(es).<br>
> ><br>
> > Rich.<br>
> ><br>
> > --<br>
> > Richard Jones, Virtualization Group, Red Hat<br>
> > <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
> > Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
> > virt-builder quickly builds VMs from scratch<br>
> > <a href="http://libguestfs.org/virt-builder.1.html" rel="noreferrer" target="_blank">http://libguestfs.org/virt-builder.1.html</a><br>
> ><br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
libguestfs lets you edit virtual machines.  Supports shell scripting,<br>
bindings from many languages.  <a href="http://libguestfs.org" rel="noreferrer" target="_blank">http://libguestfs.org</a><br>
</blockquote></div>