]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Replace warnings.warn in sqlalchemy-api
authorAnkit Agrawal <ankit11.agrawal@nttdata.com>
Tue, 3 Nov 2015 09:43:27 +0000 (01:43 -0800)
committerAnkit Agrawal <ankit11.agrawal@nttdata.com>
Tue, 3 Nov 2015 09:45:24 +0000 (01:45 -0800)
There is a warning message logged by warnings.warn,
it should be replaced by LOG.warning.

Change-Id: Ifcff0ebeb13b07420f6b8073b094c3b1687999e8

cinder/db/sqlalchemy/api.py

index 5ed047e59fee1add39098cce026ea4924c9d54d4..47a84492e42a014f54c44819bc0ed384560abda0 100644 (file)
@@ -25,7 +25,6 @@ import sys
 import threading
 import time
 import uuid
-import warnings
 
 from oslo_config import cfg
 from oslo_db import exception as db_exc
@@ -109,8 +108,8 @@ def get_backend():
 def is_admin_context(context):
     """Indicates if the request context is an administrator."""
     if not context:
-        warnings.warn(_('Use of empty request context is deprecated'),
-                      DeprecationWarning)
+        LOG.warning(_LW('Use of empty request context is deprecated'),
+                    DeprecationWarning)
         raise Exception('die')
     return context.is_admin