* [PATCH net-next] tools: ynl: throw a more meaningful exception if family not supported
@ 2023-04-07 14:56 Jakub Kicinski
2023-04-11 13:40 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 2+ messages in thread
From: Jakub Kicinski @ 2023-04-07 14:56 UTC (permalink / raw)
To: davem; +Cc: netdev, edumazet, pabeni, kory.maincent, Jakub Kicinski
cli.py currently throws a pure KeyError if kernel doesn't support
a netlink family. Users who did not write ynl (hah) may waste
their time investigating what's wrong with the Python code.
Improve the error message:
Traceback (most recent call last):
File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 362, in __init__
self.family = GenlFamily(self.yaml['name'])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 331, in __init__
self.genl_family = genl_family_name_to_id[family_name]
~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^
KeyError: 'netdev'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/kicinski/devel/linux/./tools/net/ynl/cli.py", line 52, in <module>
main()
File "/home/kicinski/devel/linux/./tools/net/ynl/cli.py", line 31, in main
ynl = YnlFamily(args.spec, args.schema)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 364, in __init__
raise Exception(f"Family '{self.yaml['name']}' not supported by the kernel")
Exception: Family 'netdev' not supported by the kernel
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
tools/net/ynl/lib/ynl.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tools/net/ynl/lib/ynl.py b/tools/net/ynl/lib/ynl.py
index 7690e0b0cb3f..aa77bcae4807 100644
--- a/tools/net/ynl/lib/ynl.py
+++ b/tools/net/ynl/lib/ynl.py
@@ -358,7 +358,10 @@ genl_family_name_to_id = None
bound_f = functools.partial(self._op, op_name)
setattr(self, op.ident_name, bound_f)
- self.family = GenlFamily(self.yaml['name'])
+ try:
+ self.family = GenlFamily(self.yaml['name'])
+ except KeyError:
+ raise Exception(f"Family '{self.yaml['name']}' not supported by the kernel")
def ntf_subscribe(self, mcast_name):
if mcast_name not in self.family.genl_family['mcast']:
--
2.39.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] tools: ynl: throw a more meaningful exception if family not supported
2023-04-07 14:56 [PATCH net-next] tools: ynl: throw a more meaningful exception if family not supported Jakub Kicinski
@ 2023-04-11 13:40 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-04-11 13:40 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: davem, netdev, edumazet, pabeni, kory.maincent
Hello:
This patch was applied to netdev/net-next.git (main)
by Paolo Abeni <pabeni@redhat.com>:
On Fri, 7 Apr 2023 07:56:09 -0700 you wrote:
> cli.py currently throws a pure KeyError if kernel doesn't support
> a netlink family. Users who did not write ynl (hah) may waste
> their time investigating what's wrong with the Python code.
> Improve the error message:
>
> Traceback (most recent call last):
> File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 362, in __init__
> self.family = GenlFamily(self.yaml['name'])
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 331, in __init__
> self.genl_family = genl_family_name_to_id[family_name]
> ~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^
> KeyError: 'netdev'
>
> [...]
Here is the summary with links:
- [net-next] tools: ynl: throw a more meaningful exception if family not supported
https://git.kernel.org/netdev/net-next/c/ebe3bdc4359e
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-04-11 13:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-07 14:56 [PATCH net-next] tools: ynl: throw a more meaningful exception if family not supported Jakub Kicinski
2023-04-11 13:40 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).