versionutils.deprecated is used to mark callables as deprecated

Bug #1447472 reported by Dave Chen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Medium
Dave Chen

Bug Description

https://github.com/openstack/keystone/blob/master/keystone/token/providers/common.py#L420 ~ #L423 intends to use versionutils.deprecated to deprecate the passing of `extras`, this is not the valid case versionutils.deprecated could be applied.

see: https://github.com/openstack/keystone/blob/master/keystone/openstack/common/versionutils.py#L165 for where `versionutils.deprecated` could be used.

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/176646

Changed in keystone:
assignee: nobody → Dave Chen (wei-d-chen)
status: New → In Progress
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

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

commit 94aaf34bf65d75968d67b46f637561ea5f71870d
Author: Dave Chen <email address hidden>
Date: Thu Apr 23 15:17:38 2015 +0800

    Fix the misuse of `versionutils.deprecated`

    Co-Authored-By:David Stanek <email address hidden>
    Closes-Bug: #1447472
    Change-Id: I819d16f0bbcc8a1e06166a7cb58201833b939a08

Changed in keystone:
status: In Progress → Fix Committed
Changed in keystone:
milestone: none → liberty-1
Dave Chen (wei-d-chen)
description: updated
Changed in keystone:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: liberty-1 → 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.