change name of rally.common.log module

Bug #1521573 reported by Andriy Kurilin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Wishlist
Bo Chi

Bug Description

Since most of imports for rally.common.log module looks like`from rally.common import log as logging`, I suppose we should rename `rally.common.log` to `rally.common.logging`.

To import original(builtin) module from `rally.common.logging`(there will be a name conflict: local_module_name = builtin_mnodule_name) we can do something like `log = __import__("logging")`

Tags: logging
Bo Chi (bochi-michael)
Changed in rally:
assignee: nobody → Bo Chi (bochi-michael)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to rally (master)

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

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

Reviewed: https://review.openstack.org/254009
Committed: https://git.openstack.org/cgit/openstack/rally/commit/?id=79adee9c00e475d8f742470c1657ea8502566443
Submitter: Jenkins
Branch: master

commit 79adee9c00e475d8f742470c1657ea8502566443
Author: Bo Chi <email address hidden>
Date: Sun Dec 6 23:59:16 2015 -0500

    change name of rally.common.log module

    Copied `rally.common.log` to `rally.common.logging`
    Marked `rally.common.log` as deprecated

    Change-Id: Ia7926e5f8a716a8a5b74c0f46d9b797e4fe23ddc
    Closes-Bug: #1521573

Changed in rally:
status: In Progress → Fix Released
Changed in rally:
milestone: none → 0.1.2
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.