summaryrefslogtreecommitdiffstats
path: root/runtime-acm/src/main
diff options
context:
space:
mode:
authorbrunomilitzer <bruno.militzer@est.tech>2022-03-07 15:13:02 +0000
committerbrunomilitzer <bruno.militzer@est.tech>2022-03-07 16:00:45 +0000
commite27362ae319ac0aa302381e274c87a82cd4ec9f1 (patch)
treefb3c9455c38e74bfd298b2531e54fe8fddb051f8 /runtime-acm/src/main
parent13d6c4f7ac31fd2bd234d351c22702f594905553 (diff)
Fixed Instance Properties Name
Fixed issue where the system put append _Instance1 so the end user can put their own custom name Issue-ID: POLICY-3978 Change-Id: I04500148aeee49c64f278ce00daa55d14f964b90 Signed-off-by: brunomilitzer <bruno.militzer@est.tech>
Diffstat (limited to 'runtime-acm/src/main')
-rw-r--r--runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java6
-rw-r--r--runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java36
2 files changed, 12 insertions, 30 deletions
diff --git a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
index dfb9d151b..9e07cde51 100644
--- a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
+++ b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/commissioning/CommissioningProvider.java
@@ -63,7 +63,7 @@ import org.springframework.transaction.annotation.Transactional;
@Transactional
public class CommissioningProvider {
public static final String AUTOMATION_COMPOSITION_NODE_TYPE = "org.onap.policy.clamp.acm.AutomationComposition";
- private static final String INSTANCE_TEXT = "_Instance";
+ private static final String HYPHEN = "-";
private final ServiceTemplateProvider serviceTemplateProvider;
private final AutomationCompositionProvider acProvider;
@@ -356,7 +356,7 @@ public class CommissioningProvider {
Map<String, ToscaNodeTemplate> toscaNodeTemplates = new HashMap<>();
serviceTemplate.getToscaTopologyTemplate().getNodeTemplates().forEach((key, nodeTemplate) -> {
- if (!nodeTemplate.getName().contains(INSTANCE_TEXT)) {
+ if (!nodeTemplate.getName().contains(HYPHEN)) {
toscaNodeTemplates.put(key, nodeTemplate);
}
});
@@ -377,7 +377,7 @@ public class CommissioningProvider {
*/
private boolean verifyIfInstancePropertiesExists() {
return acProvider.getAllNodeTemplates().stream()
- .anyMatch(nodeTemplate -> nodeTemplate.getKey().getName().contains(INSTANCE_TEXT));
+ .anyMatch(nodeTemplate -> nodeTemplate.getKey().getName().contains(HYPHEN));
}
}
diff --git a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
index 39d84026b..40d4c5a85 100644
--- a/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
+++ b/runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java
@@ -81,7 +81,7 @@ public class AutomationCompositionInstantiationProvider {
private static final String PARTICIPANT_TYPE_PROPERTY_KEY = "participantType";
private static final String AC_ELEMENT_NAME = "name";
private static final String AC_ELEMENT_VERSION = "version";
- private static final String INSTANCE_TEXT = "_Instance";
+ private static final String HYPHEN = "-";
private static final Gson GSON = new Gson();
@@ -101,7 +101,7 @@ public class AutomationCompositionInstantiationProvider {
public InstancePropertiesResponse createInstanceProperties(ToscaServiceTemplate serviceTemplate)
throws PfModelException {
- String instanceName = generateSequentialInstanceName();
+ String instanceName = serviceTemplate.getName();
AutomationComposition automationComposition = new AutomationComposition();
Map<UUID, AutomationCompositionElement> automationCompositionElements = new HashMap<>();
@@ -114,9 +114,9 @@ public class AutomationCompositionInstantiationProvider {
nodeTemplates.forEach((key, template) -> {
ToscaNodeTemplate newNodeTemplate = new ToscaNodeTemplate();
- String name = key + instanceName;
+ String name = key + "-" + instanceName;
String version = template.getVersion();
- String description = template.getDescription() + instanceName;
+ String description = template.getDescription() + " " + instanceName;
newNodeTemplate.setName(name);
newNodeTemplate.setVersion(version);
newNodeTemplate.setDescription(description);
@@ -528,9 +528,9 @@ public class AutomationCompositionInstantiationProvider {
automationCompositionElements.put(automationCompositionElement.getId(), automationCompositionElement);
}
- automationComposition.setName("PMSH" + instanceName);
+ automationComposition.setName(instanceName);
automationComposition.setVersion(template.getVersion());
- automationComposition.setDescription("PMSH automation composition " + instanceName);
+ automationComposition.setDescription("Automation composition " + instanceName);
automationComposition.setState(AutomationCompositionState.UNINITIALISED);
automationComposition.setOrderedState(AutomationCompositionOrderedState.UNINITIALISED);
}
@@ -550,10 +550,10 @@ public class AutomationCompositionInstantiationProvider {
return toscaDefinitionsNames.stream().reduce("", (s1, s2) -> {
- if (s2.contains(INSTANCE_TEXT)) {
- String[] instances = s2.split(INSTANCE_TEXT);
+ if (s2.contains(HYPHEN)) {
+ String[] instances = s2.split(HYPHEN);
- return INSTANCE_TEXT + instances[1];
+ return HYPHEN + instances[1];
}
return s1;
@@ -561,24 +561,6 @@ public class AutomationCompositionInstantiationProvider {
}
/**
- * Generates Instance Name in sequential order and return it to append to the Node Template Name.
- *
- * @return instanceName
- */
- private String generateSequentialInstanceName() {
- List<ToscaNodeTemplate> nodeTemplates = automationCompositionProvider.getAllNodeTemplates();
-
- int instanceNumber = nodeTemplates.stream().map(ToscaNodeTemplate::getName)
- .filter(name -> name.contains(INSTANCE_TEXT)).map(n -> {
- String[] defNameArr = n.split(INSTANCE_TEXT);
-
- return Integer.parseInt(defNameArr[1]);
- }).reduce(0, Math::max);
-
- return INSTANCE_TEXT + (instanceNumber + 1);
- }
-
- /**
* Retrieves automation composition Definition.
*
* @param template tosca node template