yahoo-eng-team team mailing list archive

[Brant Knudson <bknudson@xxxxxxxxxx>]

Public bug reported:

There have been several removals of code due to

1) switching to oslo libraries from oslo-incubator
2) not testing old versions of python-keystoneclient
3) deprecations getting removed, etc.

Along with these changes, one should look through requirements.txt and
test-requirements.txt to see if the requirements are still used, but it
looks like this hasn't happened so now we've got a bunch of things in
*requirements.txt which should be removed.

Having extra requirements listed makes it so that downstream packagers
might have extra unnecessary requirements on their packages. Also, if
the requirement isn't used by keystone or any of the other dependencies
then it won't need to be installed in the tox venv and make testing
faster.

** Affects: keystone
     Importance: Undecided
     Assignee: Brant Knudson (blk-u)
         Status: In Progress

** Changed in: keystone
     Assignee: (unassigned) => Brant Knudson (blk-u)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1408423

Title:
  Requirements are out of date

Status in OpenStack Identity (Keystone):
  In Progress

Bug description:
  There have been several removals of code due to

  1) switching to oslo libraries from oslo-incubator
  2) not testing old versions of python-keystoneclient
  3) deprecations getting removed, etc.

  Along with these changes, one should look through requirements.txt and
  test-requirements.txt to see if the requirements are still used, but
  it looks like this hasn't happened so now we've got a bunch of things
  in *requirements.txt which should be removed.

  Having extra requirements listed makes it so that downstream packagers
  might have extra unnecessary requirements on their packages. Also, if
  the requirement isn't used by keystone or any of the other
  dependencies then it won't need to be installed in the tox venv and
  make testing faster.

To manage notifications about this bug go to:
https://bugs.launchpad.net/keystone/+bug/1408423/+subscriptions