Remove deprecated methods from assignment manager

Bug #1482662 reported by Brant Knudson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Wishlist
Brant Knudson

Bug Description

The assignment manager doesn't provide a stable interface (yet), so there's no need to go through deprecation.

Also, leaving the deprecated methods around makes it too easy for developers of new features to use the old deprecated method rather than the new one.

Brant Knudson (blk-u)
Changed in keystone:
assignee: nobody → Brant Knudson (blk-u)
Changed in keystone:
status: New → In Progress
Revision history for this message
Brant Knudson (blk-u) wrote :
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/210174
Committed: https://git.openstack.org/cgit/openstack/keystone/commit/?id=c7f2ec44b9f0d813b838459817c25f1fe3007b5e
Submitter: Jenkins
Branch: master

commit c7f2ec44b9f0d813b838459817c25f1fe3007b5e
Author: Brant Knudson <email address hidden>
Date: Thu Aug 6 20:04:31 2015 -0500

    Remove deprecated methods from assignment.Manager

    assignment.Manager provided several methods that forwarded calls to
    the role and resource managers. These are removed.

    assignment.Manager's interface isn't stable anyways so there was
    no need to mark these as deprecated.

    DocImpact
    ReleaseNoteImpact

    Closes-Bug: 1482662
    Change-Id: Ie2340a2fd0e7acef4e285db23a575ee5824513b9

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