yahoo-eng-team team mailing list archive

[OpenStack Infra <1568718@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/303929
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=667ea2ac7c1ef8ce6f52072639a13a833533f3ea
Submitter: Jenkins
Branch:    master

commit 667ea2ac7c1ef8ce6f52072639a13a833533f3ea
Author: lzklibj <lzklibj@xxxxxxxxxx>
Date:   Mon Apr 11 15:15:05 2016 +0800

    Allow OVSDB Connection to register certain tables
    
    Current OVSDB Connection will register all tables with schema_helper.
    It doesn't matter for most cases, but for implementation for bp
    routed-networks in networking-ovn, we don't need all tables in OVN_
    Southbound DB are registered. We only need a certain table named
    Chassis can be registered.
    
    This patch add a parameter for OVSDB Connection to allow it to
    register certain tables, instead of all tables.
    
    Change-Id: I79df60a08a7a6c555b07a9b566d7c09b97e2463c
    Closes-Bug: #1568718


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1568718

Title:
  ovsdb.native.connection.Connection should allow schema_helper to
  register certain tables, instead of all

Status in neutron:
  Fix Released

Bug description:
  Patch https://review.openstack.org/#/c/302623 is partial implement for
  blueprint routed-networks, it tries to add Chassis table access in
  networking-ovn. In Chassis table, it contains column hostname and
  external_ids, which are needed for segment-hostname mapping in bp
  routed-networks.

  To access Chassis table, we need build a connection to OVN_Southbound
  DB, and the schema_helper in that connection object will register all
  tables in DB [1]. But indeed, we don't need all tables in
  OVN_Southbound DB registered. Currently, we only need table Chassis be
  registered.

  We should add a parameter for connection.Connection to allow it only
  register certain tables, and in OVS, method register_table [2] can be
  used for this purpose.

  [1] https://github.com/openstack/neutron/blob/master/neutron/agent/ovsdb/native/connection.py#L68-L82
  [2] https://github.com/open-switch/ops-openvswitch/blob/master/python/ovs/db/idl.py#L1359-L1366

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1568718/+subscriptions