diff options
4 files changed, 16 insertions, 8 deletions
diff --git a/vid-app-common/src/main/java/org/onap/vid/job/command/MsoRequestBuilder.kt b/vid-app-common/src/main/java/org/onap/vid/job/command/MsoRequestBuilder.kt index 17948d4b6..b69f76dfe 100644 --- a/vid-app-common/src/main/java/org/onap/vid/job/command/MsoRequestBuilder.kt +++ b/vid-app-common/src/main/java/org/onap/vid/job/command/MsoRequestBuilder.kt @@ -189,11 +189,13 @@ class MsoRequestBuilder } private fun generateServiceName(jobId: UUID?, payload: ServiceInstantiation, optimisticUniqueServiceInstanceName: String): String? { - var serviceInstanceName: String? = null - if (StringUtils.isNotEmpty(optimisticUniqueServiceInstanceName)) { - serviceInstanceName = peekServiceName(jobId, payload, optimisticUniqueServiceInstanceName) + var serviceInstanceName: String + if (StringUtils.isEmpty(optimisticUniqueServiceInstanceName)) { + serviceInstanceName = payload.modelInfo.modelName + } else { + serviceInstanceName = optimisticUniqueServiceInstanceName } - return serviceInstanceName + return peekServiceName(jobId, payload, serviceInstanceName) } private fun peekServiceName(jobId: UUID?, payload: ServiceInstantiation, optimisticUniqueServiceInstanceName: String): String { @@ -202,7 +204,7 @@ class MsoRequestBuilder if (isNameFreeInAai(optimisticUniqueServiceInstanceName, ResourceType.SERVICE_INSTANCE)) { serviceInstanceName = optimisticUniqueServiceInstanceName } else { - serviceInstanceName = asyncInstantiationBL.getUniqueName(payload.instanceName, ResourceType.SERVICE_INSTANCE) + serviceInstanceName = asyncInstantiationBL.getUniqueName(optimisticUniqueServiceInstanceName, ResourceType.SERVICE_INSTANCE) }//otherwise we used the original service instance name (from payload) to get a new unique name from DB and AAI //update serviceInfo with new name if needed diff --git a/vid-app-common/src/test/resources/payload_jsons/bulk_service_no_homing.json b/vid-app-common/src/test/resources/payload_jsons/bulk_service_no_homing.json index 4c8b4cd96..967a464c3 100644 --- a/vid-app-common/src/test/resources/payload_jsons/bulk_service_no_homing.json +++ b/vid-app-common/src/test/resources/payload_jsons/bulk_service_no_homing.json @@ -18,6 +18,7 @@ "globalSubscriberId": "{some subscriber id}" }, "requestInfo": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "productFamilyId": "a9a77d5a-123e-4ca2-9eb9-0b015d2ee0fb", "source": "VID", "suppressRollback": false, @@ -28,6 +29,7 @@ "aLaCarte": false, "userParams": [{ "service": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "modelInfo": { "modelType": "service", "modelName": "MOW AVPN vMX BV vPE 1 Service", @@ -97,4 +99,4 @@ ] } } -}
\ No newline at end of file +} diff --git a/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_ecomp_naming.json b/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_ecomp_naming.json index 8676d53d2..1d5188eca 100644 --- a/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_ecomp_naming.json +++ b/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_ecomp_naming.json @@ -18,6 +18,7 @@ "globalSubscriberId": "{some subscriber id}" }, "requestInfo": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "productFamilyId": "a9a77d5a-123e-4ca2-9eb9-0b015d2ee0fb", "source": "VID", "suppressRollback": false, @@ -28,6 +29,7 @@ "aLaCarte": false, "userParams": [{ "service": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "modelInfo": { "modelType": "service", "modelName": "MOW AVPN vMX BV vPE 1 Service", @@ -95,4 +97,4 @@ }] } } -}
\ No newline at end of file +} diff --git a/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_no_vfmodule_ecomp_naming.json b/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_no_vfmodule_ecomp_naming.json index 22ac8951c..8345efa18 100644 --- a/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_no_vfmodule_ecomp_naming.json +++ b/vid-app-common/src/test/resources/payload_jsons/bulk_service_request_no_vfmodule_ecomp_naming.json @@ -18,6 +18,7 @@ "globalSubscriberId": "{some subscriber id}" }, "requestInfo": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "productFamilyId": "a9a77d5a-123e-4ca2-9eb9-0b015d2ee0fb", "source": "VID", "suppressRollback": false, @@ -28,6 +29,7 @@ "aLaCarte": false, "userParams": [{ "service": { + "instanceName": "MOW AVPN vMX BV vPE 1 Service", "modelInfo": { "modelType": "service", "modelName": "MOW AVPN vMX BV vPE 1 Service", @@ -66,4 +68,4 @@ }] } } -}
\ No newline at end of file +} |