LinuxBridgePluginV2.py is not consistent with naming rule

Bug #1020468 reported by yong sheng gong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Gary Kotton

Bug Description

Modules should have short, all-lowercase names. Underscores can be used in the module name if it improves readability. Python packages should also have short, all-lowercase names, although the use of underscores is discouraged.

We just let the v1 alone since we will drop all v1 codes.

http://www.python.org/dev/peps/pep-0008/

Revision history for this message
Gary Kotton (garyk) wrote :

I think that we can address this when the V1 support is dropped. Do you agree?

Revision history for this message
Gary Kotton (garyk) wrote :

Please note that this update also requires devstack support

tags: added: low-hanging-fruit
Revision history for this message
yong sheng gong (gongysh) wrote :

I think we should fix it as soon as possible, because as time passes by, others will depend on it, such as tempest and devstack codes.

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/9261

Changed in quantum:
assignee: nobody → Gary Kotton (garyk)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/9261
Committed: http://github.com/openstack/quantum/commit/e39fbbcedb1653c68771af1ea28face10a7be79f
Submitter: Jenkins
Branch: master

commit e39fbbcedb1653c68771af1ea28face10a7be79f
Author: Gary Kotton <email address hidden>
Date: Tue Jul 3 06:34:20 2012 -0400

    Fix linux bridge plugin to be consistent with naming rules

    Fixes bug 1020468

    Change-Id: I4a84e6c89c450032a791b9f91bc5e4dac6c9f647

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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.