openstack
/
tacker
Code Issues Proposed changes

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 <navum.gupta@india.nec.com>

Closes-Bug: #1923478
Change-Id: Idc146eed1b2b6a8ee3d4dd6c230324a8aef006f6
This commit is contained in:
Sheel Rana 2021-05-17 22:02:35 +09:00
parent ccbfe08ecb
commit cf5150571b
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tacker/conductor/conductor_server.py
View File

@ -1571,13 +1571,11 @@ class Conductor(manager.Manager):
'vnfLcmOpOccId': vnf_lcm_op_occs_id, 'vnfLcmOpOccId': vnf_lcm_op_occs_id,
'_links': { '_links': {
'vnfInstance': { 'vnfInstance': {
'href': 'href': self._get_vnf_instance_href(
'/vnflcm/v1/vnf_instances/%s' vnf_instance_id)},
% vnf_instance_id},
'vnfLcmOpOcc': { 'vnfLcmOpOcc': {
'href': 'href': self._get_vnf_lcm_op_occs_href(
'/vnflcmv1/vnf_lcm_op_occs/%s' vnf_lcm_op_occs_id)}}}
% vnf_lcm_op_occs_id}}}
if(operation_state == fields.LcmOccsOperationState.COMPLETED or if(operation_state == fields.LcmOccsOperationState.COMPLETED or
operation_state == fields.LcmOccsOperationState.FAILED_TEMP or operation_state == fields.LcmOccsOperationState.FAILED_TEMP or