Description of problem: The pure-python implementation of OVS's JSON parser is 60x slower than the C version. Parsing JSON is currently takes more time than anything else according to profiling. There is a python C extension available to speed up this parsing, but it isn't built in the packaging process. Version-Release number of selected component (if applicable): 2.6+ How reproducible: Always Steps to Reproduce: 1. Install python-openvswitch 2. 3. Actual results: Expected results: Additional info:
Um, I'm not sure why it posted that while I was still typing, but ok: Steps to reproduce: 1. Install python-openvswitch 2. Try to import ovs._json Actual Results: ImportError Expected results: Import succeeds. I have heard that packages built here: https://github.com/numansiddique/overcloud_image_for_ovn/tree/master/rpms have a separate package for python-openvswitch which includes support for the C extension, so maybe the work numans did there can be re-used.
Is this still relevant? If so, maybe we can be of service?
amuller: it is still relevant. I'll see if I can figure out how the OVS packaging even works.
Upstream patch to add the C extension to the Fedora/CentOS python-openvswitch RPMs: https://patchwork.ozlabs.org/patch/802892/
Ping. What is the current status here and what's there left to do?
*** Bug 1546756 has been marked as a duplicate of this bug. ***
These were the attempts to achieve getting the C JSON parser in by default: * https://patchwork.ozlabs.org/patch/699900/ * https://patchwork.ozlabs.org/patch/802892/ It'd be nice to have in OVS 2.9 :)
Hi, Looks like the direction upstream was to be a subpackage of OVS and not a standalone new package. Another important thing is that upstream compiles as shared libraries while RHEL is build statically. We will need to come up with a way to change to shared while still linking statically with DPDK. fbl
to verify, try import ovs._json, if it succeeds, the C json extensions were built.
According to our records, this should be resolved by openvswitch-2.9.0-56.el7fdp. This build is available now.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2019:0552