yahoo-eng-team team mailing list archive

Thread
Date

[Bug 1398656] Re: ceilometer import oslo.concurrency failed issue

To: yahoo-eng-team@xxxxxxxxxxxxxxxxxxx
From: Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>
Date: Thu, 18 Dec 2014 15:59:16 -0000
Reply-to: Bug 1398656 <1398656@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

** Changed in: ceilometer
       Status: Fix Committed => Fix Released

** Changed in: ceilometer
    Milestone: None => kilo-1

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1398656

Title:
  ceilometer import oslo.concurrency failed issue

Status in OpenStack Telemetry (Ceilometer):
  Fix Released
Status in Cinder:
  In Progress
Status in Designate:
  Fix Released
Status in OpenStack Bare Metal Provisioning Service (Ironic):
  Fix Released
Status in OpenStack Identity (Keystone):
  In Progress
Status in OpenStack Compute (Nova):
  Fix Committed

Bug description:
  during ceilometer review and find jenkis failed with following
  message:

  2014-12-03 01:28:04.969 | pep8 runtests: PYTHONHASHSEED='0'
  2014-12-03 01:28:04.969 | pep8 runtests: commands[0] | flake8
  2014-12-03 01:28:04.970 |   /home/jenkins/workspace/gate-ceilometer-pep8$ /home/jenkins/workspace/gate-ceilometer-pep8/.tox/pep8/bin/flake8 
  2014-12-03 01:28:21.508 | ./ceilometer/utils.py:30:1: H302  import only modules.'from oslo.concurrency import processutils' does not import a module
  2014-12-03 01:28:21.508 | from oslo.concurrency import processutils
  2014-12-03 01:28:21.508 | ^
  2014-12-03 01:28:21.508 | ./ceilometer/ipmi/platform/ipmitool.py:19:1: H302  import only modules.'from oslo.concurrency import processutils' does not import a module
  2014-12-03 01:28:21.508 | from oslo.concurrency import processutils
  2014-12-03 01:28:21.508 | ^
  2014-12-03 01:28:21.696 | ERROR: InvocationError: '/home/jenkins/workspace/gate-ceilometer-pep8/.tox/pep8/bin/flake8'
  2014-12-03 01:28:21.697 | pep8 runtests: commands[1] | flake8 --filename=ceilometer-* bin

  
  This seems 

  https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py

  should change to

  from  oslo_concurrency import processutils

To manage notifications about this bug go to:
https://bugs.launchpad.net/ceilometer/+bug/1398656/+subscriptions