X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=lcm%2Flcm%2Fnf%2Fviews%2Fheal_vnf_view.py;h=24b33e34d3d1501b96de6d81651fd0d2c2f1a8d3;hb=fd8aa2a4eec4a6d8e7775b5bd63454694c3ff0a4;hp=6b3b31397e9b2569a0a6c2decad683d0e04a6a5b;hpb=d31d7c3c00b7facb05a17906e9152ee076d15a03;p=vfc%2Fgvnfm%2Fvnflcm.git

diff --git a/lcm/lcm/nf/views/heal_vnf_view.py b/lcm/lcm/nf/views/heal_vnf_view.py
index 6b3b3139..24b33e34 100644
--- a/lcm/lcm/nf/views/heal_vnf_view.py
+++ b/lcm/lcm/nf/views/heal_vnf_view.py
@@ -13,7 +13,6 @@
 # limitations under the License.
 
 import logging
-import traceback
 
 from drf_yasg.utils import swagger_auto_schema
 from rest_framework import status
@@ -23,10 +22,13 @@ from rest_framework.views import APIView
 from lcm.nf.biz.heal_vnf import HealVnf
 from lcm.nf.serializers.heal_vnf_req import HealVnfRequestSerializer
 from lcm.nf.serializers.response import ProblemDetailsSerializer
-from lcm.pub.exceptions import NFLCMException, NFLCMExceptionNotFound, NFLCMExceptionConflict
+from lcm.pub.exceptions import NFLCMException
+from lcm.pub.exceptions import NFLCMExceptionNotFound
+from lcm.pub.exceptions import NFLCMExceptionConflict
 from lcm.pub.utils.jobutil import JobUtil
 from lcm.pub.database.models import NfInstModel
 from lcm.nf.const import VNF_STATUS
+from .common import view_safe_call_with_log
 
 logger = logging.getLogger(__name__)
 
@@ -41,40 +43,22 @@ class HealVnfView(APIView):
             status.HTTP_500_INTERNAL_SERVER_ERROR: "Internal error"
         }
     )
+    @view_safe_call_with_log(logger=logger)
     def post(self, request, instanceid):
         logger.debug("HealVnf--post::> %s" % request.data)
-        try:
-            heal_vnf_request_serializer = HealVnfRequestSerializer(data=request.data)
-            if not heal_vnf_request_serializer.is_valid():
-                raise NFLCMException(heal_vnf_request_serializer.errors)
 
-            job_id = JobUtil.create_job('NF', 'HEAL', instanceid)
-            JobUtil.add_job_status(job_id, 0, "HEAL_VNF_READY")
-            self.heal_pre_check(instanceid, job_id)
-            HealVnf(heal_vnf_request_serializer.data, instanceid, job_id).start()
-            response = Response(data={"jobId": job_id}, status=status.HTTP_202_ACCEPTED)
-            # todo, heal_vnf codes uses job as the status storage, not in VNFLcmOpOccModel.
-            # response["Location"] = "/api/vnflcm/v1/vnf_lc_ops/%s" % lcmopoccid
-            return response
-        except NFLCMExceptionNotFound as e:
-            probDetail = ProblemDetailsSerializer(data={"status": status.HTTP_404_NOT_FOUND, "detail": "VNF Instance not found"})
-            resp_isvalid = probDetail.is_valid()
-            if not resp_isvalid:
-                raise NFLCMException(probDetail.errors)
-            return Response(data=probDetail.data, status=status.HTTP_404_NOT_FOUND)
-        except NFLCMExceptionConflict as e:
-            probDetail = ProblemDetailsSerializer(data={"status": status.HTTP_409_CONFLICT, "detail": "VNF Instance not in Instantiated State"})
-            resp_isvalid = probDetail.is_valid()
-            if not resp_isvalid:
-                    raise NFLCMException(probDetail.errors)
-            return Response(data=probDetail.data, status=status.HTTP_409_CONFLICT)
-        except NFLCMException as e:
-            logger.error(e.message)
-            return Response(data={'error': '%s' % e.message}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
-        except Exception as e:
-            logger.error(e.message)
-            logger.error(traceback.format_exc())
-            return Response(data={'error': 'unexpected exception'}, status=status.HTTP_500_INTERNAL_SERVER_ERROR)
+        heal_vnf_request_serializer = HealVnfRequestSerializer(data=request.data)
+        if not heal_vnf_request_serializer.is_valid():
+            raise NFLCMException(heal_vnf_request_serializer.errors)
+
+        job_id = JobUtil.create_job('NF', 'HEAL', instanceid)
+        JobUtil.add_job_status(job_id, 0, "HEAL_VNF_READY")
+        self.heal_pre_check(instanceid, job_id)
+        HealVnf(heal_vnf_request_serializer.data, instanceid, job_id).start()
+        response = Response(data={"jobId": job_id}, status=status.HTTP_202_ACCEPTED)
+        # todo, heal_vnf codes uses job as the status storage, not in VNFLcmOpOccModel.
+        # response["Location"] = "/api/vnflcm/v1/vnf_lc_ops/%s" % lcmopoccid
+        return response
 
     def heal_pre_check(self, nf_inst_id, job_id):
         vnf_insts = NfInstModel.objects.filter(nfinstid=nf_inst_id)