Added null check for execute() result to remove
static analyzer warning

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

Merge "Set WorkflowException on BPMN WF error"

Unit test cases for SVNFM Simulator

Change-Id: Ibc6e4d47007fb14dbcad97b27d47cdd808251c9d
Issue-ID: SO-1853
Signed-off-by: AndrewLamb <andrew.a.lamb@est.tech>

Set WorkflowException on BPMN WF error

WorkflowException needs to be set when execution of a custom workflows fails due to an error from BPMN

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

Merge "Added missing headers"

Merge "- Added serviceDeployed variable for"

Merge "Upgrade swagger in mso-catalog-db-adapter"

Merge "sdn post 1908 rollback and resume mso should"

Merge "check for null before setting headers"

Merge "mso should not send content type http header to"

Update Resume Logic and Add Workflow Listeners

Updated with the error log messages
Changed the code to do string compare for eventName
update workflowaction to only persist if not resume

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

check for null before setting headers

check for null before setting headers

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

mso should not send content type http header to

Updated getHeaders to not add Content-Type header on GET requests
Updated getHttpHeaders boolean to includeContentType

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

Added missing headers

Added missing headers to files

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

sdn post 1908 rollback and resume mso should

Do not remove UnassignFabricConfigurationBB when rolling back to created
Do not remove UnassignFabricConfigurationBB when rolling back to
created. Added unit test.
Enhace error message for unassign and deactivate fabric flows if they
fail during rollback
Looks like a bad whitespace in pom file
Updated rollback failure message for fabric. Added unit test and robot
tests.

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

- Added serviceDeployed variable for

- Added serviceDeployed variable for distributions with 2 resources.

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

Merge "Prevent reading twice from ResponseEntity stream, since this caused IOErrors."

Merge "VNFM adapter relationship handling improvements"

Merge "Fix default homing"

Merge "Exception string in checkRelationshipRelatedTo"

Exception string in checkRelationshipRelatedTo

Exception string in checkRelationshipRelatedTo not being returned
properly. Added a new exception - UnassignNetworkException.
Added a new unit test to test the exception case as per review comment.
Updated UnassignNetworkBBTest.java file to fix the formatting errors.
Updated unit test , removed setVariable, it wasn't needed to do as it
was performed in the actual method call.
Updated pom.xmp  file to fix the formatting errors.
Updated pom.xml file to fix the formatting errors.

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

update logging to match onap logging library

Updated to use changes added to logging library in ONAP
Updated RestClient to set mdc rather than extend filter
Updated filter removal to include new spring filter
Updated to exclude filter causing failure in catdb
Excluded filter causing failures in requestdb
Updated requestdbatt to exclude filter
Updated check for spring filters to fix failing tests

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

VNFM adapter relationship handling improvements

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

Prevent reading twice from ResponseEntity stream, since this caused IOErrors.

Also moving empty response checks to parseJsonToGelAllPnfCorrelationId

Issue-ID: SO-2349
Signed-off-by: grabinsk <maciej.grabinski@nokia.com>
Change-Id: I316856d56106a1ae715812628695406f4147765d
Signed-off-by: grabinsk <maciej.grabinski@nokia.com>

Merge "Async vnf adapter response not getting to bpmn"

Add SO Update Check if SDC activities deployment end point server Up for custom worlflows

Make changes to verify if SDC activities deployment
end point(mso.asdc.config.activity.endpoint) server up
for custom workflows
Issue-ID: SO-2348
Signed-off-by: Bonkur, Venkat (vb8416) <vb8416@att.com>
Change-Id: I3b90b6d8e95681a685b23dfaa2f65596532e5b0e

Fix default homing

Issue-ID: SO-2307
Change-Id: Iee2562c6f38a440006d3a6108ce6f6fd6dbedcf6
Signed-off-by: kurczews <krzysztof.kurczewski@nokia.com>

Merge "Build payload APPC parameter from UserParams"

Merge "Remove the declaration of thrown exception 'org.onap.so.apihandlerinfra.infra.rest.exception.NoRecipeException', as it cannot be thrown from method's body."

Merge "getentity csar logging"

Async vnf adapter response not getting to bpmn

Async vnf adapter response is not getting to bpmn @OneWay web
methods in VnfAdapterNotifyServiceImpl class do not work with the
@Async  handleCallback() method in CallbackHandlerService. Updating
code to call the non-async version of the (overloaded) handleCallback
method until we can figure out how to properly do @oneway to @async.
Added junits to all methods in VnfAdapterNotifyServiceimpl

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

Use the configurationId associated to vnfc if

Use the configurationId associated to vnfc if present.
removing the  autowiring of the Configuration bean class

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

Upgrade swagger in mso-catalog-db-adapter

Issue-ID: SO-2305
Change-Id: I25ac20a96cf44a7c8abaa3b78f3b256cb2002d3f
Signed-off-by: arindamm <arind.mondal@samsung.com>

Merge "- Removed the cloud_technology artifact type."

- Removed the cloud_technology artifact type.

- Removed the cloud_technology artifact type.

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

Remove unused table requestdb.active_requests.

Remove unused table requestdb.active_requests.
renamed file to nearest flyway versioned number

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

ServiceRecipeRepository id wrong type Long (vs

ServiceRecipeRepository id wrong type Long (vs Integer).

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

Build payload APPC parameter from UserParams

If a payload parameter is not specified on VID request for APPC interaction, we will need to build it from UserParams from requestCOntext.

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

Merge "bump the SO version to 1.5.2"

bump the SO version to 1.5.2

Issue-ID: SO-2310

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

Merge "Fix the ingestion of mixed resource"

Merge "Added null check to avoid crash at getErrorCode() call; fixed typos in error messages; removed unnecessary try...catch block"

Merge "Fixing null issue and adding zooming"

getentity csar logging

- Updated to use the getEntity API for csar logging.
- Formatted the logging file to pass the build.

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

Added null check to avoid crash at
getErrorCode() call; fixed typos in error
messages; removed unnecessary try...catch block

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

Merge "error message formatter now handles percent"

Merge "Removed redundant null checks (at these points vnfCustom can be empty, but not null)"

Merge "Integrate Logging Library"

Fixing null issue and adding zooming

Change-Id: I6a3ff57b6131c10965400beac43c4c8101b24397
Issue-ID: SO-2274
Signed-off-by: rokenny <ronan.kenny@est.tech>

Merge "Set payload to empty when it is not specified"

Merge "fix of the instanceParams adding to the CDS request"

Merge "bug fix - avoid npe in pnf flow"

Removed redundant null checks (at these points
vnfCustom can be empty, but not null)

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

bug fix - avoid npe in pnf flow

Change-Id: I8e848c2bdcec0822ae08280223297b4825e9b7c2
Issue-ID: SO-2289
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>

Fix sonar issues

Either log or rethrow this exception
Make "attributeValue" transient or serializable
"attributeName" is already a string, there's no need to call "toString()"

Issue-ID: SO-2226
Change-Id: If6ae623202f2c243d73839d0aaefba5ba5c50dd4
Signed-off-by: Parshad Patel <pars.patel@samsung.com>

Set payload to empty when it is not specified

Handle a case of non-existent payload and improve error reporting on APPC request construction

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

error message formatter now handles percent

error message formatter now handles percent encoding

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

Merge "Add SO Update to default the controller type query as "APPC" instead of null."

Integrate Logging Library

Updated SO to use ONAP provided filters and constants to meet logging
standards.
Removed redundant classes
Updated constant references

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

Merge "remove the unused import"

Merge "Created external task utils in a common location"

Merge "Added ACTIVATED Orchestration Status same as"

Merge "updated all sql files including in tests to use"

Merge "Upgrade swagger apihandlerinfra"

Merge "Distribution fails when no user input required in design"

Upgrade swagger apihandlerinfra

Issue-ID: SO-2280
Change-Id: If9fb53295ce483b69ff20610da50227062ed5c48
Signed-off-by: arind.mondal <arind.mondal@samsung.com>

Merge "General sonar fixes"

General sonar fixes

+)Format logger
++) Remove toString()
+++) Extract nested IF

Issue-ID: SO-2226
Change-Id: Iff28408584b04ffd03b6aca0f647d0af9090a9c2
Signed-off-by: arind.mondal <arind.mondal@samsung.com>

updated all sql files including in tests to use

updated all sql files including in tests to use the correct column
name. Updated hibernate mapping to use the correct column name as well.

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

Merge "Add SO Update Check if SDC activities deployment end point for custom worlflows is alive"

Add SO Update Check if SDC activities deployment end point for custom worlflows is alive

Make changes to verify if SDC activities deployment
end point(mso.asdc.config.activity.endpoint)  for custom worlflows is alive
Issue-ID: SO-2294
Signed-off-by: Bonkur, Venkat (vb8416) <vb8416@att.com>
Change-Id: Iff916a33b403d507a383cebf8d668bb5cbf9fb55

fix of the instanceParams adding to the CDS request

Change-Id: I3bff9b5cac47d5fa29fbe503e2c3752a2b207aab
Issue-ID: SO-2208
Signed-off-by: Lukasz Muszkieta <lukasz.muszkieta@nokia.com>

Distribution fails when no user input required in design

When we distribute a design from SDC and design does
not have any user input in properties, then distribution
fails with error ArrayIndexOutOfBoundException

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

Change names of two of catalogdb tables to lowercase

Table name used in Hibernate for catalogdb tables all need to be in lowercase.

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

Added ACTIVATED Orchestration Status same as

Added ACTIVATED Orchestration Status same as ACTIVE.

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

Created external task utils in a common location

Created external task utils in a common location then refactored
external task to use these common utils
Updated junit errors and failures in openstack
Removed env param and updated unit test with class
Remove env file from parm in unit test class
Match argu to match class abstract to con.
removed implementation within anonymous class

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

Merge "Set AAIClient on AAIValidatorImpl"

Merge "- Convert NetworkCollection to use GetEntity API."

Merge "Added index infra_active_requests optimal query"

Set AAIClient on AAIValidatorImpl

AAIValidatorImpl requires setting an initialized AAIClient before validation calls

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

- Convert to use the GetEntity API

- Converted to use the GetEntity API for AllottedResources

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

Added index infra_active_requests optimal query

Added index infra_active_requests optimal query

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

- Convert NetworkCollection to use GetEntity API.

- Convert NetworkCollection to use GetEntity API.
- Updated to correctly format the code..
- Fix setting of parsing of the quantity field.

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

Merge "Log exception"

Log exception

Either log or rethrow this exception
Use isEmpty() to check whether the collection is empty or not

Issue-ID: SO-1841
Change-Id: Ia964a871497df8a7211260ad1ad8f1efa8ae4d42
Signed-off-by: Parshad Patel <pars.patel@samsung.com>

Merge "fix public NS distribution issue"

Merge "Fix rollback for ConfigAssignBB"

fix public NS distribution issue

Fix Multiple VFModule under single service distribution

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

Merge "General sonar fix add logger"

Merge "fix create call to A&AI"

General sonar fix add logger

Issue-ID: SO-2226
Change-Id: I5a826a9ea75b3065987a274ab7962c4a5fc1d614
Signed-off-by: arind.mondal <arind.mondal@samsung.com>

General Sonar fixes

+) Make field transient or serializable
++) Organize imports

Issue-ID: SO-2273
Change-Id: I04eeb02c4052e3a790db865b59f873694fc1514d
Signed-off-by: arind.mondal <arind.mondal@samsung.com>

remove the unused import

remove the unused import

Issue-ID: SO-2174
Change-Id: Idae5e586e08d6b39dcebfccfd5aca5e1c455eb34
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

Remove the declaration of thrown exception 'org.onap.so.apihandlerinfra.infra.rest.exception.NoRecipeException', as it cannot be thrown from method's body.

remove the exception

Issue-ID: SO-2174
Change-Id: Ied64bf5774e3f9ba5d5e640d1f934eb3259e4173
Signed-off-by: Thugutla sailakshmi <tsaila10@in.ibm.com>

Fix issues in so-cloudify-client

Annotate the "CloudifyClientConnector" interface with the @FunctionalInterface annotation
Make this field final
Either log or rethrow this exception
Make "response" transient or serializable

Issue-ID: SO-1841
Change-Id: I13fa6b911d8789aed26fd27da23318f6dbca3760
Signed-off-by: Parshad Patel <pars.patel@samsung.com>

fix create call to A&AI

createIfNotExists should be used rather than create

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

Merge "sonar fix for ExtractPojosForBB class"

Merge "Added back MDC.put to generated missing requestID"

Merge "- Make association from the cvnfc to the fabric"

Merge "Update requestStatus to FAILED on activity error."