diff options
| author | Donald Sharp <donaldsharp72@gmail.com> | 2025-03-22 15:24:00 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-22 15:24:00 -0400 |
| commit | fcbace3d7ea49a378bd6110bee0150ab79f6247d (patch) | |
| tree | 27469a13ace24c98ab4d54c9abd27cfcb8675bfb | |
| parent | 7a09dc12d41351095088acc69b74fe0b30f5f5e3 (diff) | |
| parent | d85c3b9189abd43a6439cdd97e058e4080ec6387 (diff) | |
Merge pull request #18456 from FRRouting/mergify/bp/stable/10.2/pr-18108
zebra: Do not flush an existing vni configuration trying to remove wrong vni (backport #18108)
| -rw-r--r-- | zebra/zebra_cli.c | 28 |
1 files changed, 24 insertions, 4 deletions
diff --git a/zebra/zebra_cli.c b/zebra/zebra_cli.c index 6ee0fdbb8d..f12ad9f5fb 100644 --- a/zebra/zebra_cli.c +++ b/zebra/zebra_cli.c @@ -2348,12 +2348,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", |