Fuel bootstrap building is broken due to absence of oslo_log module

Bug #1594850 reported by Albert Syriy
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Critical
Alexander Gordeev

Bug Description

Fuel bootstrap building is broken due to absence of oslo_log module.
Attempt to build the Ubuntu bootstrap fails with the message
====
[root@fuel ~]# fuel-bootstrap build
Traceback (most recent call last):
 File "/usr/bin/fuel-bootstrap", line 10, in <module>
   sys.exit(main())
 File "/usr/lib/python2.7/site-packages/fuel_bootstrap/main.py", line 57, in main
   return FuelBootstrap().run(argv)
 File "/usr/lib/python2.7/site-packages/cliff/app.py", line 255, in run
   result = self.run_subcommand(remainder)
 File "/usr/lib/python2.7/site-packages/cliff/app.py", line 337, in run_subcommand
   subcommand = self.command_manager.find_command(argv)
 File "/usr/lib/python2.7/site-packages/cliff/commandmanager.py", line 75, in find_command
   cmd_factory = cmd_ep.resolve()
 File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2361, in resolve
   module = __import__(self.module_name, fromlist=['__name__'], level=0)
 File "/usr/lib/python2.7/site-packages/fuel_bootstrap/commands/build.py", line 19, in <module>
   from fuel_bootstrap.utils import bootstrap_image as bs_image
 File "/usr/lib/python2.7/site-packages/fuel_bootstrap/utils/bootstrap_image.py", line 25, in <module>
   from fuel_agent import manager
 File "/usr/lib/python2.7/site-packages/fuel_agent/manager.py", line 25, in <module>
   from fuel_agent.utils import artifact as au
 File "/usr/lib/python2.7/site-packages/fuel_agent/utils/artifact.py", line 24, in <module>
   from fuel_agent.utils import utils
 File "/usr/lib/python2.7/site-packages/fuel_agent/utils/utils.py", line 38, in <module>
   from oslo_log import log as logging
ImportError: No module named oslo_log
====

Tags: need-info
Revision history for this message
Albert Syriy (asyriy) wrote :

The very most plausible reason of the fault is the commit
https://github.com/openstack/fuel-agent/commit/6966bed353dbce15d2852789d75bda4abde0c73c

Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Changed in fuel:
status: New → In Progress
Revision history for this message
Alexander Gordeev (a-gordeev) wrote :

From first look it seems that the reason is that RPM spec didn't contain oslo-log dependency entry

https://github.com/openstack/fuel-agent/blob/master/specs/fuel-agent.spec#L24-L25

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-agent (master)

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

Revision history for this message
Alexander Gordeev (a-gordeev) wrote :

I've checked, RPM spec should be updated to reflect the changes in dependencies.

DEB spec could be left untouched just because dh-python will automatically add dependencies from requirements.txt.

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

Reviewed: https://review.openstack.org/332251
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=d40821e9a7b12a7ec6045c765c0e4b65f6df3239
Submitter: Jenkins
Branch: master

commit d40821e9a7b12a7ec6045c765c0e4b65f6df3239
Author: Alexander Gordeev <email address hidden>
Date: Tue Jun 21 19:55:28 2016 +0300

    Fix order of imports

    oslo_* imports belong to 3rd party libraries and should be imported
    before local imports.

    Change-Id: I5773343ef53741c28414fa738b90717f5eacab53
    Partial-Bug: #1594850

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-agent (master)

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

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

Reviewed: https://review.openstack.org/332757
Committed: https://git.openstack.org/cgit/openstack/fuel-agent/commit/?id=72a7f39573dcb49f0d91f1841d6266259f0e5f4d
Submitter: Jenkins
Branch: master

commit 72a7f39573dcb49f0d91f1841d6266259f0e5f4d
Author: Alexander Gordeev <email address hidden>
Date: Wed Jun 22 14:25:07 2016 +0300

    Add oslo_log requirement to RPM spec

    fuel-agent recently was graduaded to oslo_log,
    but RPM spec wasn't updated

    Change-Id: I15d3f9c817f452deb6f887163009ad5d7d6b7715
    Closes-Bug: #1594850

Changed in fuel:
status: In Progress → Fix Committed
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.