]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Drop conf_key_mgr warning message!
authorDan Prince <dprince@redhat.com>
Mon, 7 Oct 2013 16:41:28 +0000 (12:41 -0400)
committerDan Prince <dprince@redhat.com>
Mon, 7 Oct 2013 18:08:17 +0000 (14:08 -0400)
By default ConfKeyManager logs tons of WARNING message stating
that it isn't production ready...

Given that it is currently the only Cinder key manager option
which can be used/selected I don't think repeatedly logging
warnings is helpful. Lets just drop the warning message
for now and when a good "production ready" cinder key manager
implementation is implemented perhaps we can re-add a warning to
this class (hopefully making the production ready impl the default).

Change-Id: Id1fdddc20a963f9fa4749ad57f355cd83d0e14e3
Closes-Bug: #1236459

cinder/keymgr/conf_key_mgr.py

index 7b53e0cb49c1aefaddfdf2d15099f5962e889289..1c9ff2d64262d9cd70cc191bfb0bde7a7dda644f 100644 (file)
@@ -64,8 +64,6 @@ class ConfKeyManager(key_mgr.KeyManager):
     """
 
     def __init__(self):
-        LOG.warn(_('This key manager is insecure and is not recommended for '
-                   'production deployments'))
         super(ConfKeyManager, self).__init__()
 
         self.key_id = '00000000-0000-0000-0000-000000000000'