Proper deprecation for is_ans1_token

is_ans1_token wasn't properly deprecated since it used LOG.warn
rather than warnings/debtcollector. Proper deprecation requires use
of warnings and documentation.

bp deprecations

Change-Id: I81be2844014745a5951ce91a336e9e9ecf4d5328

1 files changed, 9 insertions, 4 deletions

diff --git a/keystoneclient/common/cms.py b/keystoneclient/common/cms.py
index 68af1dd..c5b35da 100644
--- a/keystoneclient/common/cms.py
+++ b/keystoneclient/common/cms.py
@@ -26,10 +26,11 @@ import logging
 import textwrap
 import zlib
 
+from debtcollector import removals
 import six
 
 from keystoneclient import exceptions
-from keystoneclient.i18n import _, _LE, _LW
+from keystoneclient.i18n import _, _LE
 
 
 subprocess = None
@@ -297,10 +298,14 @@ def is_asn1_token(token):
     return token[:3] == PKI_ASN1_PREFIX
 
 
+@removals.remove(message='Use is_asn1_token() instead.', version='1.7.0',
+                 removal_version='2.0.0')
 def is_ans1_token(token):
-    """Deprecated. Use is_asn1_token() instead."""
-    LOG.warning(_LW('The function is_ans1_token() is deprecated, '
-                    'use is_asn1_token() instead.'))
+    """Deprecated.
+
+    This function is deprecated as of the 1.7.0 release in favor of
+    :func:`is_asn1_token` and may be removed in the 2.0.0 release.
+    """
     return is_asn1_token(token)