]> git.puffer.fish Git - matthieu/frr.git/commitdiff
docs: clean up some developer doc warnings
authorMark Stapp <mjs@labn.net>
Tue, 31 Oct 2023 13:57:13 +0000 (09:57 -0400)
committerMergify <37929162+mergify[bot]@users.noreply.github.com>
Tue, 31 Oct 2023 16:03:26 +0000 (16:03 +0000)
A few fixups to clean up some dev doc warnings, in the
northbound files especially.

Signed-off-by: Mark Stapp <mjs@labn.net>
(cherry picked from commit bb6d4cea60b3a91afacedaf265d935cc94453d57)

doc/developer/northbound/demos.rst
doc/developer/northbound/ppr-basic-test-topology.rst
doc/developer/northbound/retrofitting-configuration-commands.rst
doc/developer/northbound/yang-tools.rst
doc/developer/topotests.rst

index 21ab43a49b6006d8bc7805cbf92373cf686f69db..876bd2522236eeb3f33a3fffe9b03add2d3bcdf3 100644 (file)
@@ -2,7 +2,9 @@ Transactional CLI
 -----------------
 
 This short demo shows some of the capabilities of the new transactional
-CLI: |asciicast|
+CLI:
+
+|asciicast1|
 
 ConfD + NETCONF + Cisco YDK
 ---------------------------
@@ -17,9 +19,9 @@ using Cisco’s YANG Development Kit (YDK).
 -  Script used in the demo:
    https://gist.github.com/rwestphal/defa9bd1ccf216ab082d4711ae402f95
 
-|asciicast|
+|asciicast2|
 
-.. |asciicast| image:: https://asciinema.org/a/jL0BS5HfP2kS6N1HfgsZvfZk1.png
+.. |asciicast1| image:: https://asciinema.org/a/jL0BS5HfP2kS6N1HfgsZvfZk1.png
    :target: https://asciinema.org/a/jL0BS5HfP2kS6N1HfgsZvfZk1
-.. |asciicast| image:: https://asciinema.org/a/VfMElNxsjLcdvV7484E6ChxWv.png
+.. |asciicast2| image:: https://asciinema.org/a/VfMElNxsjLcdvV7484E6ChxWv.png
    :target: https://asciinema.org/a/VfMElNxsjLcdvV7484E6ChxWv
index 582c76c059dc576ee69ceddf84e1b2c2737c93bd..a680ed7dfa84e5ce33e36b3c85eec6bea138e438 100644 (file)
@@ -1258,7 +1258,7 @@ Using the ``show isis ppr`` command, verify that all routers installed
 the PPR-IDs for the paths they are part of. Example:
 
 Router RT11
-'''''''''''
+^^^^^^^^^^^
 
 ::
 
index c13332bf1fb5d2e4856671065754f268ec63208c..b407246049d2f9e9b07654d65915c22e7e65617f 100644 (file)
@@ -1,24 +1,5 @@
-Table of Contents
------------------
-
--  `Introduction <#introduction>`__
--  `Retrofitting process <#retrofitting-process>`__
-
-   -  `Step 1: writing a YANG module <#step1>`__
-   -  `Step 2: generate skeleton northbound callbacks <#step2>`__
-   -  `Step 3: update the frr_yang_module_info array of all relevant
-      daemons <#step3>`__
-   -  `Step 4: implement the northbound configuration
-      callbacks <#step4>`__
-   -  `Step 5: rewrite the CLI commands as dumb wrappers around the
-      northbound callbacks <#step5>`__
-   -  `Step 6: implement the ``cli_show`` callbacks <#step6>`__
-   -  `Step 7: consolidation <#step7>`__
-
--  `Final Considerations <#final-considerations>`__
-
-Introduction
-------------
+Retrofitting Configuration Commands
+-----------------------------------
 
 This page explains how to convert existing CLI configuration commands to
 the new northbound model. This documentation is meant to be the primary
@@ -88,7 +69,7 @@ possible. As we’ll see later, not all constraints can be expressed using
 the YANG language and sometimes we’ll need to resort to code-level
 validation in the northbound callbacks.
 
-   Tip: the [[YANG tools]] page details several tools and commands that
+   Tip: the :doc:`yang-tools` page details several tools and commands that
    might be useful when writing a YANG module, like validating YANG
    files, indenting YANG files, validating instance data, etc.
 
@@ -1875,7 +1856,7 @@ Final Considerations
 --------------------
 
 Testing
-^^^^^^^
+~~~~~~~
 
 Converting CLI commands to the new northbound model can be a complicated
 task for beginners, but the more commands one converts, the easier it
@@ -1888,7 +1869,7 @@ to use valgrind and static code analyzers to catch other types of
 problems like memory leaks.
 
 Amount of work
-^^^^^^^^^^^^^^
+~~~~~~~~~~~~~~
 
 The output below gives a rough estimate of the total number of
 configuration commands that need to be converted per daemon:
index 4eb1a2fb2cb438b4cde4093e9c87933c3fd52cc5..346efcaaee2a60c48755a0031ee67015cc3c1230 100644 (file)
@@ -1,3 +1,9 @@
+Yang Tools
+~~~~~~~~~~
+
+Here's some information about various tools for working with yang
+models.
+
 yanglint cheat sheet
 ~~~~~~~~~~~~~~~~~~~~
 
index 0e4f05ed0041577ecfbb778ac549eb66c5ebfc63..b8f213b99e41939d9283aa5da8c804d9baab2570 100644 (file)
@@ -470,7 +470,7 @@ One can live view daemon or the frr logs in separate windows using the
 
 For each capture a window is opened displaying a live summary of the captured
 packets. Additionally, the entire packet stream is captured in a pcap file in
-the tests log directory e.g.,::
+the tests log directory e.g.,
 
 When using a unified log file ``frr.log`` one substitutes ``frr`` for the
 daemon name in the ``--logd`` CLI option, e.g.,