yahoo-eng-team team mailing list archive

[Dong Jun <dongj@xxxxxxxxxxx>]

Public bug reported:

Networking-ovn used neutron.agent.ovsdb.native.connection[1] to connect OVN_Northbound and OVN_Southbound, but it need not to enable_connection_uri when get_schema_helper, thus get_schema_helper was overrided[2].
Latter the main code in connect.get_schema_helper was moved to idlutils.get_schema_helper[3], that affected OVN.
It's better to use idlutils.get_schema_helper directly, could we add a parameter to make enable_connection_uri optional in idlutils.get_schema_helper . 

[1] https://github.com/openstack/networking-ovn/blob/master/networking_ovn/ovsdb/ovsdb_monitor.py#L300
[2] https://github.com/openstack/networking-ovn/commit/70a8ecb06f398c7c177b67ab8fcafc79d2a78253
[3] https://github.com/openstack/neutron/commit/2013058e137c564459b246c3fb6c1587196f7a52

** 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/1672590

Title:
  make enable_connection_uri optional in OVSDB idlutils

Status in neutron:
  New

Bug description:
  Networking-ovn used neutron.agent.ovsdb.native.connection[1] to connect OVN_Northbound and OVN_Southbound, but it need not to enable_connection_uri when get_schema_helper, thus get_schema_helper was overrided[2].
  Latter the main code in connect.get_schema_helper was moved to idlutils.get_schema_helper[3], that affected OVN.
  It's better to use idlutils.get_schema_helper directly, could we add a parameter to make enable_connection_uri optional in idlutils.get_schema_helper . 

  [1] https://github.com/openstack/networking-ovn/blob/master/networking_ovn/ovsdb/ovsdb_monitor.py#L300
  [2] https://github.com/openstack/networking-ovn/commit/70a8ecb06f398c7c177b67ab8fcafc79d2a78253
  [3] https://github.com/openstack/neutron/commit/2013058e137c564459b246c3fb6c1587196f7a52

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