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

Bug #1568718 reported by ZongKai LI
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
ZongKai LI

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/303929

Changed in neutron:
assignee: nobody → ZongKai LI (lzklibj)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

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 <email address hidden>
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
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/neutron 9.0.0.0b1

This issue was fixed in the openstack/neutron 9.0.0.0b1 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.