placement allocations handler module does not use util.extract_json

Bug #1695356 reported by Chris Dent
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Chris Dent

Bug Description

nova.api.openstack.placement.handlers.allocation uses and _extract_allocations method which could be remove in favor of nova.api.openstack.placement.util:extract_json

A nice tidy up, leaving as a bug as a reminder to cdent or a low hanging fruit for someone else to pick up.

Chris Dent (cdent)
Changed in nova:
assignee: nobody → Chris Dent (cdent)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

Reviewed: https://review.openstack.org/471067
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=d0957d72e25a4ca010df4807fd8224efe57df3f0
Submitter: Jenkins
Branch: master

commit d0957d72e25a4ca010df4807fd8224efe57df3f0
Author: Chris Dent <email address hidden>
Date: Mon Jun 5 17:03:50 2017 +0000

    [placement] Use util.extract_json in allocations handler

    The handler had its own version of the exact same code, so may as
    well use the utility method.

    Nothing changes at the API level with this change. Error messages
    and response codes remain the same.

    Change-Id: Ibaf7120d03fe4e8be5b9f38bc37004a43e7675aa
    Closes-Bug: #1695356

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 16.0.0.0b3

This issue was fixed in the openstack/nova 16.0.0.0b3 development milestone.

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

Other bug subscribers

Remote bug watches

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