On Mon, 24 Aug 2026 at 22:33, Zhijie Hou (Fujitsu)
<[email protected]> wrote:
> The fix looks good to me. I just noticed one nit in the test:
>
> +       ok(!$node->log_contains(qr/resource was not closed/),
> +               'unclosed resources on ' . $node->name);
>
> I think the test message should be: "no unclosed resources on".

Sorry, I must be missing something here. You want the message that
complains when there *is* a resource leak WARNING say "no unclosed
resources on $server"?

Maybe if that message is confusing, then it might be better to write
"resource leak detected on $server".  Is that any better?

David


Reply via email to