On Wed, 24 Apr 2024 16:19:49 -0700 Joe Damato wrote:
I'm not sure whether:
tools/net/ynl/lib/ynl.py should be raising NlError when there is an extack in this case (I think this is probably the way to go?), or
the tests should be changed so that they don't expect an exception to be raised but (ideally?) hide the warning report from tools/net/ynl/lib/ynl.py when the warning is expected.
I don't know python at all so this is definitely wrong, but here's a small change I made to fix the test (a similar change was made for the test which follows).
Sorry about this, we pushed the related fix to the net tree, to get it to Linus sooner: https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/commit/?id=a4...
You can cherry-pick that locally, perhaps. net will get merged into net-next tomorrow, I should have waited before extending the test :(