The python-openvswitch package should build the json C extension

Bug #1672522 reported by Terry Wilson
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openvswitch (Ubuntu)

Bug Description

The C extension wrapping OVS's own json parser is 60x faster than the pure-python version currently being used. Profiling shows JSON parsing to be the biggest bottleneck using python-openvswitch IDL library.

setuptools can build the extension as long as the python-dev packages are installed and openvswitch-dev is installed. It should be available in OVS 2.6+

James Page (james-page)
Changed in openvswitch (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
James Page (james-page) wrote :

I've added the -dev BD's to the package with a recent upload; however the python native extension self depends on the package being built which I've not figured out yet; we probably need to expose some sort of libopenvswitch0{-dev} type package but I'm not sure.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package openvswitch - 2.9.0~git20180205.5a39582ca-0ubuntu1

openvswitch (2.9.0~git20180205.5a39582ca-0ubuntu1) bionic; urgency=medium

  * Snapshot from branch-2.9 (LP: #1735248, LP: #1686406, LP: #1709272):
    - d/p/dpif-kernel-gre-mtu-workaround.patch,
      d/p/ovs-netdev-dpdk-17.11-upgrade.patch: Drop, included upstream.
  * d/control: Switch BD's to python{3}-all-dev to enable json native
    extensions (LP: #1672522).
  * d/control,rules,openvswitch-doc.*: Add documentation (LP: #965283).
  * Misc tidy of packaging:
    - d/*: wrap-and-sort -bast.
    - d/control: Priority optional->extra.
    - d/openvswitch-doc.doc-base: Update Section.
    - d/copyright: Pick machine formatted version from Debian
      (thanks Thomas).
    - d/control: Add missing depends on lsb-release.

 -- James Page <email address hidden> Mon, 05 Feb 2018 17:12:07 +0000

Changed in openvswitch (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers