aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Dudycz <jakub.dudycz@nokia.com>2018-02-20 16:29:13 +0100
committerPatrick Brady <pb071s@att.com>2018-02-22 18:12:02 +0000
commitff14a98360534622511e71a210eac52bfb8254c6 (patch)
treeff5f6354cf47e0c4cdc0d5b9de47a71ca38509c1
parent8d0eb7ff441fbf6f6927b2d2a33c0a442c2fafec (diff)
AAIPluginImpl sonar fixes
Change-Id: Ifa07addb19b34a1f1a2f546c5eb21c2ac0bb3e1a Issue-ID: APPC-651 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
-rw-r--r--appc-dg/appc-dg-shared/appc-dg-aai/src/main/java/org/onap/appc/dg/aai/impl/AAIPluginImpl.java268
1 files changed, 154 insertions, 114 deletions
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<String, String> 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<String, String> data = new HashMap<>();
for (Map.Entry<String, String> 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<String, String> 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<String, String> params, SvcLogicContext ctx) throws APPCException {
Set<Vnfc> 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<String, String> params, SvcLogicContext ctx) throws APPCException {
- String resourceType = params.get("resourceType"), ctx_prefix = params.get("prefix"), resourceKey =
- params.get("resourceKey");
+ @Override
+ public void getResource(Map<String, String> 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<String, String> 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<String, String> 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<String, String> 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<String, String> params, SvcLogicContext ctx) throws APPCException {
- String resourceType = params.get("resourceType"), resourceKey = params.get("resourceKey");
+ @Override
+ public void deleteResource(Map<String, String> 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======");