summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDonald Sharp <donaldsharp72@gmail.com>2025-03-22 15:25:35 -0400
committerGitHub <noreply@github.com>2025-03-22 15:25:35 -0400
commit24e51889e8605a9dbbb65b80464ebc4185a0abd8 (patch)
treee15a5fca7bb516917008a3e36986ea526a97e466
parentfd2a3696b18ddb1fd35b27589f8de1dcdb7b83c0 (diff)
parentb3923c5cfbc740a6e7a35563f4a0b3954e17a881 (diff)
Merge pull request #18458 from FRRouting/mergify/bp/stable/10.0/pr-18108
zebra: Do not flush an existing vni configuration trying to remove wrong vni (backport #18108)
-rw-r--r--zebra/zebra_cli.c28
1 files changed, 24 insertions, 4 deletions
diff --git a/zebra/zebra_cli.c b/zebra/zebra_cli.c
index 3e03d74775..4f05c56009 100644
--- a/zebra/zebra_cli.c
+++ b/zebra/zebra_cli.c
@@ -2317,12 +2317,32 @@ DEFPY_YANG (vni_mapping,
"VNI-ID\n"
"prefix-routes-only\n")
{
- if (!no)
+ const struct lyd_node *dnode;
+ const char *vrf;
+
+ if (!no) {
nb_cli_enqueue_change(vty, "./frr-zebra:zebra/l3vni-id", NB_OP_MODIFY,
vni_str);
- else
- nb_cli_enqueue_change(vty, "./frr-zebra:zebra/l3vni-id", NB_OP_DESTROY,
- NULL);
+ } else {
+ if (vty->node == CONFIG_NODE) {
+ if (yang_dnode_existsf(vty->candidate_config->dnode,
+ "/frr-vrf:lib/vrf[name='%s']/frr-zebra:zebra[l3vni-id='%lu']",
+ VRF_DEFAULT_NAME, vni))
+ nb_cli_enqueue_change(vty, "./frr-zebra:zebra/l3vni-id",
+ NB_OP_DESTROY, NULL);
+ } else {
+ dnode = yang_dnode_get(vty->candidate_config->dnode, VTY_CURR_XPATH);
+ if (dnode) {
+ vrf = yang_dnode_get_string(dnode, "name");
+
+ if (yang_dnode_existsf(vty->candidate_config->dnode,
+ "/frr-vrf:lib/vrf[name='%s']/frr-zebra:zebra[l3vni-id='%lu']",
+ vrf, vni))
+ nb_cli_enqueue_change(vty, "./frr-zebra:zebra/l3vni-id",
+ NB_OP_DESTROY, NULL);
+ }
+ }
+ }
if (filter)
nb_cli_enqueue_change(vty, "./frr-zebra:zebra/prefix-only",