yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #49266
[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