Comment 10 for bug 1605573

Revision history for this message
Numan Siddique (numansiddique) wrote :

Please note that the ovs python master and the 2.6 (https://pypi.python.org/pypi/ovs/2.6.0.dev2) is broken. As networking-ovn CI uses the ovs and ovn master code, the ovs python IDL will use the ovsdb update2 method by default. The recent patches to use update2 in ovs python has a bug because of which networking_ovn/ovsdb/ovsdb_monitor.py will not get the port up and down notification as expected.
Please see [1] for more details.

I have submitted a patch [2] to use the ovs python from master to confirm the failures. If we want to use the fix - "python: add set type for ovs.idl.data.Datum.from_python" we also need [1] to be merged.

[1] - https://patchwork.ozlabs.org/patch/651140/
[2] - https://review.openstack.org/#/c/280591/4