From d67bf35d358de09054049641892cab72eacbb7ce Mon Sep 17 00:00:00 2001 From: "Determe, Sebastien (sd378r)" Date: Wed, 25 Apr 2018 14:22:52 +0200 Subject: [PATCH] Fix the Sdc Controller Fix the controller bugs, it was not able to decode properly the Yaml received and there was a bug with Csar content Issue-ID: CLAMP-151 Change-Id: If42577b221c31b48a2b955e35bb3c1a957240611 Signed-off-by: Determe, Sebastien (sd378r) --- src/main/java/org/onap/clamp/clds/model/CldsModel.java | 5 +++-- .../org/onap/clamp/clds/sdc/controller/installer/CsarHandler.java | 2 +- .../onap/clamp/clds/sdc/controller/installer/CsarInstallerImpl.java | 4 ++-- 3 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/main/java/org/onap/clamp/clds/model/CldsModel.java b/src/main/java/org/onap/clamp/clds/model/CldsModel.java index b92ff5f7..af4d6c66 100644 --- a/src/main/java/org/onap/clamp/clds/model/CldsModel.java +++ b/src/main/java/org/onap/clamp/clds/model/CldsModel.java @@ -108,10 +108,11 @@ public class CldsModel { /** * Save model to DB. */ - public void save(CldsDao cldsDao, String userid) { - cldsDao.setModel(this, userid); + public CldsModel save(CldsDao cldsDao, String userid) { + CldsModel cldsModel = cldsDao.setModel(this, userid); determineStatus(); determinePermittedActionCd(); + return cldsModel; } /** diff --git a/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarHandler.java b/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarHandler.java index 87c9c15f..88354386 100644 --- a/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarHandler.java +++ b/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarHandler.java @@ -113,7 +113,7 @@ public class CsarHandler { for (IResourceInstance resource : this.sdcNotification.getResources()) { String filteredString = resource.getResourceInstanceName().replaceAll("-", ""); filteredString = filteredString.replaceAll(" ", ""); - if (filteredString.equals(blueprintResourceInstanceName)) { + if (filteredString.equalsIgnoreCase(blueprintResourceInstanceName)) { return resource; } } diff --git a/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarInstallerImpl.java b/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarInstallerImpl.java index 0498ca75..e28e8ab7 100644 --- a/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarInstallerImpl.java +++ b/src/main/java/org/onap/clamp/clds/sdc/controller/installer/CsarInstallerImpl.java @@ -154,7 +154,7 @@ public class CsarInstallerImpl implements CsarInstaller { List policyNameList = new ArrayList<>(); Map templateNodes = ((Map) ((Map) yaml .load(blueprintArtifact.getDcaeBlueprint())).get("node_templates")); - templateNodes.entrySet().stream().filter(e -> e.getKey().contains("policy_")).forEach(ef -> { + templateNodes.entrySet().stream().filter(e -> e.getKey().contains("policy")).forEach(ef -> { String filteredPolicyName = (String) ((Map) ((Map) ef.getValue()) .get("properties")).get("policy_filter"); if (policyName != null) { @@ -230,7 +230,7 @@ public class CsarInstallerImpl implements CsarInstaller { + blueprintArtifact.getResourceAttached().getResourceInvariantUUID() + "\"]},{\"name\":\"actionSet\",\"value\":[\"vnfRecipe\"]},{\"name\":\"location\",\"value\":[\"DC1\"]}," + inputParams + "]}"); - cldsModel.save(cldsDao, null); + cldsModel = cldsModel.save(cldsDao, null); logger.info("Fake Clds Model created for blueprint " + blueprintArtifact.getBlueprintArtifactName() + " with name " + cldsModel.getName()); return cldsModel; -- 2.16.6