Magnum

RequestContextSerializer class is defined twice

Bug #1459090 reported by Bradley Jones
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
Undecided
Bradley Jones
Magnum mitaka-1

Bug Description

The RequestContextSerializer class is defined in both of the following files:
 - magnum/common/rpc.py
 - magnum/common/rpc_service.py

rpc_service.py already imports rpc.py so it should just use the RequestContextSerializer from there

Bradley Jones (bradjones)
Changed in magnum:
assignee: nobody → Bradley Jones (bradjones)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to magnum (master)

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

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

Reviewed: https://review.openstack.org/185837
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=58a13e38a41e026840255ddcb5c382003832f3c5
Submitter: Jenkins
Branch: master

commit 58a13e38a41e026840255ddcb5c382003832f3c5
Author: Bradley Jones <email address hidden>
Date: Wed May 27 05:40:46 2015 +0100

    Only define RequestContextSerializer once

    rpc_service.py is already importing rpc.py so it should reuse
    RequestContextSerializer rather than redefining it.

    Change-Id: Ie8a6deca7e7c238b8076a8bbbe9598a025222aa1
    Closes-Bug: #1459090

Changed in magnum:
status: In Progress → Fix Committed
Adrian Otto (aotto)
Changed in magnum:
milestone: none → mitaka-1
status: Fix Committed → Fix Released
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.