utils.runthis has unused argument and unused valiables

Reported by Nachi Ueno on 2011-11-01
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Low
Russell Bryant
OpenStack Quality Assurance
Confirmed
Low
Unassigned

Bug Description

Nachi Ueno (nati-ueno) on 2011-11-01
Changed in openstack-qa:
assignee: nobody → Nachi Ueno (nati-ueno)
Changed in nova:
assignee: nobody → Nachi Ueno (nati-ueno)
Changed in openstack-qa:
importance: Undecided → Low
status: New → Confirmed
Thierry Carrez (ttx) on 2011-11-10
Changed in nova:
importance: Undecided → Low
status: New → Confirmed
Nachi Ueno (nati-ueno) on 2011-11-18
Changed in nova:
assignee: Nachi Ueno (nati-ueno) → nobody
Changed in openstack-qa:
assignee: Nachi Ueno (nati-ueno) → nobody

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

Changed in nova:
assignee: nobody → Russell Bryant (russellb)
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/3587
Committed: http://github.com/openstack/nova/commit/d22b0210b0c4ef80b64531e6826985341e9af838
Submitter: Jenkins
Branch: master

commit d22b0210b0c4ef80b64531e6826985341e9af838
Author: Russell Bryant <email address hidden>
Date: Tue Jan 31 14:48:12 2012 -0500

    Remove utils.runthis().

    Fixes bug 884825. The bug was pointing out some unused variables
    noticed by pylint.

    utils.runthis() was only used in one place and doesn't really provide
    much value over utils.execute(), so just remove it.

    Change-Id: I3dcc34bd840a128418b58a8a66acb62adf3fc9cc

Changed in nova:
status: In Progress → Fix Committed
Changed in nova:
milestone: none → essex-4
Thierry Carrez (ttx) on 2012-02-29
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in nova:
milestone: essex-4 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers