Replaced constructor calls with valueOf() method calls
to remove static analyzer warnings about inefficient
constructor calls

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: If5b0c9c3569759f738e01c0352e267ee8a995c5f

Merge "Implement TLS for calls into VNFM adapter"

Merge "Vulnerability fixes in ServicePluginFactory"

Merge "Upgrade springboot.version from 2.0.5 to 2.1.5."

Merge "validate model info for existing service with"

Merge "Use BuildingBlockExecution for PauseForManualTasks"

Merge "Add SpringClientFilter in ValetClient to autolog"

Merge "set multi-stage flag in listener to log success"

Merge "store heat env file sent to heat"

Vulnerability fixes in ServicePluginFactory

Vulnerability fixes in ServicePluginFactory according to the Sonar.
-Made constants final.
-Changed access scopes from public to private.

Issue-ID: SDC-2106
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: Iabaca0bd138b5dee22c95a4a0cc6cb5ebe442fd2

Upgrade springboot.version from 2.0.5 to 2.1.5.

Upgrade springboot.version from 2.0.5 to 2.1.5.
Fixed APIHandler junit test cases with expected DB error messages.

Issue-ID: SO-2157
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I77ce873018b88268b9221226b95c9c106f93a8d0

validate model info for existing service with

verify modelVersionId equals during SI AAI validation
correct SI name in junit test
update SI name in junit test comparison
add name/module id verifications for other objects
add back junit file after merge from remote
format jnit file after verify job failure
updated test input values for clarity
update name verifications with related ID values
adding junit for Configuration name verification
updated aai verify call, service dup error msg
removed aai depth 2 and refactored private test method
implement SI name verification with 3 additional fields
fix formatting comment line length limitation

Issue-ID: SO-2161
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I43f5cb0601dfa8b41f474615b639fce0572f5613

Merge changes I3e363f64,I52ec2fc9

* changes:
  include a bak folder for obsolete files
  updated the doc as per the request

Merge "Fixed sonar issue in QueryServiceVnfs.java"

Merge "Fix the License issue inside the sample csar"

Merge "Enhanced List Level flow with backward support"

Merge "Javadoc and logging improvement"

Merge "fix image urls in docs"

Enhanced List Level flow with backward support

Modified the e2e service instance flow to
support both new list types groups along with
backward compatibility to support old types
alloted resources, network etc.

Change-Id: I1cb128e259c54b80009840914b2c1cee8cf8a4df
Issue-ID: SO-1393
Signed-off-by: sarada prasad sahoo <sarada.prasad.sahoo@huawei.com>

Use BuildingBlockExecution for PauseForManualTasks

Fix variable settings and execution object usage for manual tasks.

Issue-ID: SO-2113
Signed-off-by: Kuleshov, Elena <evn@att.com>
Change-Id: I3fa24d70b46658ce5afcdc843d5482e157641112

Javadoc and logging improvement

Added Javadocs for VnfAdapterVfModuleResources and
VnfTopologyOperationRequestMapper classes

Issue-ID: SO-2052
Signed-off-by: sunilb <sb00577584@techmahindra.com>
Change-Id: Iec6addb415720f63c32b5ab99e63aefe0ef3235a

Merge "update readme.md"

Add SpringClientFilter in ValetClient to autolog

Add SpringClientFilter in ValetClient to autolog msgs
Modify to use our RestTemplateConfig for RestTemplate
Use Provider for thread-safe RestTemplate autowire

Issue-ID: SO-2160
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I02d1995ebd6d2c93477008feb13193d3c0b3cd05

set multi-stage flag in listener to log success

fixed null pointer error when multistage not set

Issue-ID: SO-2159
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I1f2313d074bea404776877ef56607a814e46e514

store heat env file sent to heat

Initial commit, include heat environment info in cloud-api_requests.
Updated MSOHeatUtilsTest.saveStackTest unit test to assert
cloudApiRequests requestBody

Issue-ID: SO-2158
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I333cae7746459917ba5e64feb07db4f710b8d99b

fix image urls in docs

Corrected image URL for SO Install Docker documentation page
Corrected images URLs for SO Configure git & gerrit documentation page

Issue-ID: SO-2150, SO-2151
Signed-off-by: Katsiaryna Kazak <k.kazak@samsung.com>
Change-Id: Ib163c3159ae284f6c731795b0cbb53ed176ade11

update readme.md

Updated developer documentation in readme.md:
components, how to compile, format, build docker images,
images list, how to start them

Issue-ID: SO-2148
Signed-off-by: Katsiaryna Kazak <k.kazak@samsung.com>
Change-Id: Ib0e8e78af3e4497d8d9a6789c3af49f7025ba428

Merge "Fix delete of generic neutron network"

Fix delete of generic neutron network

Delete of Generic Neutron network was not working.
Logic checking for association with vf-module appears to
have been reversed.

Change-Id: If541c1133322f75c70f86cd77d8bccd88b94f356
Issue-ID: SO-2082
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>

set multi-stage flag in listener to log success

set multi-stage flag in listener to log success msg
added post listener for requests db object

Issue-ID: SO-2153
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I4b27cbb69b983bd2f7c0d72c8ebb4d9c43201120

Merge "Major Sonar fixes in InstanceManagement"

Merge "Removed not used loggers"

Merge "Major Sonar fixes in E2EServiceInstances"

Merge "Major Sonar fixes"

Merge "enhance get orchestration request"

Merge "Update SO to handle Async VNF operations to SDNC"

include a bak folder for obsolete files

Issue-ID: SO-1230

Signed-off-by: seshukm <seshu.kumar.m@huawei.com>
Change-Id: I3e363f6445f6f76b62f0ca1bf9467fd795db50e7

updated the doc as per the request

Issue-ID: SO-1230

Signed-off-by: seshukm <seshu.kumar.m@huawei.com>
Change-Id: I52ec2fc97abfd71b31a59fcb0416b0472e83d39c

Removed not used loggers

-Removed unnecessary loggers according to Sonar.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I05a0cc3187692fa17a6734d1c2a6be33d4b5ee34

Major Sonar fixes in InstanceManagement

-Removed unnecessary code.
-Removed unused variables.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: Ie85cb51da959dacc80b61aa9d5bb608b4198a5fb

Major Sonar fixes in E2EServiceInstances

-Removed unnecessary code.
-Removed unused variables.
-Introduced constants for repeating Strings.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I4ac692849508178390796d149b2c3f2fddcd093b

- Converted NetworkResource to use the parser

- Converted NetworkResource to use the parser getEntity method.
- run the formatting command so that it aligns correctly.

Issue-ID: SO-2147
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: Id6aa6c2967ec4014d6512851fa7dfff088946081

Major Sonar fixes

Major Sonar fixes in different classes:
-Removed unnecessary code.
-Removed unused variables.
-Removed unnecessary imports.
-Simplified diamond operators.
-Improved String comparisons in equals.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I279c63f8e7161d6e33d5d6a28b938c0c64e90f1f

Merge "Enhanced List Level flow with backward support"

Implement TLS for calls into VNFM adapter

Issue-ID: SO-2143
Change-Id: I2fcacab7aebc9a22b952d881b0bf2404e1638b37
Signed-off-by: MichaelMorris <michael.morris@est.tech>

enhance get orchestration request

Enhance get orchestration request to include workflow step
Began working on adding call for activity name
Added call to return activity information
Updated to return last task executed
Added last task executed info to returned string
Updated mocks to fix failing unit tests
Added responses for mocks to Camunda for process and activity history
Added tests to cover taskName changes
Began adding tests to cover changes to RequestHandlerUtils
Added more unit tests for activity lookup
Updated error messages for taskName
Moved properties to application.yaml file
Added unit tests to add more coverage to task retrieval
Moved Camunda calls to CamundaRequestHandler class
Removed unused imports from RequestHandlerUtils
Fixed formatting of tests and removed handler where not used
Removed CamundaRequestHandler from JerseyConfiguration
Added retryTemplate for camunda calls

Issue-ID: SO-2142
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I09a82e3458e8f611e4bbaf3ce9ab5de36b1b060a

Update SO to handle Async VNF operations to SDNC

Update SO to handle Async VNF operations to SDNC

Issue-ID: SO-2141
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I20037306e9ce5b8df434511a46c0a3399e335c3d

Merge "Added servicename to MDC so that it gets logged"

Vulnerability fixes in ServicePluginFactory

Sonar vulnerability fixes in ServicePluginFactory:
-Added logger.

Issue-ID: SO-2139
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: Id1752ea35db970b20d02e730dac9900c411958ca

Added servicename to MDC so that it gets logged

Added servicename to MDC so that it gets logged and added enter and
exit markers.

Issue-ID: SO-2122
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I63ff5e5de21d851c989ffa5db2051755751d9ead

Merge "basic auth for so-monitoring"

basic auth for so-monitoring

Add basic auth for so-monitoring app
Issue submitted in pentest report

Change-Id: I8e826da9b9f66e893826fd9b40b3b26623b2ab8d
Issue-ID: OJSI-169
Signed-off-by: k.kazak <k.kazak@samsung.com>

Merge "Set networkName value to the name from L3network"

Fix the License issue inside the sample csar

Issue-ID: SO-2138

Change-Id: I0dd532abf983a248b60f91aea4a6fbd05e33daab
Signed-off-by: seshukm <seshu.kumar.m@huawei.com>

Merge "Added null check for yaml to prevent crash at getParameterListFromEnvt()"

Merge "Commented out updatedAttribute declaration to prevent static analyzer warning about unused variable"

Merge changes I5e72e1c6,I5e1cb624

* changes:
  Added null check for function param pFqdns to prevent crash at for loop.
  Added check for null ptr before dereferencing at gBBInput.getCustomer().getServiceSubscription() - to prevent static analyzer warning

Merge "Fix error updating request_processing_data"

Merge "Remove duplicate name check in createAAITasks."

Set networkName value to the name from L3network

Set networkName value to the name from L3network object.
Add JUnit argument captures for naming creates
Fix formatting errors in the JUnit file.

Issue-ID: SO-2123
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: Ie4d2bd86f1a47bd5742aa7786dc9cba47fa11392

Removing application-local.yaml files

Removing the application-local.yaml files because they're no longer
maintained or working.

Issue-ID: SO-2121
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I191c8775de931671a4657be8940e6aec64023359

Fix error updating request_processing_data

Fix error when updating existing record in request_processing_data
table.
Use correct field name in the toString method in RequestProcessingData
bean class.
RequestDbClient.updateRequestProcessingData should include entity id in
the URI on the PUT request.
Add new Config class ExposeEntityIdResourceConfiguration to make
Spring data Rest return id in the response for RequestProcessingData.
Make spring-boot-starter-data-rest not optional dependency in
mso-requests-db-repositories
Added unit test for RequestDbClient.updateRequestProcessingData()
method.

Issue-ID: SO-2120
Signed-off-by: Benjamin, Max (mb388a) <mb388a@att.com>
Change-Id: I733be06185e5c19a0b7112918ce0a0f2abf861d7

Merge "Implement TLS for calls from VNFM adapter to VNFM"

Remove duplicate name check in createAAITasks.

Remove duplicate name check in createAAITasks. This hould be done prior
to status check in WorkflowAction or BBInputSetup.

Issue-ID: SO-2119
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
Change-Id: I0d6d34476b1ab2b351314c9a23926f2fff0e4f89

Merge "Commented out initialization of workflowResponse to prevent static analyzer warning about workflowResponse never being set to anything but null."

Merge "Fixes in WorkflowAction"

Merge "Major Sonar fixes"

Merge "Vulnerability fixes in ServicePluginFactory"

Merge "Add requestContext to unassignVfModule tasks and"

Merge "Add SO BPMN name for NAME in tbl workflow"

Merge "We need to expand column request_status"

Implement TLS for calls from VNFM adapter to VNFM

Issue-ID: SO-2116
Change-Id: I1e5bdfcf3164545c89fb370014d49ef3ae6a9cf1
Signed-off-by: MichaelMorris <michael.morris@est.tech>

Add SO BPMN name for NAME in tbl workflow

Added  SO update the R__WorkflowDesignerData.sql  script to insert name of BPMN process
instead of  description in the NAME field of workflow table for VnfInPlaceUpdate and
VnfConfigUpdate
Issue-ID: SO-2112
Signed-off-by: Bonkur, Venkat (vb8416) <vb8416@att.com>
Change-Id: I288a1159e95c3798c37ff41a599662e5df8317ba

Add requestContext to unassignVfModule tasks and

Add requestContext to unassignVfModule tasks and tests

Issue-ID: SO-2115
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
Change-Id: Ifddfe32fab9406afb69ad9192351500cfbed3610

We need to expand column request_status

New script to expand column request_status on table
archived_infra_requests as well.

Issue-ID: SO-2114
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
Change-Id: Ife1168e658bbb9e7fa69541de42173947564cac8

add query stack data and populate table step

Need to check if audit is enabled first in the delete record task
Updates to audit delete redeisgn and data to table
Various updates to logging, audit object, added tag to query, updated
tag value.
Updated write method to make sure an entry for this heat stack doesnt
already exist to avoid duplicate entrys
Updated unit test to account for get query
Added so that id is not hidden from query results
Updated the was delete is done plus minor updates.
Updated delete method to use resttemplate instead of bowman client
Combined the write and get stack data task into the external tasks.
Removed the task to delete the stack data from db
Updates to junit request id so its not a random id for test
Updated service tasks in att BB to call external task
Update to be a real uuid format due to failing unit test
Updated fields to protected and updated unit test

Issue-ID: SO-2111
Signed-off-by: Benjamin, Max (mb388a) <mb388a@us.att.com>
Change-Id: I8f479a046cf837403f60c330ed6ad8bc08a9c82f

Added null check for yaml to prevent crash at
getParameterListFromEnvt()

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: I8fabba8d99e311f01a5fd5dad166e0bb643044e3

Fixing ASDCRestInterface for CSIT to simulate SDC req

Change-Id: I6331d08441b9bf1f621f56ec1b7d911851010e62
Issue-ID: SO-1948
Signed-off-by: waqas.ikram <waqas.ikram@est.tech>

Commented out updatedAttribute declaration to prevent
static analyzer warning about unused variable

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: I0d46bcc817d08a6fbad3501c56336dc532bb5907

Added null check for function param pFqdns to
prevent crash at for loop.

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: I5e72e1c6d74c5f197479062348090ab2b87fafae

Commented out initialization of workflowResponse to prevent
static analyzer warning about workflowResponse never being
set to anything but null.

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: Ib150e3836fda9b7d9785a574f904b2c61bfa39f4

Added check for null ptr before dereferencing at
gBBInput.getCustomer().getServiceSubscription() -
to prevent static analyzer warning

Issue-ID: SO-1841
Signed-off-by: Oleksandr Moliavko <o.moliavko@samsung.com>
Change-Id: I5e1cb6248412ae9e3036011bd96d708f36170d70

Fixes in WorkflowAction

Fixes in WorkflowAction according to the Sonar analysis:
-Fixed String comparison.
-Removed unnecessary method parameters.
-Updated junit test.

Issue-ID: SO-2107
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I7522269e237f736521a627bcd1c8f77b0c776a70

Major Sonar fixes

Major Sonar fixes in different classes:
-Removed unused variables.
-Simplified diamond operators.
-Removed unnecessary imports.
-Removed unnecessary code.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: Ib54fbe048700a4e31042481f0a9b13838552aee7

Vulnerability fixes in ServicePluginFactory

Sonar vulnerability fixes in ServicePluginFactory.
-Moved hardcoded IP addresses to the property file.
-Added junit test.

Issue-ID: SO-2106
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: Id33e17f7845c106212bbf4d4a8ca78641cea1396

Merge "Sonar Fix: SDNCAdapterApplication.java"

Merge "Major Sonar fixes in E2EServiceInstances"

Sonar Fix: SDNCAdapterApplication.java

Removed unused imports from the file

Issue-ID: SO-2103
Change-Id: Id126f5203789d866b340d86b55bbc616991afc99
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>

Merge "Sonar fix in CreateInventoryTask.java"

Merge "Sonar fixes in ManualTasks"

Merge "Sonar fixes in OrchestrationRequests"

Major Sonar fixes

Major Sonar fixes in different classes:
-Removed unnecessary code.
-Removed unused variables.
-Removed unnecessary imports.
-Simplified diamond operators.

Issue-ID: SO-2102
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I098e3dcd93b804e58220bb41ea15325e820e3a59

Sonar fixes in OrchestrationRequests

Sonar fixes in OrchestrationRequests according to the Sonar analysis.
-Removed unnecessary code.
-Removed unused variables.

Issue-ID: SO-2101
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I564f35519c5063028e1cf540eb81ab81cf923fd5

Sonar fixes in ManualTasks

Code fixes in ManualTasks according to the Sonar analysis.
-Removed unnecessary code.
-Removed unused imports.
-Removed unused variables.

Issue-ID: SO-2100
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I79848456cc88b7f2cb812f7054bb6cec12b584c0

Major Sonar fixes in E2EServiceInstances

Major code fixes in E2EServiceInstances according to the Sonar analysis.
-Removed unnecessary code.
-Removed unnecessary method parameters.
-Removed unused variables.

Issue-ID: SO-2099
Signed-off-by: Robert Bogacki <r.bogacki@samsung.com>
Change-Id: I0e217174455a6748c663b3d5cc92fdc22b67ce2d

Sonar fix in CreateInventoryTask.java

Fixed multiple sonar issues in this file

Issue-ID: SO-2098
Change-Id: I7da9ffae8ee11d8475445b5263bc7cfb78834a9c
Signed-off-by: Arundathi Patil <arundpil@in.ibm.com>

Merge "remove unneeded code"

Merge "Added null check to prevent call of getVnfResources() method for null vf object"

Merge "Added null check for heatStack to prevent crash at getCanonicalName() call"

Merge "add junit coverage for VnfmHelper"