From f518496ce1ca757928ca585fb2c011c6a5fd3f10 Mon Sep 17 00:00:00 2001 From: siddharth0905 Date: Fri, 20 Jul 2018 20:14:53 +0530 Subject: Deployment Flavour created incorrect Component Id Deployment flavour created with incorrect component id Change-Id: I463a74b8139ec9689457e01602faf37384d01993 Issue-ID: SDC-1548 Signed-off-by: siddharth0905 --- .../errors/DeploymentFlavorErrorBuilder.java | 20 +++- .../impl/DeploymentFlavorManagerFactoryImpl.java | 4 +- .../impl/DeploymentFlavorManagerImpl.java | 121 ++++++++++++--------- 3 files changed, 88 insertions(+), 57 deletions(-) (limited to 'openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org') diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/errors/DeploymentFlavorErrorBuilder.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/errors/DeploymentFlavorErrorBuilder.java index 8fa8bd2088..ab0881268e 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/errors/DeploymentFlavorErrorBuilder.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/errors/DeploymentFlavorErrorBuilder.java @@ -35,6 +35,8 @@ public class DeploymentFlavorErrorBuilder { + ": name : must match %s"; private static final String INVALID_COMPUTE_FLAVOR_ID_MSG = "Invalid request, Compute Flavor provided does not exist for this VFC."; + private static final String INVALID_COMPONENT_ID_MSG = + "Invalid request, Component provided does not exist for this VSP."; private static final String INVALID_COMPONENT_COMPUTE_ASSOCIATION_ERROR_MSG="VSP cannot be " + "submitted with an invalid Deployment Flavor. All Deployment Flavor should have atleast a VFC included with it's required Compute needs. " + "Please fix the Deployment Flavor %s and re-submit the VSP."; @@ -55,8 +57,7 @@ public class DeploymentFlavorErrorBuilder { return builder.build(); } - public static ErrorCode getFeatureGroupNotexistErrorBuilder( String featureGroupId, String - vspId, Version activeVersion){ + public static ErrorCode getFeatureGroupNotexistErrorBuilder(){ ErrorCode.ErrorCodeBuilder builder = new ErrorCode.ErrorCodeBuilder(); builder.withId(VendorSoftwareProductErrorCodes.FEATURE_GROUP_NOT_EXIST_FOR_VSP); builder.withCategory(ErrorCategory.APPLICATION); @@ -89,13 +90,20 @@ public class DeploymentFlavorErrorBuilder { builder.withMessage(String.format(DUPLICATE_DEPLOYMENT_FLAVOR_MODEL_NOT_ALLOWED_MSG,name,vspId)); return builder.build(); } - public static ErrorCode getInvalidComputeIdErrorBuilder(String computeFlavorId, String - vfcId){ + + public static ErrorCode getInvalidComputeIdErrorBuilder() { ErrorCode.ErrorCodeBuilder builder = new ErrorCode.ErrorCodeBuilder(); builder.withId(VendorSoftwareProductErrorCodes.INVALID_COMPUTE_FLAVOR_ID); builder.withCategory(ErrorCategory.APPLICATION); - builder.withMessage(String.format(INVALID_COMPUTE_FLAVOR_ID_MSG,computeFlavorId, - vfcId)); + builder.withMessage(INVALID_COMPUTE_FLAVOR_ID_MSG); + return builder.build(); + } + + public static ErrorCode getInvalidComponentIdErrorBuilder() { + ErrorCode.ErrorCodeBuilder builder = new ErrorCode.ErrorCodeBuilder(); + builder.withId(VendorSoftwareProductErrorCodes.INVALID_COMPONENT_ID); + builder.withCategory(ErrorCategory.APPLICATION); + builder.withMessage(INVALID_COMPONENT_ID_MSG); return builder.build(); } diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerFactoryImpl.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerFactoryImpl.java index 0eef5246c3..4c0d8de014 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerFactoryImpl.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerFactoryImpl.java @@ -19,6 +19,7 @@ package org.openecomp.sdc.vendorsoftwareproduct.impl; import org.openecomp.sdc.vendorsoftwareproduct.DeploymentFlavorManager; import org.openecomp.sdc.vendorsoftwareproduct.DeploymentFlavorManagerFactory; +import org.openecomp.sdc.vendorsoftwareproduct.dao.ComponentDaoFactory; import org.openecomp.sdc.vendorsoftwareproduct.dao.ComputeDaoFactory; import org.openecomp.sdc.vendorsoftwareproduct.dao.DeploymentFlavorDaoFactory; import org.openecomp.sdc.vendorsoftwareproduct.dao.VendorSoftwareProductInfoDaoFactory; @@ -30,7 +31,8 @@ public class DeploymentFlavorManagerFactoryImpl extends DeploymentFlavorManagerF VendorSoftwareProductInfoDaoFactory.getInstance().createInterface(), DeploymentFlavorDaoFactory.getInstance().createInterface(), CompositionEntityDataManagerFactory.getInstance().createInterface(), - ComputeDaoFactory.getInstance().createInterface() + ComputeDaoFactory.getInstance().createInterface(), + ComponentDaoFactory.getInstance().createInterface() ); @Override diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerImpl.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerImpl.java index 56678abfd5..cb5533a0a3 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerImpl.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/DeploymentFlavorManagerImpl.java @@ -16,14 +16,23 @@ package org.openecomp.sdc.vendorsoftwareproduct.impl; +import java.util.ArrayList; +import java.util.Collection; +import java.util.Iterator; +import java.util.List; +import java.util.Map; + import org.apache.commons.collections4.CollectionUtils; +import org.apache.commons.lang.StringUtils; import org.openecomp.sdc.common.errors.CoreException; import org.openecomp.sdc.common.errors.ErrorCode; import org.openecomp.sdc.vendorsoftwareproduct.CompositionEntityDataManager; import org.openecomp.sdc.vendorsoftwareproduct.DeploymentFlavorManager; +import org.openecomp.sdc.vendorsoftwareproduct.dao.ComponentDao; import org.openecomp.sdc.vendorsoftwareproduct.dao.ComputeDao; import org.openecomp.sdc.vendorsoftwareproduct.dao.DeploymentFlavorDao; import org.openecomp.sdc.vendorsoftwareproduct.dao.VendorSoftwareProductInfoDao; +import org.openecomp.sdc.vendorsoftwareproduct.dao.type.ComponentEntity; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.ComputeEntity; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.DeploymentFlavorEntity; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.VspDetails; @@ -40,26 +49,22 @@ import org.openecomp.sdc.vendorsoftwareproduct.types.schemagenerator.SchemaTempl import org.openecomp.sdc.versioning.VersioningUtil; import org.openecomp.sdc.versioning.dao.types.Version; -import java.util.ArrayList; -import java.util.Collection; -import java.util.Iterator; -import java.util.List; -import java.util.Map; - public class DeploymentFlavorManagerImpl implements DeploymentFlavorManager { private final VendorSoftwareProductInfoDao vspInfoDao; private final DeploymentFlavorDao deploymentFlavorDao; private final CompositionEntityDataManager compositionEntityDataManager; private final ComputeDao computeDao; + private final ComponentDao componentDao; public DeploymentFlavorManagerImpl(VendorSoftwareProductInfoDao vspInfoDao, DeploymentFlavorDao deploymentFlavorDao, CompositionEntityDataManager compositionEntityDataManager, - ComputeDao computeDao) { + ComputeDao computeDao, ComponentDao componentDao) { this.vspInfoDao = vspInfoDao; this.deploymentFlavorDao = deploymentFlavorDao; this.compositionEntityDataManager = compositionEntityDataManager; this.computeDao = computeDao; + this.componentDao = componentDao; } @@ -72,8 +77,7 @@ public class DeploymentFlavorManagerImpl implements DeploymentFlavorManager { public DeploymentFlavorEntity createDeploymentFlavor( DeploymentFlavorEntity deploymentFlavorEntity) { DeploymentFlavorEntity createDeploymentFlavor; - if (!vspInfoDao.isManual(deploymentFlavorEntity.getVspId(), - deploymentFlavorEntity.getVersion())) { + if (!vspInfoDao.isManual(deploymentFlavorEntity.getVspId(), deploymentFlavorEntity.getVersion())) { ErrorCode deploymentFlavorErrorBuilder = DeploymentFlavorErrorBuilder .getAddDeploymentNotSupportedHeatOnboardErrorBuilder(); throw new CoreException(deploymentFlavorErrorBuilder); @@ -100,8 +104,7 @@ public class DeploymentFlavorManagerImpl implements DeploymentFlavorManager { if (featureGroup != null && featureGroup.trim().length() > 0 && (isEmpty(featureGroups) || (!(validFeatureGroup(featureGroups, featureGroup))))) { ErrorCode deploymentFlavorErrorBuilder = DeploymentFlavorErrorBuilder - .getFeatureGroupNotexistErrorBuilder(featureGroup, deploymentFlavorEntity.getVspId(), - version); + .getFeatureGroupNotexistErrorBuilder(); throw new CoreException(deploymentFlavorErrorBuilder); } validateComponentComputeAssociation(deploymentFlavorEntity, version); @@ -145,47 +148,66 @@ public class DeploymentFlavorManagerImpl implements DeploymentFlavorManager { return valid; } - private void validateComponentComputeAssociation(DeploymentFlavorEntity deploymentFlavorEntity, - Version version) { - List componentComputeAssociationList = deploymentFlavorEntity - .getDeploymentFlavorCompositionData().getComponentComputeAssociations(); - List vfcList = new ArrayList<>(); - if (!isEmpty(componentComputeAssociationList)) { - componentComputeAssociationList.forEach(componentComputeAssociation -> - validateComponentComputeAssocoationList(deploymentFlavorEntity, - version, vfcList, componentComputeAssociation)); - Map frequencyMapping = CollectionUtils.getCardinalityMap(vfcList); - - for (Integer vfcCount : frequencyMapping.values()) { - if (vfcCount != 1) { - ErrorCode duplicateVfcAssociationErrorBuilder = DeploymentFlavorErrorBuilder - .getDuplicateVfcAssociationErrorBuilder(); - throw new CoreException(duplicateVfcAssociationErrorBuilder); + private void validateComponentComputeAssociation(DeploymentFlavorEntity deploymentFlavorEntity, + Version version) { + List componentComputeAssociationList = + deploymentFlavorEntity.getDeploymentFlavorCompositionData().getComponentComputeAssociations(); + List vfcList = new ArrayList<>(); + if (!isEmpty(componentComputeAssociationList)) { + componentComputeAssociationList.forEach( + componentComputeAssociation -> validateComponentComputeAssocoationList(deploymentFlavorEntity, + version, + vfcList, componentComputeAssociation)); + Map frequencyMapping = CollectionUtils.getCardinalityMap(vfcList); + + for (Integer vfcCount : frequencyMapping.values()) { + if (vfcCount != 1) { + ErrorCode duplicateVfcAssociationErrorBuilder = + DeploymentFlavorErrorBuilder.getDuplicateVfcAssociationErrorBuilder(); + throw new CoreException(duplicateVfcAssociationErrorBuilder); + } + } } - } } - } - private void validateComponentComputeAssocoationList( - DeploymentFlavorEntity deploymentFlavorEntity, - Version version, - List vfcList, - ComponentComputeAssociation componentComputeAssociation) { - if ((componentComputeAssociation.getComponentId() == null || componentComputeAssociation - .getComponentId().trim().length() == 0) - && (componentComputeAssociation - .getComputeFlavorId() != null && componentComputeAssociation - .getComputeFlavorId().trim().length() > 0)) { - ErrorCode invalidAssociationErrorBuilder = DeploymentFlavorErrorBuilder - .getInvalidAssociationErrorBuilder(); - throw new CoreException(invalidAssociationErrorBuilder); - } else if (componentComputeAssociation.getComponentId() != null - && componentComputeAssociation.getComponentId().trim().length() > 0) { - validateComponentComputeAssociationFlavour(deploymentFlavorEntity, - version, componentComputeAssociation); - vfcList.add(componentComputeAssociation.getComponentId()); + private void validateComponentComputeAssocoationList( + DeploymentFlavorEntity deploymentFlavorEntity, + Version version, + List vfcList, + ComponentComputeAssociation componentComputeAssociation) { + if ((componentComputeAssociation.getComponentId() == null || componentComputeAssociation + .getComponentId().trim().length() == 0) + && (componentComputeAssociation + .getComputeFlavorId() != null && componentComputeAssociation + .getComputeFlavorId().trim().length() > 0)) { + ErrorCode invalidAssociationErrorBuilder = DeploymentFlavorErrorBuilder + .getInvalidAssociationErrorBuilder(); + throw new CoreException(invalidAssociationErrorBuilder); + } else if (componentComputeAssociation.getComponentId() != null + && componentComputeAssociation.getComponentId().trim().length() > 0) { + validateComponentAssociation(deploymentFlavorEntity, + version, componentComputeAssociation); + + validateComponentComputeAssociationFlavour(deploymentFlavorEntity, + version, componentComputeAssociation); + vfcList.add(componentComputeAssociation.getComponentId()); + } + } + + private void validateComponentAssociation(DeploymentFlavorEntity deploymentFlavorEntity, Version version, + ComponentComputeAssociation componentComputeAssociation) { + + if (StringUtils.isNotBlank(componentComputeAssociation.getComponentId())) { + ComponentEntity componentEntity = + componentDao.get(new ComponentEntity(deploymentFlavorEntity.getVspId(), version, + componentComputeAssociation.getComponentId())); + if (componentEntity == null) { + ErrorCode invalidComputeIdErrorBuilder = + DeploymentFlavorErrorBuilder.getInvalidComponentIdErrorBuilder(); + throw new CoreException(invalidComputeIdErrorBuilder); + } + } } - } private void validateComponentComputeAssociationFlavour( DeploymentFlavorEntity deploymentFlavorEntity, @@ -199,8 +221,7 @@ public class DeploymentFlavorManagerImpl implements DeploymentFlavorManager { componentComputeAssociation.getComputeFlavorId())); if (computeFlavor == null) { ErrorCode invalidComputeIdErrorBuilder = DeploymentFlavorErrorBuilder - .getInvalidComputeIdErrorBuilder(componentComputeAssociation.getComputeFlavorId(), - componentComputeAssociation.getComponentId()); + .getInvalidComputeIdErrorBuilder(); throw new CoreException(invalidComputeIdErrorBuilder); } } -- cgit 1.2.3-korg