dynamic nova.conf generation does not work without python-barbicanclient installed

Bug #1453268 reported by Matthew Thode
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Low
Unassigned
Kilo
Invalid
Undecided
Unassigned
oslo.config
Invalid
Undecided
Unassigned

Bug Description

In gentoo we dynamically generate the docs (we also package stable branch releases that install from git). This doesn't work unless python-barbicanclient is installed.

What I would like to see is either a silent failure with that section of nova.conf removed or just add python-barbicanclient to requirements from test-requirements. I understand that it is expected that example conf generation is expected to need test-requirements, but I don't agree :P

Let me know if you need more info.

Revision history for this message
Joe Gordon (jogo) wrote :

IMHO, we should be able to fail gracefully if some part of the sample config generation fails. Barbican isn't needed by default so I don't think we should move it into requirements. This whole thing is yet another manifestation of having optional dependencies and how to represent that.

Changed in nova:
status: New → Confirmed
importance: Undecided → Low
tags: added: low-hanging-fruit
Revision history for this message
Joe Gordon (jogo) wrote :

Issue discovered on stable/kilo

Revision history for this message
Matt Riedemann (mriedem) wrote :

python-barbicanclient is in test-requirements.txt, the readme says to use tox -e genconfig, this isn't nova's problem.

If you're not using tox, then install the packages via site-packages and pip, or rpm, or apt-get, or whatever your distro builds require.

Changed in nova:
status: Confirmed → Invalid
Changed in oslo.config:
status: New → Invalid
Revision history for this message
Matthew Thode (prometheanfire) wrote :

Here's the snippet from tox.ini

[testenv:genconfig]
commands = bash tools/config/generate_sample.sh -b . -p nova -o etc/nova

That command has this line in it, it makes me think that tests should not be needed...

FILES=$(find $TARGETDIR -type f -name "*.py" ! -path "*/tests/*" \
              -exec grep -l "Opt(" {} + | sed -e "s/^$BASEDIRESC\///g" | sort -u)

Changed in nova:
status: Invalid → New
Changed in oslo.config:
status: Invalid → New
Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

But tox.ini has both requirements.txt and test-requirements.txt as dependencies, so you need to pull all the packages.

Changed in nova:
status: New → Invalid
Changed in oslo.config:
status: New → Invalid
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.