diff options
author | Christian Poessinger <christian@poessinger.com> | 2022-08-01 17:27:59 +0200 |
---|---|---|
committer | Christian Poessinger <christian@poessinger.com> | 2022-08-01 17:28:01 +0200 |
commit | 71f2f2d789bd3452ca7b5306f39df5f6537bfef0 (patch) | |
tree | a363d1af9b05d445e94f2d1cc3712b517d534a63 /python | |
parent | b6bab8573a8df082494d2c0e4635b228f281d5c6 (diff) | |
download | vyos-1x-71f2f2d789bd3452ca7b5306f39df5f6537bfef0.tar.gz vyos-1x-71f2f2d789bd3452ca7b5306f39df5f6537bfef0.zip |
bridge: T4565: bugfix error message when member interface contains an address
We should not print the entire dictionary - we only need the bridge interface
name:
Bug:
Cannot assign address to interface "eth1" as it is a member of bridge
"{'br0': {'allowed_vlan': ['5-50', '101'], 'native_vlan': '101'}}"!
Fixed:
Cannot assign address to interface "eth1" as it is a member of bridge
"br0"!
Diffstat (limited to 'python')
-rw-r--r-- | python/vyos/configverify.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/python/vyos/configverify.py b/python/vyos/configverify.py index 137eb9f79..ac56da204 100644 --- a/python/vyos/configverify.py +++ b/python/vyos/configverify.py @@ -243,10 +243,10 @@ def verify_address(config): of a bridge or bond. """ if {'is_bridge_member', 'address'} <= set(config): - raise ConfigError( - 'Cannot assign address to interface "{ifname}" as it is a ' - 'member of bridge "{is_bridge_member}"!'.format(**config)) - + interface = config['ifname'] + bridge_name = next(iter(config['is_bridge_member'])) + raise ConfigError(f'Cannot assign address to interface "{interface}" ' + f'as it is a member of bridge "{bridge_name}"!') def verify_bridge_delete(config): """ @@ -256,9 +256,9 @@ def verify_bridge_delete(config): """ if 'is_bridge_member' in config: interface = config['ifname'] - for bridge in config['is_bridge_member']: - raise ConfigError(f'Interface "{interface}" cannot be deleted as it ' - f'is a member of bridge "{bridge}"!') + bridge_name = next(iter(config['is_bridge_member'])) + raise ConfigError(f'Interface "{interface}" cannot be deleted as it ' + f'is a member of bridge "{bridge_name}"!') def verify_interface_exists(ifname): """ |