Cinder

pep8 test should skip openstack-common

Bug #1027774 reported by Huang Zhiteng on 2012-07-23

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Cinder	Fix Released	Undecided	Unassigned	Cinder 2012.2 "folsom"

Bug Description

Pep8 test report issue on openstack-common modules, which would fail CI test. Pep8 test should skip openstack-common and assuming they are problem-free code. Even it's not, bug/fix should go to openstack-common first, then resync back to Cinder.

[zhiteng@zhuan10-mobl5 cinder]$ ./run_tests.sh -p
Running PEP8 and HACKING compliance check...
cinder/openstack/common/gettextutils.py:32:7: CINDER N701: Empty localization string

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2012-07-24: Fix merged to cinder (master)

Reviewed: https://review.openstack.org/10118
Committed: http://github.com/openstack/cinder/commit/612b1bd95510252721ba1a0531bd27e2706cad1d
Submitter: Jenkins
Branch: master

commit 612b1bd95510252721ba1a0531bd27e2706cad1d
Author: Zhiteng Huang <email address hidden>
Date: Mon Jul 23 14:39:48 2012 +0800

Don't do PEP8 test for openstack-common code.

    Pep8 test reports issue on openstack-common modules, which would
    fail CI test. Pep8 test should skip openstack-common and assuming
    they are problem-free code. Even it's not, bug/fix should go to
    openstack-common first, then resync back to Cinder. Therefore,
    Cinder should skip pep8 test for openstack-common code.

Fix bug #1027774

Change-Id: I22c45542401022a6bad9f2320e8657f972915f69