[ovs-dev] [PATCH v2] Copy external_ids from Logical_Switch_Port to SB database

Russell Bryant russell at ovn.org
Mon Jun 5 19:01:08 UTC 2017


On Mon, Jun 5, 2017 at 11:13 AM, Daniel Alvarez <dalvarez at redhat.com> wrote:
> This patch makes ovn-northd copy all string-string pairs in
> external_ids column of the Logical_Switch_Port table in Northbound
> database to the equivalent column of the Port_Binding table in
> Southbound database.
>
> OpenStack Neutron will add some useful data to NB database that can be
> later read by networking-ovn-metadata-agent without the need of
> maintaining a connection to NB database. This data would include
> the CIDR's of a port or the project and device ID's which are needed
> when talking to Nova to request metadata.
> ---
>  ovn/northd/ovn-northd.c | 11 +++++++----
>  ovn/ovn-nb.xml          | 11 ++++++++++-
>  2 files changed, 17 insertions(+), 5 deletions(-)
>

This looks good to me.  Can you please append this doc addition and
add your "Signed-off-by" to the patch?

Thanks.

diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml
index f3c3212..0e0fe12 100644
--- a/ovn/ovn-sb.xml
+++ b/ovn/ovn-sb.xml
@@ -2179,7 +2179,16 @@ tcp.flags = RST;

     <group title="Common Columns">
       <column name="external_ids">
-        See <em>External IDs</em> at the beginning of this document.
+        <p>
+          See <em>External IDs</em> at the beginning of this document.
+        </p>
+
+        <p>
+          The <code>ovn-northd</code> program populates this column with
+          all entries into the <ref column="external_ids"/> column of the
+          <ref table="Logical_Switch_Port"/> table of the
+          <ref db="OVN_Northbound"/> database.
+        </p>
       </column>
     </group>
   </table>


More information about the dev mailing list