ldappool is not present in requirements.txt

Bug #1366062 reported by Ajaya Agrawal
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Won't Fix
Undecided
Ajaya Agrawal

Bug Description

ldappool package is imported in keystone/common/ldap/core.py

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

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

Changed in keystone:
assignee: nobody → Ajaya Agrawal (ajayaa)
status: New → In Progress
Revision history for this message
Brant Knudson (blk-u) wrote :

It's in test-requirements.txt.

Changed in keystone:
status: In Progress → Won't Fix
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on keystone (master)

Change abandoned by Dolph Mathews (<email address hidden>) on branch: master
Review: https://review.openstack.org/119392
Reason: see above

Revision history for this message
David Comay (comay) wrote :

Perhaps a silly question but is there a reason this shouldn't be in the requirements.txt file instead of the test-requirements.txt? It is, after all, a run-time dependency for Keystone.

Revision history for this message
Ajaya Agrawal (ajayaa) wrote :

@David Brant and Dolph would be the best person to answer to this question. I thought the same and proposed a fix earlier which was rejected as you can see from the description of the bug.

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.