exclude xenapi plugins from pep8/hacking checks

Bug #987055 reported by Dan Prince
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Dan Prince

Bug Description

The Xenapi plugins are currently written in Python 2.4. As such they do not satisfy all of our pep8 checks.

Lets just exclude them for now. These plugins arguably shouldn't be in Nova's code base anyway.

Revision history for this message
Dan Prince (dan-prince) wrote :

The offending file is 'glance' plugin which makes use of md5.

tools/hacking.py:122: DeprecationWarning: the md5 module is deprecated; use hashlib instead
  __import__(mod, globals(), locals(), [], -1)

Changed in nova:
assignee: nobody → Dan Prince (dan-prince)
importance: Undecided → Low
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

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

Reviewed: https://review.openstack.org/6708
Committed: http://github.com/openstack/nova/commit/2b490fd2ef3a74402147893be646c00505da8628
Submitter: Jenkins
Branch: master

commit 2b490fd2ef3a74402147893be646c00505da8628
Author: Dan Prince <email address hidden>
Date: Sun Apr 22 21:11:03 2012 -0400

    Exclude xenapi plugins from pep8/hacking checks.

    Fixes LP Bug #987055.

    Change-Id: I7132b88c5e0a90b16d0f8e35df45b82e7991280f

Changed in nova:
status: In Progress → Fix Committed
Devin Carlen (devcamcar)
Changed in nova:
milestone: none → folsom-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-1 → 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.