[ovs-dev] [PATCH 13/15] doc: Convert DESIGN to rST

[Russell Bryant]

On Tue, Oct 18, 2016 at 10:03 PM, Stephen Finucane <stephen at that.guru>
wrote:

> -```
> -                                             master/
> -  message and reason code                     other     slave
> -  ----------------------------------------   -------    -----
> -  OFPT_PACKET_IN / NXT_PACKET_IN
> -    OFPR_NO_MATCH                              yes       ---
> -    OFPR_ACTION                                yes       ---
> -    OFPR_INVALID_TTL                           ---       ---
> -    OFPR_ACTION_SET (OF1.4+)                   yes       ---
> -    OFPR_GROUP (OF1.4+)                        yes       ---
> -
> -  OFPT_FLOW_REMOVED / NXT_FLOW_REMOVED
> -    OFPRR_IDLE_TIMEOUT                         yes       ---
> -    OFPRR_HARD_TIMEOUT                         yes       ---
> -    OFPRR_DELETE                               yes       ---
> -    OFPRR_GROUP_DELETE (OF1.4+)                yes       ---
> -    OFPRR_METER_DELETE (OF1.4+)                yes       ---
> -    OFPRR_EVICTION (OF1.4+)                    yes       ---
> -
> -  OFPT_PORT_STATUS
> -    OFPPR_ADD                                  yes       yes
> -    OFPPR_DELETE                               yes       yes
> -    OFPPR_MODIFY                               yes       yes
> -
> -  OFPT_ROLE_REQUEST / OFPT_ROLE_REPLY (OF1.4+)
> -    OFPCRR_MASTER_REQUEST                      ---       ---
> -    OFPCRR_CONFIG                              ---       ---
> -    OFPCRR_EXPERIMENTER                        ---       ---
> -
> -  OFPT_TABLE_STATUS (OF1.4+)
> -    OFPTR_VACANCY_DOWN                         ---       ---
> -    OFPTR_VACANCY_UP                           ---       ---
> -
> -  OFPT_REQUESTFORWARD (OF1.4+)
> -    OFPRFR_GROUP_MOD                           ---       ---
> -    OFPRFR_METER_MOD                           ---       ---
> -```
>



> ++-------------------------------------------+---------+-------+
> +|                                           | master/ |       |
> +|         message and reason code           | other   | slave |
> ++===========================================+=========+=======+
> +| ``OFPT_PACKET_IN`` / ``NXT_PACKET_IN``                      |
> ++-------------------------------------------+---------+-------+
> +|  ``OFPR_NO_MATCH``                        |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_ACTION``                           |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_INVALID_TTL``                      |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_ACTION_SET`` (OF1.4+)              |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPR_GROUP`` (OF1.4+)                   |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_FLOW_REMOVED`` / ``NXT_FLOW_REMOVED``                |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_IDLE_TIMEOUT``                    |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_HARD_TIMEOUT``                    |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_DELETE``                          |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_GROUP_DELETE`` (OF1.4+)           |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_METER_DELETE`` (OF1.4+)           |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRR_EVICTION`` (OF1.4+)               |   yes   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_PORT_STATUS``                                        |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_ADD``                             |   yes   |  yes  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_DELETE``                          |   yes   |  yes  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPPR_MODIFY``                          |   yes   |  yes  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_ROLE_REQUEST`` / ``OFPT_ROLE_REPLY`` (OF1.4+)        |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_MASTER_REQUEST``                 |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_CONFIG``                         |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPCRR_EXPERIMENTER``                   |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_TABLE_STATUS`` (OF1.4+)                              |
> ++-------------------------------------------+---------+-------+
> +| ``OFPTR_VACANCY_DOWN``                    |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPTR_VACANCY_UP``                      |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPT_REQUESTFORWARD`` (OF1.4+)                            |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRFR_GROUP_MOD``                      |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
> +| ``OFPRFR_METER_MOD``                      |   ---   |  ---  |
> ++-------------------------------------------+---------+-------+
>
>
I find the new formatting less clear than before.  I think I'll leave it
for now, but I'd be interested in an alternative.  Maybe each message
should have its own separate table?

-- 
Russell Bryant