proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-07-25 22:30:52 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #15251 from qlyoung/fix-doc-nbcli-numlist

Browse Source 
doc: fix misformatted list in nbapi docs
This commit is contained in:
Igor Ryzhov 2024-01-31 15:06:25 +02:00 committed by GitHub
commit 72949f5f94
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 11 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
doc/developer/northbound/retrofitting-configuration-commands.rst
View File

@ -989,27 +989,23 @@ CLI commands should be rewritten but maintained in the same file.
Since all CLI configuration commands from FRR will need to be rewritten, Since all CLI configuration commands from FRR will need to be rewritten,
this is an excellent opportunity to rework this part of the code to make this is an excellent opportunity to rework this part of the code to make
the commands easier to maintain and extend. These are the three main the commands easier to maintain and extend. These are the three main
recommendations: 1. Always use DEFPY instead of DEFUN to improve code recommendations:
readability. 2. Always try to join multiple DEFUNs into a single DEFPY
whenever possible. As an example, there’s no need to have both #. Always use DEFPY instead of DEFUN to improve code readability
``distance (1-255) A.B.C.D/M`` and ``distance (1-255) A.B.C.D/M WORD`` #. Always try to join multiple DEFUNs into a single DEFPY whenever possible. As
when a single ``distance (1-255) A.B.C.D/M [WORD]`` would suffice. 3. an example, there’s no need to have both ``distance (1-255) A.B.C.D/M`` and
When necessary, create a separate DEFPY for ``no`` commands so that part ``distance (1-255) A.B.C.D/M WORD`` when a single ``distance (1-255)
of the configuration command can be made optional for convenience. A.B.C.D/M [WORD]`` would suffice.
Example: #. When making a negative form of a command, put ``[no]`` in the positive form
``no timers basic [(5-2147483647) (5-2147483647) (5-2147483647)]``. In and use ``![...]`` to mark portions of the command that should be optional
this example, everything after ``no timers basic`` is ignored by FRR, so only in the ``no`` version.
it makes sense to accept ``no timers basic`` as a valid command. But it
also makes sense to accept all parameters
(``no timers basic (5-2147483647) (5-2147483647) (5-2147483647)``) to
make it easier to remove the command just by prefixing a “no” to it.
To rewrite a CLI command as a dumb wrapper around the northbound To rewrite a CLI command as a dumb wrapper around the northbound
callbacks, use the ``nb_cli_cfg_change()`` function. This function callbacks, use the ``nb_cli_cfg_change()`` function. This function
accepts as a parameter an array of ``cli_config_change`` structures that accepts as a parameter an array of ``cli_config_change`` structures that
specify the changes that need to performed on the candidate specify the changes that need to performed on the candidate
configuration. Here’s the declaration of this structure (taken from the configuration. Here’s the declaration of this structure (taken from the
*lib/northbound_cli.h* file): ``lib/northbound_cli.h`` file):
.. code:: c .. code:: c