summaryrefslogtreecommitdiff
path: root/python/vyos/configverify.py
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2022-08-01 17:27:59 +0200
committerChristian Poessinger <christian@poessinger.com>2022-08-01 17:28:01 +0200
commit71f2f2d789bd3452ca7b5306f39df5f6537bfef0 (patch)
treea363d1af9b05d445e94f2d1cc3712b517d534a63 /python/vyos/configverify.py
parentb6bab8573a8df082494d2c0e4635b228f281d5c6 (diff)
downloadvyos-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/vyos/configverify.py')
-rw-r--r--python/vyos/configverify.py14
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):
"""