keystoneclient requires unittest2 for production install

Bug #1014845 reported by Ken Thomas
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Undecided
Ken Thomas

Bug Description

We did a production install of keystone client and ran into a problem. When we fire up the keystone client, it fails with:

pkg_resources.DistributionNotFound: unittest2

It appears that the problem is that unittest2 is listed in tools/pip-requires. Yes, it's required for a dev environment, but not for a production install. Also yes, we could work around this by installing unittest2 but I'd rather fix the problem than treat the symptom.

Can we just remove that reference from pip-requires?

Revision history for this message
Ken Thomas (krt) wrote :

What do you know! I put that line in there four months ago. Looks like that should have gone into test_requires instead of pip_requires.

Changed in keystone:
assignee: nobody → Ken Thomas (krt)
Revision history for this message
Ken Thomas (krt) wrote :

Just confirmed, the fix for bug 933076 added the unittest2 dependency to the wrong file.

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

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

Changed in keystone:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-keystoneclient (master)

Reviewed: https://review.openstack.org/8675
Committed: http://github.com/openstack/python-keystoneclient/commit/1231ae03e5686da787192ddc254fcc7bd4666c66
Submitter: Jenkins
Branch: master

commit 1231ae03e5686da787192ddc254fcc7bd4666c66
Author: Ken Thomas <email address hidden>
Date: Mon Jun 18 22:19:53 2012 +0000

    Move unittest2 dependency

    bug 1014845

    It looks like an earlier fix for bug 933076 put the unittest2
    dependency in the wrong file. Moving it to the correct file.

    Added a version to the unittest2 dependency
    Removed a blank line

    Change-Id: I06a8963fc5aca80438348d26569d8efb4568f754

Changed in keystone:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in keystone:
milestone: none → folsom-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: folsom-2 → 2012.2
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.