Now that we don't test generate conf, we broke it with storpool driver
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Committed
|
High
|
John Griffith |
Bug Description
While setting up a job to perform nightly sample.conf generation and publish results to a web-page, discovered that a commit made a few days ago:
Broke the ability to parse config options in Cinder. If you try and run 'tox -egenconfig' you get this:
git/cinder.reviews - [master] » tox -egenconfig
genconfig create: /Users/
genconfig installdeps: -r/Users/
genconfig develop-inst: /Users/
genconfig runtests: commands[0] | /Users/
Error importing module cinder.
Traceback (most recent call last):
File "/System/
"__main__", fname, loader, pkg_name)
File "/System/
exec code in run_globals
File "/Users/
main()
File "/Users/
generate(
File "/Users/
raise RuntimeError(
RuntimeError: Unable to import module cinder.
Can not generate /Users/
ERROR: InvocationError: '/Users/
_______
ERROR: genconfig: commands failed
git/cinder.reviews - [master●] »
This appears to be due to an import in the storpool driver that doesn't exist in requirements.
description: | updated |
summary: |
- Now that we don't test generate conf, we broke it + Now that we don't test generate conf, we broke it with storpool driver |
Changed in cinder: | |
status: | New → Confirmed |
importance: | Undecided → High |
Fix proposed to branch: master /review. openstack. org/143570
Review: https:/