From cf5150571b0bab4ce5c7539b5f0ef34da1b3efd5 Mon Sep 17 00:00:00 2001 From: Sheel Rana Date: Mon, 17 May 2021 22:02:35 +0900 Subject: [PATCH] Wrong subscription notification URL Subscription notifications contains /vnflcmv1/ in URL 'vnfLcmOpOcc': { 'href': '/vnflcmv1/vnf_lcm_op_occs/%s' % vnf_lcm_op_occs_id}}} It impacts following operations - heal - terminate - instantiate - change_ext_conn Also subscription notification data->href does not contain host path. Co-Authored-By: Navum Gupta Closes-Bug: #1923478 Change-Id: Idc146eed1b2b6a8ee3d4dd6c230324a8aef006f6 --- tacker/conductor/conductor_server.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/tacker/conductor/conductor_server.py b/tacker/conductor/conductor_server.py index 8d12d7771..6a52d8d28 100644 --- a/tacker/conductor/conductor_server.py +++ b/tacker/conductor/conductor_server.py @@ -1571,13 +1571,11 @@ class Conductor(manager.Manager): 'vnfLcmOpOccId': vnf_lcm_op_occs_id, '_links': { 'vnfInstance': { - 'href': - '/vnflcm/v1/vnf_instances/%s' - % vnf_instance_id}, + 'href': self._get_vnf_instance_href( + vnf_instance_id)}, 'vnfLcmOpOcc': { - 'href': - '/vnflcmv1/vnf_lcm_op_occs/%s' - % vnf_lcm_op_occs_id}}} + 'href': self._get_vnf_lcm_op_occs_href( + vnf_lcm_op_occs_id)}}} if(operation_state == fields.LcmOccsOperationState.COMPLETED or operation_state == fields.LcmOccsOperationState.FAILED_TEMP or