Fuel for OpenStack

Use oslo.serialization

Bug #1400661 reported by Sebastian Kalinowski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Maciej Kwiek
Fuel for OpenStack 6.1

Bug Description

In Fuel we use old version on OSLO's jsonutils from time when it was still in oslo-incubator [1].

Now this library live as oslo.serialization [2] and we should switch to it.

oslo.serialization is already available in our repos in version 1.0.0 but it is worth to consider an update of it to the newest version if possible.

[1] https://github.com/stackforge/fuel-web/blob/master/nailgun/nailgun/openstack/common/jsonutils.py
[2] https://github.com/openstack/oslo.serialization

Vladimir Kuklin (vkuklin)
Changed in fuel:
status: New → Confirmed
Sebastian Kalinowski (prmtl)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Maciej Kwiek (maciej-iai)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-main (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/158674

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/159032

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/159032
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=1523c0701afcda8a34f044818a612898f7b299d3
Submitter: Jenkins
Branch: master

commit 1523c0701afcda8a34f044818a612898f7b299d3
Author: Maciej Kwiek <email address hidden>
Date: Wed Feb 25 08:55:38 2015 +0100

    Replace jsonutils with oslo.serialization version

    jsonutils is now in separate library, and the oslo.serialization will be
    used from now on.

    Change-Id: I28b6b70e437f154cf1b9ce0b15b0c3e716d170ee
    Depends-On: https://review.openstack.org/#/c/158674/
    Closes-Bug: #1400661

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-main (master)

Reviewed: https://review.openstack.org/158674
Committed: https://git.openstack.org/cgit/stackforge/fuel-main/commit/?id=eb4298591ecb86c16826cbb60a37d231a9af2cfd
Submitter: Jenkins
Branch: master

commit eb4298591ecb86c16826cbb60a37d231a9af2cfd
Author: Maciej Kwiek <email address hidden>
Date: Tue Feb 24 13:43:33 2015 +0100

    Add dependency on oslo-serialization for nailgun

    Nailgun needs to use oslo jsonutils as a package instead of having it
    hardcoded in its own repository.

    There will be commits in nailgun that will migrate the code to use
    oslo.serialization lib.

    Change-Id: I3b042405b605c352a8169ea63b46ce7a38ed90b6
    Related-Bug: #1400661

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.