]> git.proxmox.com Git - mirror_frr.git/commit - isisd/isis_nb.h
isisd, yang: unified lsp-timers command
authorEmanuele Di Pascale <emanuele@voltanet.io>
Mon, 6 Apr 2020 15:05:41 +0000 (17:05 +0200)
committerEmanuele Di Pascale <emanuele@voltanet.io>
Mon, 6 Apr 2020 15:38:27 +0000 (17:38 +0200)
commitd2c970ffb255e681bfcede881d28476b745ee2b8
treec0f53fe4d15351b59d8a63ff8ada7f5f3ed08df9
parentb71f5218db1e5eb6479691ffa2684308b2beecbe
isisd, yang: unified lsp-timers command

Yang constraints enforced by the northbound callbacks require that
the maximum lifetime be >= than (refresh interval + 300). When we are
moving from one config to another through frr-reload.py, we issue
a number of vtysh -c commands ('no lsp-refresh-interval level-1 500',
'no max-lsp-lifetime level-1 1000'), which reset these parameters to their
default values, respectively 900 and 1200. Depending on the actual
values in the current config, the order in which these commands are sent
might be the wrong one, in that we hit an invalid intermediate state and
make vtysh (and by extension frr-reload.py) return an error.

As a workaround, let's add a one-liner command that sets all these
inter-related parameters in one go, and make isisd display them as a
single line too, so that the diff will be computed as a single command.
The old individual commands are kept to ensure backwards compatibility.

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>
isisd/isis_cli.c
isisd/isis_nb.c
isisd/isis_nb.h
isisd/isis_nb_config.c
isisd/isisd.c
yang/frr-isisd.yang