neutron

cleanup unused util functions

Bug #1007557 reported by dan wendlandt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Yaguang Tang
neutron 2012.2 "folsom"

Bug Description

some functions in quantum/common/utils.py (e.g., to_primitive) are not used in quantum, and were likely copied over to quantum from nova. We should identify and remove unused functions.

Code coverage from unit tests may help with this, though its possible that we have unit tests for code that is still not actually used by quantum.

dan wendlandt (danwent)
tags: added: low-hanging-fruit
Yaguang Tang (heut2008)
Changed in quantum:
assignee: nobody → Yaguang Tang (heut2008)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

Changed in quantum:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/8100
Committed: http://github.com/openstack/quantum/commit/1f405a2b121eaedb91b9a9c89cc820cd3aa1a615
Submitter: Jenkins
Branch: master

commit 1f405a2b121eaedb91b9a9c89cc820cd3aa1a615
Author: Yaguang Tang <email address hidden>
Date: Tue Jun 5 13:48:01 2012 +0800

    fix bug lp:1007557,remove unused functions in utils.py.

    Change-Id: Ic8cfe61effd357c49f651fd6cd6e6b721f6addd1

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
milestone: none → folsom-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in quantum:
milestone: folsom-2 → 2012.2
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.