Move pauseInstantiation from VfModule to BaseResource 34/107634/5
authorEinat Vinouze <einat.vinouze@intl.att.com>
Wed, 13 May 2020 15:19:11 +0000 (18:19 +0300)
committerIttay Stern <ittay.stern@att.com>
Sun, 17 May 2020 12:56:54 +0000 (12:56 +0000)
Issue-ID: VID-821
Change-Id: Iacb80bba25596b26cb33fd91d9b2c4a4f9a6d08b
Signed-off-by: Einat Vinouze <einat.vinouze@intl.att.com>
vid-app-common/src/main/java/org/onap/vid/job/command/VnfCommand.kt
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/BaseResource.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/InstanceGroup.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/InstanceGroupMember.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/Network.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/ServiceInstantiation.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/VfModule.java
vid-app-common/src/main/java/org/onap/vid/model/serviceInstantiation/Vnf.java
vid-app-common/src/test/java/org/onap/vid/job/command/VnfCommandTest.kt
vid-app-common/src/test/java/org/onap/vid/model/serviceInstantiation/InstantiationModelSerializationTest.java

index 016c66c..1eb9799 100644 (file)
@@ -7,8 +7,8 @@ import org.onap.vid.job.*
 import org.onap.vid.job.impl.JobSharedData
 import org.onap.vid.model.Action
 import org.onap.vid.model.serviceInstantiation.BaseResource
+import org.onap.vid.model.serviceInstantiation.BaseResource.PauseInstantiation
 import org.onap.vid.model.serviceInstantiation.VfModule
-import org.onap.vid.model.serviceInstantiation.VfModule.PauseInstantiation
 import org.onap.vid.model.serviceInstantiation.Vnf
 import org.onap.vid.mso.RestMsoImplementation
 import org.onap.vid.properties.Features
index 7617d77..04dfdb2 100644 (file)
@@ -30,6 +30,7 @@ import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import javax.annotation.Nullable;
 import org.apache.commons.lang3.StringUtils;
 import org.onap.vid.job.JobAdapter;
 import org.onap.vid.job.JobType;
@@ -38,6 +39,10 @@ import org.onap.vid.mso.model.ModelInfo;
 
 public abstract class BaseResource implements JobAdapter.AsyncJobRequest {
 
+       public enum PauseInstantiation {
+               afterCompletion
+       }
+
        protected String instanceId;
 
        protected ModelInfo modelInfo;
@@ -65,6 +70,8 @@ public abstract class BaseResource implements JobAdapter.AsyncJobRequest {
        @JsonInclude(NON_NULL)
        protected String originalName; //not used at backend, but stored for fronted
 
+       @JsonInclude(NON_NULL)
+       protected final PauseInstantiation pauseInstantiation;
 
        private static final Map<String, Action> actionStingToEnumMap = ImmutableMap.<String, Action>builder()
                        .put("Delete", Action.Delete)
@@ -91,6 +98,7 @@ public abstract class BaseResource implements JobAdapter.AsyncJobRequest {
                @JsonProperty("isFailed") Boolean isFailed,
                @JsonProperty("statusMessage") String statusMessage,
                @JsonProperty("position") Integer position,
+               @JsonProperty("pauseInstantiation") PauseInstantiation pauseInstantiation,
                @JsonProperty("originalName") String originalName) {
                this.modelInfo = modelInfo;
                this.modelInfo.setModelType(getModelType());
@@ -105,6 +113,7 @@ public abstract class BaseResource implements JobAdapter.AsyncJobRequest {
                this.isFailed = isFailed!= null ? isFailed: false;
                this.statusMessage = statusMessage;
                this.position = position;
+               this.pauseInstantiation = pauseInstantiation;
                this.originalName = originalName;
        }
 
@@ -193,4 +202,9 @@ public abstract class BaseResource implements JobAdapter.AsyncJobRequest {
 
        @JsonIgnore
        public abstract JobType getJobType();
+
+       @Nullable
+       public PauseInstantiation getPauseInstantiation() {
+               return pauseInstantiation;
+       }
 }
index 642e28b..b273e14 100644 (file)
@@ -45,7 +45,7 @@ public class InstanceGroup extends BaseResource implements JobAdapter.AsyncJobRe
         @JsonProperty("originalName") String originalName) {
 
         super(modelInfo, instanceName, action, null, null, null, null, rollbackOnFailure, instanceId, trackById, isFailed, statusMessage,
-            position, originalName);
+            position, null, originalName);
         this.vnfGroupMembers = vnfGroupMembers;
     }
 
index 758ee06..1fa283f 100644 (file)
@@ -37,7 +37,7 @@ public class InstanceGroupMember extends BaseResource implements JobAdapter.Asyn
         @JsonProperty("position") Integer position,
         @JsonProperty("originalName") String originalName) {
         super(new ModelInfo(), null, action, null, null, null, null, false, instanceId, trackById, isFailed, statusMessage,
-            position, originalName);
+            position, null, originalName);
     }
 
     @Override
index cec61f6..b81e8c2 100644 (file)
@@ -56,7 +56,7 @@ public class Network extends BaseResource implements JobAdapter.AsyncJobRequest
                @JsonProperty("originalName") String originalName) {
 
                super(modelInfo, instanceName, action, lcpCloudRegionId, legacyRegion, tenantId, instanceParams, rollbackOnFailure, instanceId, trackById, isFailed, statusMessage,
-            position, originalName);
+            position, null, originalName);
                this.productFamilyId = productFamilyId;
                this.platformName = platformName;
                this.lineOfBusiness = lineOfBusiness;
index 7c04a91..cae28f7 100644 (file)
@@ -107,7 +107,7 @@ public class ServiceInstantiation extends BaseResource implements JobAdapter.Asy
                                 @JsonProperty("vidNotions") VidNotions vidNotions,
                                 @JsonProperty("originalName") String originalName) {
         super(modelInfo, instanceName, action, lcpCloudRegionId, legacyRegion, tenantId, instanceParams, rollbackOnFailure, instanceId, trackById, isFailed, statusMessage,
-            null, originalName);
+            null, null, originalName);
         this.owningEntityId = owningEntityId;
         this.owningEntityName = owningEntityName;
         this.projectName = projectName;
index 46d4de8..dc6a2c6 100644 (file)
@@ -48,9 +48,6 @@ public class VfModule extends BaseResource implements JobAdapter.AsyncJobRequest
        @JsonInclude(NON_NULL)
        private final Boolean retainVolumeGroups;
 
-       @JsonInclude(NON_NULL)
-       private final PauseInstantiation pauseInstantiation;
-
        @JsonInclude(NON_NULL)
        private Boolean retainAssignments;
 
@@ -75,13 +72,12 @@ public class VfModule extends BaseResource implements JobAdapter.AsyncJobRequest
                @JsonProperty("pauseInstantiation") PauseInstantiation pauseInstantiation,
                @JsonProperty("originalName") String originalName) {
                super(modelInfo, instanceName, action, lcpCloudRegionId, legacyRegion, tenantId, instanceParams, rollbackOnFailure, instanceId, trackById, isFailed, statusMessage,
-                       position, originalName);
+                       position, pauseInstantiation, originalName);
                this.volumeGroupInstanceName = volumeGroupInstanceName;
                this.usePreload = usePreload;
                this.supplementaryParams = supplementaryParams;
                this.retainAssignments = retainAssignments;
                this.retainVolumeGroups = retainVolumeGroups;
-               this.pauseInstantiation = pauseInstantiation;
        }
 
        public String getVolumeGroupInstanceName() {
@@ -93,10 +89,6 @@ public class VfModule extends BaseResource implements JobAdapter.AsyncJobRequest
                return usePreload;
        }
 
-       public enum PauseInstantiation {
-               afterCompletion
-       }
-
        public List<UserParamNameAndValue> getSupplementaryParams() {
                return supplementaryParams;
        }
@@ -126,9 +118,6 @@ public class VfModule extends BaseResource implements JobAdapter.AsyncJobRequest
                return retainVolumeGroups;
        }
 
-       @Nullable
-       public PauseInstantiation getPauseInstantiation() {return pauseInstantiation;}
-
        public VfModule cloneWith(ModelInfo modelInfo) {
                return new VfModule(
                                modelInfo,
index 2ed2196..867e759 100644 (file)
@@ -66,7 +66,7 @@ public class Vnf extends BaseResource implements JobAdapter.AsyncJobRequest {
                @JsonProperty("originalName") String originalName) {
 
                super(modelInfo, instanceName, action, lcpCloudRegionId, legacyRegion, tenantId, instanceParams, rollbackOnFailure, instanceId, trackById, isFailed, statusMessage,
-                       position, originalName);
+                       position, null, originalName);
                this.productFamilyId = productFamilyId;
                this.platformName = platformName;
                this.lineOfBusiness = lineOfBusiness;
index ab54406..75640d4 100644 (file)
@@ -32,8 +32,8 @@ import org.onap.vid.job.JobsBrokerService
 import org.onap.vid.job.command.ResourceCommandTest.FakeResourceCreator
 import org.onap.vid.job.impl.JobSharedData
 import org.onap.vid.model.Action
+import org.onap.vid.model.serviceInstantiation.BaseResource.PauseInstantiation.afterCompletion
 import org.onap.vid.model.serviceInstantiation.VfModule
-import org.onap.vid.model.serviceInstantiation.VfModule.PauseInstantiation.afterCompletion
 import org.onap.vid.mso.RestMsoImplementation
 import org.onap.vid.properties.Features
 import org.onap.vid.services.AsyncInstantiationBusinessLogic
index 26e8f82..81ba957 100644 (file)
@@ -32,7 +32,7 @@ import static org.hamcrest.Matchers.hasProperty;
 import static org.hamcrest.Matchers.nullValue;
 import static org.hamcrest.Matchers.samePropertyValuesAs;
 import static org.onap.vid.model.Action.Create;
-import static org.onap.vid.model.serviceInstantiation.VfModule.PauseInstantiation.afterCompletion;
+import static org.onap.vid.model.serviceInstantiation.BaseResource.PauseInstantiation.afterCompletion;
 import static org.onap.vid.testUtils.TestUtils.setStringsInStringFields;
 import static org.onap.vid.utils.KotlinUtilsKt.JACKSON_OBJECT_MAPPER;