Thread (1 message) 1 message, 1 author, 25d ago

Re: [PATCH net-next] selftests/net/openvswitch: add flow modify test

From: Ilya Maximets <i.maximets@ovn.org>
Date: 2026-06-06 01:33:04
Also in: linux-kselftest

On 6/6/26 1:42 AM, 侯敏熙 wrote:
Also sending a v2 shortly
to fix two pylint warnings that nipa flagged (+2 new warnings: missing
docstring on mod_flow, and a %-format string).
Hi, Minxi.  Thanks for working on all the test improvements, but,
please, do not send new versions just yet.  Wait for Aaron's review.

Simply re-sending the same patch with minor changes flagged by CI
creates a lot of churn and discourages people from reviewing the
series, as they don't really have time to look at version N while
there is already N+1.

As a general suggestion, try to wait for a human review.  It's hard
to define what the "good" amount of waiting time should be.  But if
the number of human replies is noticeably smaller than the patch
version number, that might be a good sign to slow down a little.

I would also suggest to wait for the Adrian's fix to be merged and
synced into net-next before sending the new version of the test to
avoid any potential mishaps.

Best regards, Ilya Maximets.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help