From 28fabef6ec5a88f9ee7da0f3a56628efc71347fb Mon Sep 17 00:00:00 2001 From: jiaopengju Date: Tue, 12 Nov 2019 15:59:43 +0800 Subject: [PATCH] Remove invalid exception catching List operation of verifications will never raise VerificationNotFound exception in _get_all method, we should remove it to normalize the error code. Change-Id: Ia1a6345b1e38b9ccc4246d2d39d11cc8e537cb4e --- karbor/api/v1/verifications.py | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/karbor/api/v1/verifications.py b/karbor/api/v1/verifications.py index bfe06773..deab8782 100644 --- a/karbor/api/v1/verifications.py +++ b/karbor/api/v1/verifications.py @@ -142,14 +142,11 @@ class VerificationsController(wsgi.Controller): CONF.query_verification_filters) utils.check_filters(filters) - try: - verifications = self._get_all(context, marker, limit, - sort_keys=sort_keys, - sort_dirs=sort_dirs, - filters=filters, - offset=offset) - except exception.VerificationNotFound as error: - raise exc.HTTPNotFound(explanation=error.msg) + verifications = self._get_all(context, marker, limit, + sort_keys=sort_keys, + sort_dirs=sort_dirs, + filters=filters, + offset=offset) retval_verifications = self._view_builder.detail_list(req, verifications)