← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1568718] [NEW] ovsdb.native.connection.Connection should allow schema_helper to register certain tables, instead of all

 

Public bug reported:

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

** Affects: neutron
     Importance: Undecided
         Status: New

-- 
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:
  New

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


Follow ups