From 7ee08ea0b486b651b9e4300a8f450e9eeb2955f7 Mon Sep 17 00:00:00 2001 From: fujinhua Date: Sat, 6 Apr 2019 08:52:10 +0800 Subject: [PATCH] Refactor codes for scale vnf to level Change-Id: Ia003535e6b0c8b0ed98d79b33574769c887ebaba Issue-ID: VFC-1306 Signed-off-by: fujinhua --- lcm/lcm/nf/views/scale_to_level_view.py | 57 +++++++++++---------------------- 1 file changed, 19 insertions(+), 38 deletions(-) diff --git a/lcm/lcm/nf/views/scale_to_level_view.py b/lcm/lcm/nf/views/scale_to_level_view.py index 9ae2cd36..8d4f41cb 100644 --- a/lcm/lcm/nf/views/scale_to_level_view.py +++ b/lcm/lcm/nf/views/scale_to_level_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 @@ -22,11 +21,14 @@ from rest_framework.views import APIView from lcm.nf.serializers.scale_vnf_to_level_request_serializer import ScaleVnfToLevelRequestSerializer 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 lcm.nf.biz.scale_vnf_to_level import ScaleVnfToLevel +from .common import view_safe_call_with_log logger = logging.getLogger(__name__) @@ -41,53 +43,32 @@ class ScaleVnfToLevelView(APIView): status.HTTP_500_INTERNAL_SERVER_ERROR: "Internal error" } ) + @view_safe_call_with_log(logger=logger) def post(self, request, instanceid): logger.debug("ScaleVnfToLevel--post::> %s" % request.data) - try: - scale_to_level_serializer = ScaleVnfToLevelRequestSerializer(data=request.data) - if not scale_to_level_serializer.is_valid(): - raise NFLCMException(scale_to_level_serializer.errors) - job_id = JobUtil.create_job('NF', 'SCALE_TO_LEVEL', instanceid) - JobUtil.add_job_status(job_id, 0, "SCALE_VNF_TO_LEVEL_READY") - self.scale_pre_check(instanceid, job_id) + scale_to_level_serializer = ScaleVnfToLevelRequestSerializer(data=request.data) + if not scale_to_level_serializer.is_valid(): + raise NFLCMException(scale_to_level_serializer.errors) - ScaleVnfToLevel(scale_to_level_serializer.data, instanceid, job_id).start() + job_id = JobUtil.create_job('NF', 'SCALE_TO_LEVEL', instanceid) + JobUtil.add_job_status(job_id, 0, "SCALE_VNF_TO_LEVEL_READY") + self.scale_pre_check(instanceid, job_id) - response = Response(data={"jobId": job_id}, - status=status.HTTP_202_ACCEPTED) - 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) + ScaleVnfToLevel(scale_to_level_serializer.data, instanceid, job_id).start() + + response = Response(data={"jobId": job_id}, + status=status.HTTP_202_ACCEPTED) + return response def scale_pre_check(self, nf_inst_id, job_id): vnf_insts = NfInstModel.objects.filter(nfinstid=nf_inst_id) if not vnf_insts.exists(): raise NFLCMExceptionNotFound("VNF nf_inst_id does not exist.") + if vnf_insts[0].status != 'INSTANTIATED': + raise NFLCMExceptionConflict("VNF instantiationState is not INSTANTIATED.") + vnf_insts.update(status=VNF_STATUS.SCALING) JobUtil.add_job_status(job_id, 15, 'Nf scaling to level pre-check finish') logger.info("Nf scaling to level pre-check finish") -- 2.16.6