From 5de6cd0247d77b2a679d7e1071e1f48a3e7c7a9d Mon Sep 17 00:00:00 2001 From: Christian Poessinger Date: Thu, 5 Mar 2020 19:46:28 +0100 Subject: vrf: T31: table id is mandatory --- src/conf_mode/vrf.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/conf_mode') diff --git a/src/conf_mode/vrf.py b/src/conf_mode/vrf.py index dd4b337e6..565db23d5 100755 --- a/src/conf_mode/vrf.py +++ b/src/conf_mode/vrf.py @@ -146,10 +146,14 @@ def verify(vrf_config): if len(vrf['members']) > 0: raise ConfigError('VRF "{}" can not be deleted as it has active members'.format(vrf['name'])) - # routing table id can't be changed - OS restriction for vrf in vrf_config['vrf_add']: + # table id is mandatory + if not vrf['table']: + raise ConfigError('VRF "{}": table id is mandatory'.format(vrf['name'])) + + # routing table id can't be changed - OS restriction if vrf['table_mod']: - raise ConfigError('VRF routing table id modification is not possible') + raise ConfigError('VRF "{}": modification of table id is not possible'.format(vrf['name'])) # add test to see if routing table already exists or not? -- cgit v1.2.3