From ff14a98360534622511e71a210eac52bfb8254c6 Mon Sep 17 00:00:00 2001 From: Jakub Dudycz Date: Tue, 20 Feb 2018 16:29:13 +0100 Subject: AAIPluginImpl sonar fixes Change-Id: Ifa07addb19b34a1f1a2f546c5eb21c2ac0bb3e1a Issue-ID: APPC-651 Signed-off-by: Jakub Dudycz --- .../org/onap/appc/dg/aai/impl/AAIPluginImpl.java | 268 ++++++++++++--------- 1 file changed, 154 insertions(+), 114 deletions(-) (limited to 'appc-dg/appc-dg-shared') diff --git a/appc-dg/appc-dg-shared/appc-dg-aai/src/main/java/org/onap/appc/dg/aai/impl/AAIPluginImpl.java b/appc-dg/appc-dg-shared/appc-dg-aai/src/main/java/org/onap/appc/dg/aai/impl/AAIPluginImpl.java index 7848c412e..25ea08e0a 100644 --- a/appc-dg/appc-dg-shared/appc-dg-aai/src/main/java/org/onap/appc/dg/aai/impl/AAIPluginImpl.java +++ b/appc-dg/appc-dg-shared/appc-dg-aai/src/main/java/org/onap/appc/dg/aai/impl/AAIPluginImpl.java @@ -23,42 +23,65 @@ */ package org.onap.appc.dg.aai.impl; -import org.onap.appc.domainmodel.Vnf; -import org.onap.appc.domainmodel.Vnfc; -import org.onap.appc.domainmodel.Vserver; -import org.onap.appc.exceptions.APPCException; -import org.onap.appc.i18n.Msg; -import com.att.eelf.i18n.EELFResourceManager; +import com.att.eelf.configuration.EELFLogger; +import com.att.eelf.configuration.EELFManager; +import com.att.eelf.i18n.EELFResourceManager; +import java.util.HashMap; +import java.util.HashSet; +import java.util.Map; +import java.util.Set; +import java.util.stream.Collectors; import org.onap.appc.dg.aai.AAIPlugin; import org.onap.appc.dg.aai.exception.AAIQueryException; import org.onap.appc.dg.aai.objects.AAIQueryResult; import org.onap.appc.dg.aai.objects.Relationship; -import com.att.eelf.configuration.EELFLogger; -import com.att.eelf.configuration.EELFManager; +import org.onap.appc.domainmodel.Vnf; +import org.onap.appc.domainmodel.Vnfc; +import org.onap.appc.domainmodel.Vserver; +import org.onap.appc.exceptions.APPCException; +import org.onap.appc.i18n.Msg; +import org.onap.ccsdk.sli.adaptors.aai.AAIClient; +import org.onap.ccsdk.sli.adaptors.aai.AAIService; import org.onap.ccsdk.sli.core.sli.SvcLogicContext; import org.onap.ccsdk.sli.core.sli.SvcLogicException; import org.onap.ccsdk.sli.core.sli.SvcLogicResource; -import org.onap.ccsdk.sli.adaptors.aai.AAIClient; -import org.onap.ccsdk.sli.adaptors.aai.AAIService; import org.osgi.framework.BundleContext; import org.osgi.framework.FrameworkUtil; import org.osgi.framework.ServiceReference; -import java.util.HashMap; -import java.util.HashSet; -import java.util.Map; -import java.util.Set; -import java.util.stream.Collectors; - public class AAIPluginImpl implements AAIPlugin { - protected AAIClient aaiClient; + private static final String PARAM_GENERIC_VNF = "generic-vnf"; + private static final String PARAM_RESOURCE_KEY = "resourceKey"; + + private static final String STR_AAI_RESPONSE = "AAIResponse: "; + private static final String STR_VNF_ID = "VNF ID "; + private static final String STR_VNF_VNFC = "vnf.vnfc["; + + private static final String PROPERTY_IN_MAINT = "in-maint"; + private static final String PROPERTY_PROV_STATUS = "prov-status"; + private static final String PROPERTY_LOOP_DISABLED = "is-closed-loop-disabled"; + private static final String PROPERTY_RESOURCE_VERSION = "resource-version"; + private static final String PROPERTY_VNFC_FUNC_CODE = "vnfc-function-code"; + private static final String PROPERTY_ORCHESTRATION_STATUS = "orchestration-status"; + private static final String PROPERTY_VNFC_TYPE = "vnfc-type"; + private static final String PROPERTY_VNFC_NAME = "vnfc-name"; + private static final String PROPERTY_VSERVER_ID = "vserver-id"; + private static final String PROPERTY_VSERVER_SLINK = "vserver-selflink"; + private static final String PROPERTY_VSERVER_NAME = "vserver-name"; + private static final String PROPERTY_VSERVER_NAME_2 = "vserver-name2"; + private static final String PROPERTY_HEAT_STACK_ID = "heat-stack-id"; + private static final String PROPERTY_VNF_TYPE = "vnf-type"; + private static final String PROPERTY_VNF_NEM = "vnf-name"; + private static final String PARAM_RESOURCE_TYPE = "resourceType"; + + private AAIClient aaiClient; private final EELFLogger logger = EELFManager.getInstance().getLogger(AAIPluginImpl.class); - public void initialize(){ + public void initialize() { BundleContext bctx = FrameworkUtil.getBundle(this.getClass()).getBundleContext(); ServiceReference sref = bctx.getServiceReference(AAIService.class); aaiClient = (AAIClient) bctx.getService(sref); @@ -66,38 +89,39 @@ public class AAIPluginImpl implements AAIPlugin { @Override public void postGenericVnfData(Map params, SvcLogicContext ctx) throws APPCException { - String vnf_id = ctx.getAttribute(Constants.VNF_ID_PARAM_NAME); + String vnfId = ctx.getAttribute(Constants.VNF_ID_PARAM_NAME); String prefix = ctx.getAttribute(Constants.AAI_PREFIX_PARAM_NAME); - String key = "vnf-id = '" + vnf_id + "'"; + String key = "vnf-id = '" + vnfId + "'"; Map data = new HashMap<>(); for (Map.Entry entry : params.entrySet()) { String paramKey = entry.getKey(); int pos = paramKey.indexOf(Constants.AAI_INPUT_DATA); if (pos == 0) { - data.put(paramKey.substring(Constants.AAI_INPUT_DATA.length()+1), entry.getValue()); + data.put(paramKey.substring(Constants.AAI_INPUT_DATA.length() + 1), entry.getValue()); } } try { - SvcLogicResource.QueryStatus response = aaiClient.update("generic-vnf", key, data, prefix, ctx); + SvcLogicResource.QueryStatus response = aaiClient.update(PARAM_GENERIC_VNF, key, data, prefix, ctx); if (SvcLogicResource.QueryStatus.NOT_FOUND.equals(response)) { - String msg = EELFResourceManager.format(Msg.VNF_NOT_FOUND, vnf_id); + String msg = EELFResourceManager.format(Msg.VNF_NOT_FOUND, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); throw new APPCException(msg); } - logger.info("AAIResponse: " + response.toString()); + logger.info(STR_AAI_RESPONSE + response.toString()); if (SvcLogicResource.QueryStatus.FAILURE.equals(response)) { - String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnf_id); + String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); throw new APPCException(msg); } - String msg = EELFResourceManager.format(Msg.SUCCESS_EVENT_MESSAGE, "PostGenericVnfData", "VNF ID " + vnf_id); + String msg = EELFResourceManager + .format(Msg.SUCCESS_EVENT_MESSAGE, "PostGenericVnfData", STR_VNF_ID + vnfId); ctx.setAttribute(org.onap.appc.Constants.ATTRIBUTE_SUCCESS_MESSAGE, msg); } catch (SvcLogicException e) { - String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnf_id); + String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); logger.error(msg); throw new APPCException(e); @@ -106,35 +130,40 @@ public class AAIPluginImpl implements AAIPlugin { @Override public void getGenericVnfData(Map params, SvcLogicContext ctx) throws APPCException { - String vnf_id = ctx.getAttribute(Constants.VNF_ID_PARAM_NAME); + String vnfId = ctx.getAttribute(Constants.VNF_ID_PARAM_NAME); String prefix = ctx.getAttribute(Constants.AAI_PREFIX_PARAM_NAME); - String key = "vnf-id = '" + vnf_id + "'"; + String key = "vnf-id = '" + vnfId + "'"; try { - SvcLogicResource.QueryStatus response = aaiClient.query("generic-vnf", false, null, key, prefix, null, ctx); + SvcLogicResource.QueryStatus response = aaiClient + .query(PARAM_GENERIC_VNF, false, null, key, prefix, null, ctx); if (SvcLogicResource.QueryStatus.NOT_FOUND.equals(response)) { - String msg = EELFResourceManager.format(Msg.VNF_NOT_FOUND, vnf_id); -// String errorMessage = String.format("VNF not found for vnf_id = %s", vnf_id); + String msg = EELFResourceManager.format(Msg.VNF_NOT_FOUND, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); throw new APPCException(msg); } else if (SvcLogicResource.QueryStatus.FAILURE.equals(response)) { - String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnf_id); + String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); throw new APPCException(msg); } String aaiEntitlementPoolUuid = ctx.getAttribute(Constants.AAI_ENTITLMENT_POOL_UUID_NAME); - if (null == aaiEntitlementPoolUuid) aaiEntitlementPoolUuid = ""; + if (null == aaiEntitlementPoolUuid) { + aaiEntitlementPoolUuid = ""; + } String aaiLicenseKeyGroupUuid = ctx.getAttribute(Constants.AAI_LICENSE_KEY_UUID_NAME); - if (null == aaiLicenseKeyGroupUuid) aaiLicenseKeyGroupUuid = ""; + if (null == aaiLicenseKeyGroupUuid) { + aaiLicenseKeyGroupUuid = ""; + } - ctx.setAttribute(Constants.IS_RELEASE_ENTITLEMENT_REQUIRE, Boolean.toString(!aaiEntitlementPoolUuid.isEmpty())); + ctx.setAttribute(Constants.IS_RELEASE_ENTITLEMENT_REQUIRE, + Boolean.toString(!aaiEntitlementPoolUuid.isEmpty())); ctx.setAttribute(Constants.IS_RELEASE_LICENSE_REQUIRE, Boolean.toString(!aaiLicenseKeyGroupUuid.isEmpty())); - String msg = EELFResourceManager.format(Msg.SUCCESS_EVENT_MESSAGE, "GetGenericVnfData","VNF ID " + vnf_id); + String msg = EELFResourceManager.format(Msg.SUCCESS_EVENT_MESSAGE, "GetGenericVnfData", STR_VNF_ID + vnfId); ctx.setAttribute(org.onap.appc.Constants.ATTRIBUTE_SUCCESS_MESSAGE, msg); - logger.info("AAIResponse: " + response.toString()); + logger.info(STR_AAI_RESPONSE + response.toString()); } catch (SvcLogicException e) { - String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnf_id); + String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); logger.error(msg); throw new APPCException(e); @@ -145,20 +174,21 @@ public class AAIPluginImpl implements AAIPlugin { public void getVnfHierarchy(Map params, SvcLogicContext ctx) throws APPCException { Set vnfcSet = new HashSet<>(); - String vnfType,vnfVersion; - String vnfId = params.get("resourceKey"); + String vnfType; + String vnfVersion; + String vnfId = params.get(PARAM_RESOURCE_KEY); AAIQueryResult vnfQueryResult; - int vmCount =0; + int vmCount = 0; try { vnfQueryResult = readVnf(vnfId); - vnfType = vnfQueryResult.getAdditionProperties().get("vnf-type"); + vnfType = vnfQueryResult.getAdditionProperties().get(PROPERTY_VNF_TYPE); vnfVersion = vnfQueryResult.getAdditionProperties().get(Constants.AAI_VNF_MODEL_VERSION_ID); Vnf vnf = createVnf(vnfType, vnfVersion, vnfId); - for(Relationship vnfRelationship:vnfQueryResult.getRelationshipList()){ - if("vserver".equalsIgnoreCase(vnfRelationship.getRelatedTo())){ + for (Relationship vnfRelationship : vnfQueryResult.getRelationshipList()) { + if ("vserver".equalsIgnoreCase(vnfRelationship.getRelatedTo())) { vmCount++; String tenantId = vnfRelationship.getRelationShipDataMap().get("tenant.tenant-id"); String vmId = vnfRelationship.getRelationShipDataMap().get("vserver.vserver-id"); @@ -167,25 +197,25 @@ public class AAIPluginImpl implements AAIPlugin { String cloudOwner = vnfRelationship.getRelationShipDataMap().get("cloud-region.cloud-owner"); String cloudRegionId = vnfRelationship.getRelationShipDataMap().get("cloud-region.cloud-region-id"); - AAIQueryResult vmQueryResult = readVM(vmId,tenantId,cloudOwner,cloudRegionId); - String vmURL = vmQueryResult.getAdditionProperties().get("vserver-selflink"); + AAIQueryResult vmQueryResult = readVM(vmId, tenantId, cloudOwner, cloudRegionId); + String vmURL = vmQueryResult.getAdditionProperties().get(PROPERTY_VSERVER_SLINK); Vserver vm = createVserver(tenantId, vmId, vmRelatedLink, vmName, vmURL); vnf.addVserver(vm); - for(Relationship vmRelation:vmQueryResult.getRelationshipList()){ + for (Relationship vmRelation : vmQueryResult.getRelationshipList()) { - if("vnfc".equalsIgnoreCase(vmRelation.getRelatedTo())){ + if ("vnfc".equalsIgnoreCase(vmRelation.getRelatedTo())) { String vnfcName = vmRelation.getRelationShipDataMap().get("vnfc.vnfc-name"); AAIQueryResult vnfcQueryResult = readVnfc(vnfcName); - String vnfcType = vnfcQueryResult.getAdditionProperties().get("vnfc-type"); + String vnfcType = vnfcQueryResult.getAdditionProperties().get(PROPERTY_VNFC_TYPE); Vnfc newVnfc = createVnfc(vnfcName, vnfcType); - if(vnfcSet.contains(newVnfc)){ - Vnfc vnfcFromSet = vnfcSet.stream().filter(vnfc -> vnfc.equals(newVnfc)).collect(Collectors.toList()).get(0); + if (vnfcSet.contains(newVnfc)) { + Vnfc vnfcFromSet = vnfcSet.stream().filter(vnfc -> vnfc.equals(newVnfc)) + .collect(Collectors.toList()).get(0); vnfcFromSet.addVserver(vm); vm.setVnfc(vnfcFromSet); - } - else{ + } else { vm.setVnfc(newVnfc); newVnfc.addVserver(vm); vnfcSet.add(newVnfc); @@ -194,19 +224,19 @@ public class AAIPluginImpl implements AAIPlugin { } } } - ctx.setAttribute("VNF.VMCount",String.valueOf(vmCount)); - populateContext(vnf,ctx); + ctx.setAttribute("VNF.VMCount", String.valueOf(vmCount)); + populateContext(vnf, ctx); } catch (AAIQueryException e) { ctx.setAttribute("getVnfHierarchy_result", "FAILURE"); String msg = EELFResourceManager.format(Msg.AAI_QUERY_FAILED, vnfId); ctx.setAttribute(Constants.ATTRIBUTE_ERROR_MESSAGE, msg); logger.error("Failed in getVnfHierarchy, Error retrieving VNF details. Error message: " + ctx - .getAttribute("getResource_result")); + .getAttribute("getResource_result"), e); logger.warn("Incorrect or Incomplete VNF Hierarchy"); throw new APPCException("Error Retrieving VNF hierarchy"); } ctx.setAttribute("getVnfHierarchy_result", "SUCCESS"); - String msg = EELFResourceManager.format(Msg.SUCCESS_EVENT_MESSAGE, "GetVNFHierarchy","VNF ID " + vnfId); + String msg = EELFResourceManager.format(Msg.SUCCESS_EVENT_MESSAGE, "GetVNFHierarchy", STR_VNF_ID + vnfId); ctx.setAttribute(org.onap.appc.Constants.ATTRIBUTE_SUCCESS_MESSAGE, msg); } @@ -236,18 +266,18 @@ public class AAIPluginImpl implements AAIPlugin { return vserver; } - private void populateContext(Vnf vnf ,SvcLogicContext ctx) { - ctx.setAttribute("vnf.type",vnf.getVnfType()); - ctx.setAttribute("vnf.version",vnf.getVnfVersion()); - ctx.setAttribute("vnf.vnfcCount",String.valueOf(vnf.getVnfcs().size())); - int vnfcCount =0; - for(Vnfc vnfc:vnf.getVnfcs()){ - ctx.setAttribute("vnf.vnfc["+vnfcCount+"].name",vnfc.getVnfcName()); - ctx.setAttribute("vnf.vnfc["+vnfcCount+"].type",vnfc.getVnfcType()); - ctx.setAttribute("vnf.vnfc["+vnfcCount+"].vm_count",String.valueOf(vnfc.getVserverList().size())); - int vmCount =0; - for(Vserver vm:vnfc.getVserverList()){ - ctx.setAttribute("vnf.vnfc["+vnfcCount+"].vm["+ vmCount++ +"].url",vm.getUrl()); + private void populateContext(Vnf vnf, SvcLogicContext ctx) { + ctx.setAttribute("vnf.type", vnf.getVnfType()); + ctx.setAttribute("vnf.version", vnf.getVnfVersion()); + ctx.setAttribute("vnf.vnfcCount", String.valueOf(vnf.getVnfcs().size())); + int vnfcCount = 0; + for (Vnfc vnfc : vnf.getVnfcs()) { + ctx.setAttribute(STR_VNF_VNFC + vnfcCount + "].name", vnfc.getVnfcName()); + ctx.setAttribute(STR_VNF_VNFC + vnfcCount + "].type", vnfc.getVnfcType()); + ctx.setAttribute(STR_VNF_VNFC + vnfcCount + "].vm_count", String.valueOf(vnfc.getVserverList().size())); + int vmCount = 0; + for (Vserver vm : vnfc.getVserverList()) { + ctx.setAttribute(STR_VNF_VNFC + vnfcCount + "].vm[" + vmCount++ + "].url", vm.getUrl()); } vnfcCount++; } @@ -257,40 +287,42 @@ public class AAIPluginImpl implements AAIPlugin { String query = "vnfc.vnfc-name = '" + vnfcName + "'"; String prefix = "VNFC"; String resourceType = "vnfc"; - SvcLogicContext vnfContext = readResource(query,prefix,resourceType); - String[] additionalProperties = new String[]{"vnfc-type","vnfc-name", - "vnfc-function-code","in-maint","prov-status", - "is-closed-loop-disabled","orchestration-status","resource-version"}; - return readRelationDataAndProperties(prefix, vnfContext,additionalProperties); + SvcLogicContext vnfContext = readResource(query, prefix, resourceType); + String[] additionalProperties = new String[]{PROPERTY_VNFC_TYPE, PROPERTY_VNFC_NAME, + PROPERTY_VNFC_FUNC_CODE, PROPERTY_IN_MAINT, PROPERTY_PROV_STATUS, + PROPERTY_LOOP_DISABLED, PROPERTY_ORCHESTRATION_STATUS, PROPERTY_RESOURCE_VERSION}; + return readRelationDataAndProperties(prefix, vnfContext, additionalProperties); } - private AAIQueryResult readVM(String vmId,String tenantId,String cloudOwner,String cloudRegionId) throws AAIQueryException { - String query = "vserver.vserver-id = '" + vmId + "' AND tenant.tenant_id = '" + tenantId + "' AND cloud-region.cloud-owner = '" - + cloudOwner + "' AND cloud-region.cloud-region-id = '" + cloudRegionId + "'"; + private AAIQueryResult readVM(String vmId, String tenantId, String cloudOwner, String cloudRegionId) + throws AAIQueryException { + String query = "vserver.vserver-id = '" + vmId + "' AND tenant.tenant_id = '" + tenantId + + "' AND cloud-region.cloud-owner = '" + + cloudOwner + "' AND cloud-region.cloud-region-id = '" + cloudRegionId + "'"; String prefix = "VM"; String resourceType = "vserver"; - SvcLogicContext vnfContext = readResource(query,prefix,resourceType); - String[] additionalProperties = new String[]{"vserver-id","vserver-selflink", - "vserver-name","in-maint","prov-status","is-closed-loop-disabled", - "vserver-name2","resource-version",}; + SvcLogicContext vnfContext = readResource(query, prefix, resourceType); + String[] additionalProperties = new String[]{PROPERTY_VSERVER_ID, PROPERTY_VSERVER_SLINK, + PROPERTY_VSERVER_NAME, PROPERTY_IN_MAINT, PROPERTY_PROV_STATUS, PROPERTY_LOOP_DISABLED, + PROPERTY_VSERVER_NAME_2, PROPERTY_RESOURCE_VERSION,}; - return readRelationDataAndProperties(prefix, vnfContext,additionalProperties); + return readRelationDataAndProperties(prefix, vnfContext, additionalProperties); } private AAIQueryResult readVnf(String vnfId) throws AAIQueryException { String query = "generic-vnf.vnf-id = '" + vnfId + "'"; String prefix = "VNF"; - String resourceType = "generic-vnf"; - SvcLogicContext vnfContext = readResource(query,prefix,resourceType); + SvcLogicContext vnfContext = readResource(query, prefix, PARAM_GENERIC_VNF); - String[] additionalProperties = new String[]{"vnf-type","vnf-name", - "in-maint","prov-status","heat-stack-id", - "is-closed-loop-disabled","orchestration-status","resource-version",Constants.AAI_VNF_MODEL_VERSION_ID}; + String[] additionalProperties = new String[]{PROPERTY_VNF_TYPE, PROPERTY_VNF_NEM, + PROPERTY_IN_MAINT, PROPERTY_PROV_STATUS, PROPERTY_HEAT_STACK_ID, + PROPERTY_LOOP_DISABLED, PROPERTY_ORCHESTRATION_STATUS, PROPERTY_RESOURCE_VERSION, Constants.AAI_VNF_MODEL_VERSION_ID}; - return readRelationDataAndProperties(prefix, vnfContext,additionalProperties); + return readRelationDataAndProperties(prefix, vnfContext, additionalProperties); } - private AAIQueryResult readRelationDataAndProperties(String prefix, SvcLogicContext context,String[] additionalProperties) { + private AAIQueryResult readRelationDataAndProperties(String prefix, SvcLogicContext context, + String[] additionalProperties) { AAIQueryResult result = new AAIQueryResult(); if (context != null && context.getAttribute(prefix + ".relationship-list.relationship_length") != null) { @@ -327,7 +359,7 @@ public class AAIPluginImpl implements AAIPlugin { } } else { logger.error("Relationship-list not present in the SvcLogicContext attributes set." - + (context == null ? "" : "Attribute KeySet = "+ context.getAttributeKeySet())); + + (context == null ? "" : "Attribute KeySet = " + context.getAttributeKeySet())); } if (context != null) { @@ -341,64 +373,72 @@ public class AAIPluginImpl implements AAIPlugin { private SvcLogicContext readResource(String query, String prefix, String resourceType) throws AAIQueryException { SvcLogicContext resourceContext = new SvcLogicContext(); try { - SvcLogicResource.QueryStatus response = aaiClient.query(resourceType,false,null,query,prefix,null,resourceContext); - logger.info("AAIResponse: " + response.toString()); - if(!SvcLogicResource.QueryStatus.SUCCESS.equals(response)){ + SvcLogicResource.QueryStatus response = aaiClient + .query(resourceType, false, null, query, prefix, null, resourceContext); + logger.info(STR_AAI_RESPONSE + response.toString()); + if (!SvcLogicResource.QueryStatus.SUCCESS.equals(response)) { throw new AAIQueryException("Error Retrieving VNF hierarchy from A&AI"); } } catch (SvcLogicException e) { - logger.error(EELFResourceManager.format(Msg.AAI_GET_DATA_FAILED, query, "", e.getMessage())); + logger.error(EELFResourceManager.format(Msg.AAI_GET_DATA_FAILED, query), e); throw new AAIQueryException("Error Retrieving VNF hierarchy from A&AI"); } return resourceContext; } - @Override public void getResource(Map params, SvcLogicContext ctx) throws APPCException { - String resourceType = params.get("resourceType"), ctx_prefix = params.get("prefix"), resourceKey = - params.get("resourceKey"); + @Override + public void getResource(Map params, SvcLogicContext ctx) throws APPCException { + String resourceType = params.get(PARAM_RESOURCE_TYPE); + String ctxPrefix = params.get("prefix"); + String resourceKey = params.get(PARAM_RESOURCE_KEY); if (logger.isDebugEnabled()) { logger.debug("inside getResorce"); logger.debug("Retrieving " + resourceType + " details from A&AI for Key : " + resourceKey); } try { SvcLogicResource.QueryStatus response = - aaiClient.query(resourceType, false, null, resourceKey, ctx_prefix, null, ctx); - logger.info("AAIResponse: " + response.toString()); + aaiClient.query(resourceType, false, null, resourceKey, ctxPrefix, null, ctx); + logger.info(STR_AAI_RESPONSE + response.toString()); ctx.setAttribute("getResource_result", response.toString()); } catch (SvcLogicException e) { - logger.error(EELFResourceManager.format(Msg.AAI_GET_DATA_FAILED, resourceKey, "", e.getMessage())); + logger.error(EELFResourceManager.format(Msg.AAI_GET_DATA_FAILED, resourceKey), e); } if (logger.isDebugEnabled()) { logger.debug("exiting getResource======"); } } - @Override public void postResource(Map params, SvcLogicContext ctx) throws APPCException { - String resourceType = params.get("resourceType"), ctx_prefix = params.get("prefix"), resourceKey = - params.get("resourceKey"), att_name = params.get("attributeName"), att_value = - params.get("attributeValue"); + @Override + public void postResource(Map params, SvcLogicContext ctx) throws APPCException { + String resourceType = params.get(PARAM_RESOURCE_TYPE); + String ctxPrefix = params.get("prefix"); + String resourceKey = params.get(PARAM_RESOURCE_KEY); + String attributeName = params.get("attributeName"); + String attributeValue = params.get("attributeValue"); if (logger.isDebugEnabled()) { logger.debug("inside postResource"); logger.debug("Updating " + resourceType + " details in A&AI for Key : " + resourceKey); - logger.debug("Updating " + att_name + " to : " + att_value); + logger.debug("Updating " + attributeName + " to : " + attributeValue); } Map data = new HashMap<>(); - data.put(att_name, att_value); + data.put(attributeName, attributeValue); try { - SvcLogicResource.QueryStatus response = aaiClient.update(resourceType, resourceKey, data, ctx_prefix, ctx); - logger.info("AAIResponse: " + response.toString()); + SvcLogicResource.QueryStatus response = aaiClient.update(resourceType, resourceKey, data, ctxPrefix, ctx); + logger.info(STR_AAI_RESPONSE + response.toString()); ctx.setAttribute("postResource_result", response.toString()); } catch (SvcLogicException e) { - logger.error(EELFResourceManager.format(Msg.AAI_UPDATE_FAILED, resourceKey, att_value, e.getMessage())); + logger.error(EELFResourceManager.format(Msg.AAI_UPDATE_FAILED, resourceKey, attributeValue), e); } if (logger.isDebugEnabled()) { logger.debug("exiting postResource======"); } } - @Override public void deleteResource(Map params, SvcLogicContext ctx) throws APPCException { - String resourceType = params.get("resourceType"), resourceKey = params.get("resourceKey"); + @Override + public void deleteResource(Map params, SvcLogicContext ctx) throws APPCException { + String resourceType = params.get(PARAM_RESOURCE_TYPE); + String resourceKey = params.get(PARAM_RESOURCE_KEY); if (logger.isDebugEnabled()) { logger.debug("inside deleteResource"); @@ -406,10 +446,10 @@ public class AAIPluginImpl implements AAIPlugin { } try { SvcLogicResource.QueryStatus response = aaiClient.delete(resourceType, resourceKey, ctx); - logger.info("AAIResponse: " + response.toString()); + logger.info(STR_AAI_RESPONSE + response.toString()); ctx.setAttribute("deleteResource_result", response.toString()); } catch (SvcLogicException e) { - logger.error(EELFResourceManager.format(Msg.AAI_DELETE_FAILED, resourceKey, e.getMessage())); + logger.error(EELFResourceManager.format(Msg.AAI_DELETE_FAILED, resourceKey), e); } if (logger.isDebugEnabled()) { logger.debug("exiting deleteResource======"); -- cgit 1.2.3-korg