diff options
| author | Joe Gordon <jogo@cloudscaling.com> | 2013-05-16 20:49:42 +0000 |
|---|---|---|
| committer | Joe Gordon <jogo@cloudscaling.com> | 2013-05-16 20:49:42 +0000 |
| commit | dc887267572c821ce97bec9536a699b72bdae443 (patch) | |
| tree | 1dbe503fea078fbf6f94edb1761cb3bf2ffaa606 | |
| parent | a87b4e36e9a0ce6d565cc9ea38fcff523271e9ff (diff) | |
| download | nova-dc887267572c821ce97bec9536a699b72bdae443.tar.gz nova-dc887267572c821ce97bec9536a699b72bdae443.tar.xz nova-dc887267572c821ce97bec9536a699b72bdae443.zip | |
Don't raise unnecessary stack traces in EC2 API
exception.EC2APIError exceptions are always expected, so no reason to have a
LOG.exception when one is raised. Furthermore nova.api.ec2.ec2_error already
logs the exception so the duplicate log statement is just removed
Fix Bug 1180973
Change-Id: I4b42a9a764f6e15bb94a383e8a725d52d7436bd7
| -rw-r--r-- | nova/api/ec2/__init__.py | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/nova/api/ec2/__init__.py b/nova/api/ec2/__init__.py index 7ce18685b..f79057167 100644 --- a/nova/api/ec2/__init__.py +++ b/nova/api/ec2/__init__.py @@ -501,8 +501,6 @@ class Executor(wsgi.Application): LOG.info(_('NotFound raised: %s'), unicode(ex), context=context) return ec2_error(req, request_id, type(ex).__name__, unicode(ex)) except exception.EC2APIError as ex: - LOG.exception(_('EC2APIError raised: %s'), unicode(ex), - context=context) if ex.code: return ec2_error(req, request_id, ex.code, unicode(ex)) else: |