<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 19, 2022 at 8:12 PM Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div style="font-size:small">Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>> wrote:<br></div><div style="font-size:small"><br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> > I faintly remmeber a similar patch where we decided not to work around<br>
> > for a test harness leaking filedescriptors into testcases.<br>
> ><br>
> <br>
> This also should be a solution, I searched the mailing list and got a<br>
> patch[1].<br>
> Do you mean adding that close-on-exec flag when opening fd in harness?<br>
<br>
Yes, that way you can be sure that no file descriptors are leaked to the<br>
tests.<br></blockquote><div><br></div><div><div style="font-size:small">Ok, should I send patch v2 like this below?</div><div style="font-size:small"><br></div><div style="font-size:small">Note: the automation test open04 got passed but I'm not sure</div><div style="font-size:small">if this has a side effect on logs. But from my observation, some </div><div style="font-size:small">tests (with old-API) log can't be collected anymore.</div></div></div></div></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Seems we shouldn't fix by adding 'close-on-exec' flag simply,</div><div class="gmail_default" style="font-size:small">it brings more issues to some old-API tests, I'm still looking into</div><div class="gmail_default" style="font-size:small">the problems which look like caused by ltp-pan designed.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">So can we just merge the patch as the original?</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>