Remove unused imports

Bug #1037578 reported by Rongze Zhu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Wishlist
Rongze Zhu
openstack-common
Fix Released
Undecided
Rongze Zhu

Bug Description

Remove unused imports

cd **/keystone/keystone/
find . -type f -name "*py" -exec pylint -r n {} \; | grep -E "Unused imp|^\*"

************* Module openstack.common.timeutils
W: 24,0: Unused import time
No config file found, using default configuration

************* Module common.sql.migrate_repo.versions.001_add_initial_tables
W: 17,0: Unused import UsageError from wildcard import
W: 18,0: Unused import text from wildcard import
................
************* Module common.cms
W: 2,0: Unused import stat
W: 1,0: Unused import os
************* Module common.utils
W: 27,0: Unused import sys

Revision history for this message
Rongze Zhu (zrzhit) wrote :

cd **/openstack-common/openstack/
find . -type f -name "*py" -exec pylint -r n {} \; | grep -E "Unused imp|^\*"

************* Module common.plugin.pluginmanager
W: 16,0: Unused import imp
W: 17,0: Unused import os
************* Module common.rpc.common
W: 25,0: Unused import cfg
W: 22,0: Unused import sys
************* Module common.notifier.api
W: 16,0: Unused import inspect

description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Changed in keystone:
assignee: nobody → Rongze Zhu (zrzhit)
status: New → In Progress
Changed in openstack-common:
assignee: nobody → Rongze Zhu (zrzhit)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-common (master)

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

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

Reviewed: https://review.openstack.org/11465
Committed: http://github.com/openstack/openstack-common/commit/6a22ea45edc0f0263731944f2ad0ffe15c8a9457
Submitter: Jenkins
Branch: master

commit 6a22ea45edc0f0263731944f2ad0ffe15c8a9457
Author: RongzeZhu <email address hidden>
Date: Thu Aug 16 22:29:31 2012 +0800

    Remove unused imports

    Fixes bug #1037578

    Change-Id: I87dbd0bc5fe16a4416a676dc26f2446a54509bc1

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

Reviewed: https://review.openstack.org/11463
Committed: http://github.com/openstack/keystone/commit/64bbaf51e6aa1bbab4be2405a99f9b116e2e8f32
Submitter: Jenkins
Branch: master

commit 64bbaf51e6aa1bbab4be2405a99f9b116e2e8f32
Author: RongzeZhu <email address hidden>
Date: Thu Aug 16 22:02:48 2012 +0800

    Remove unused imports

    Fixes bug #1037578

    Change-Id: I2fdaf83e731e1b839eb0bb2ee2f0f55d7bd40f00

Changed in keystone:
status: In Progress → Fix Committed
Joseph Heck (heckj)
Changed in keystone:
importance: Undecided → Wishlist
milestone: none → folsom-rc1
Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: folsom-rc1 → 2012.2
Mark McLoughlin (markmc)
Changed in openstack-common:
milestone: none → 2012.2
status: Fix Committed → Fix Released
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.