Skip to content

Commit

Permalink
fix merge conflict
Browse files Browse the repository at this point in the history
  • Loading branch information
isabelmsft committed Dec 1, 2022
1 parent 7c898c4 commit 85581e6
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions config/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -5189,7 +5189,7 @@ def del_vrf(ctx, vrf_name):
"""Del vrf"""
config_db = ValidatedConfigDBConnector(ctx.obj['config_db'])
if not vrf_name.startswith("Vrf") and not (vrf_name == 'mgmt') and not (vrf_name == 'management'):
ctx.fail("'vrf_name' is not start with Vrf, mgmt or management!")
ctx.fail("'vrf_name' must begin with 'Vrf' or named 'mgmt'/'management' in case of ManagementVRF.")
if len(vrf_name) > 15:
ctx.fail("'vrf_name' is too long!")
syslog_table = config_db.get_table("SYSLOG_SERVER")
Expand All @@ -5198,7 +5198,9 @@ def del_vrf(ctx, vrf_name):
syslog_vrf = syslog_data.get("vrf")
if syslog_vrf == syslog_vrf_dev:
ctx.fail("Failed to remove VRF device: {} is in use by SYSLOG_SERVER|{}".format(syslog_vrf, syslog_entry))
if (vrf_name == 'mgmt' or vrf_name == 'management'):
if not is_vrf_exists(config_db, vrf_name):
ctx.fail("VRF {} does not exist!".format(vrf_name))
elif (vrf_name == 'mgmt' or vrf_name == 'management'):
vrf_delete_management_vrf(config_db)
else:
del_interface_bind_to_vrf(config_db, vrf_name)
Expand Down

0 comments on commit 85581e6

Please sign in to comment.