From a4af27a705926c5a9f6913b3eca2f3f2298d9db5 Mon Sep 17 00:00:00 2001 From: Dan Prince Date: Wed, 12 Jun 2013 09:21:10 -0400 Subject: [PATCH] Log cms_verify issues as warnings (not errors). In general we probably do want to log these types of errors... but not as ERRORS. I think log Warning messages are probably more appropriate here since it would be possible to hit this when expired certs are refreshed... Fixes LP Bug #1190230. Change-Id: I0383f7e490ddcdfb31a62cd3760102152a8d16d9 --- keystoneclient/common/cms.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/keystoneclient/common/cms.py b/keystoneclient/common/cms.py index 48b991a46..3fca24ced 100644 --- a/keystoneclient/common/cms.py +++ b/keystoneclient/common/cms.py @@ -41,7 +41,7 @@ def cms_verify(formatted, signing_cert_file_name, ca_file_name): output, err = process.communicate(formatted) retcode = process.poll() if retcode: - LOG.error('Verify error: %s' % err) + LOG.warning('Verify error: %s' % err) # NOTE(dmllr): Python 2.6 compatibility: # CalledProcessError did not have output keyword argument e = subprocess.CalledProcessError(retcode, "openssl")