Merge "fix critical sonar bugs"

Use APIH credentials in 'mso.db.auth'

Updated to use methods: env.getRequiredProperty() and msoLogger.error().
Added the msoKey and the CryptoUtils.decrypt() code, and used
"mso.camundaAuth".
Used APIH credentials in 'mso.db.auth' as source instead of
'mso.camundaAuth' (need decrypt code by CryptoUtils).

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

Merge "query clients now set their expected object"

Merge "fix issue where we attempt audit with no servers"

Merge "updated camunda springboot version"

Merge "Process query of Network Policies from A&AI"

Merge "extract subinterface_network_quantity from policy"

query clients now set their expected object

unit test is using the wrong number of keys for aai uri
query clients now set their expected object for method

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

fix issue where we attempt audit with no servers

fix issue where we attempt audit with no servers

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

updated camunda springboot version

Added required unimplemented methods to classes

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

Process query of Network Policies from A&AI

Process query of Network Policies from A&AI as a NetworkPolicies object.

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

extract subinterface_network_quantity from policy

- Removed duplicate code that was causing a NumberFormatException
- Changed name from crTarget to policyNetworkCollection
- Added code to check we are setting the quantity on the correct
NetworkCollection
- Added code to extract the subinterface_network_quantity from the
nodeTemplate policy

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

always increment gCurrentSequence

always increment gCurrentSequence to properly interact with retry and
rollback logic

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

Merge "fix fields not set correctly in Fabric Policy dist"

Merge "cloud config migration from openstack to catdb"

Merge "fix major sonar bug"

Merge "fixed CandidateType json serialization"

Merge "add new functionality to aai client"

Merge "workflow action should read config for max retries"

Merge "Update Working with SO Docker Documentation"

Merge "Remove un-needed cacheing layer for OS Clients"

Merge "Strip off TenantId from self link url"

fix critical sonar bugs

Restore interrupted state when InterruptedException is thrown in SDNCRestClient
Added test for this case
Replaced formatting anchor with text empty object in GraphInventoryResultWrapper
Escaped anchor is also considered as bug by sonar
Case is already covered in AAIResultWrapperTest
Marked comparison of the same object as false positive in EqualsAndHashCodeTester
Added clearer comment to this comparison

Change-Id: I24496c2323c33599db99a3d3b59fc4f6a008cf1f
Issue-ID: SO-1492
Signed-off-by: k.kazak <k.kazak@samsung.com>

Strip off TenantId from self link url

Strip off TenantId from self link url

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

fixed CandidateType json serialization

fixed CandidateType json serialization

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

Merge "Associate PNF instance with Service"

workflow action should read config for max retries

workflow action should also read config for max retries

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

add new functionality to aai client

Remove only the first occurrence of parentUri from uriTemplate.
add createIfNotExists to transaction clients
moved classes to generic GraphInventory location
removed unused interfaces from graphinventory client
refactored all graph inventory clients

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

Remove un-needed cacheing layer for OS Clients

Remove un-needed cacheing layer for OS Clients

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

fix fields not set correctly in Fabric Policy dist

- Added a null check for vfModuleMemberName.
- Added some extra logging to show the VF members and cvnfc template
name
- Updated to fix fields not set correctly in Fabric Policy
distributions.

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

Update Working with SO Docker Documentation

patch description: Update Working with SO Docker
documentation for casablanca release.

Change-Id: If7a4601e83e0ca83f99a9f74e1f282abc3854bdd
Issue-ID: SO-1505
Signed-off-by: mohamed waly <mohamed.waly@orange.com>

Merge "Fixed possible NPE in ResourceRequestBuilder"

Merge "use encrypted auth for dmaap"

Associate PNF instance with Service

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

Fixed possible NPE in ResourceRequestBuilder

According to Sonar current implementation could cause
NullPointerException.
Added JUnit test.

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

fix major sonar bug

Removed useless 'instanceof' operator as it always resolves as true
Replaced it with checking for null
Removed unused imports
Added information about modifications on the top
Added junit tests for the method

Change-Id: I7a902ad2852e0ed0d507a6309857a1bab74921de
Issue-ID: SO-1491
Signed-off-by: k.kazak <k.kazak@samsung.com>

Merge "Add Pagination and sorting"

Merge "Add junit tests for AdapterRestClient"

Merge "Fix for asdc service."

Merge "Fix unmarshalling errors"

Merge "made max retries configurable via mso config"

made max retries configurable via mso config

fixed an error with the wrong variable being printed
made max retries configurable via mso config

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

Fix for asdc service.

Removed exception statements and added assert statements
Updated to make sure duplicate config resources aren't getting added
to the service.

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

Fix unmarshalling errors

add data assertion to network update unit test
Fix unmarshalling errors on updateNetworkResp and err

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

restructured a&ai client

made generics a bit safer and updated groovy tests

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

Add Pagination and sorting

Change-Id: I1374472395a5a2af712c5f06c756c013de08bd27
Issue-ID: SO-1406
Signed-off-by: Andrei_Barcovschi <andrei.barcovschi@ericsson.com>

Merge "added license headers to various java files"

Merge "fix cvnfc bean and repository access"

Merge "Increase long poll DMaaP timeout value to 60"

Merge "Remove resources that are not neutron networks"

fix cvnfc bean and repository access

update repository id field type for
fix cvnfc bean and repository access

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

Increase long poll DMaaP timeout value to 60

Corrected a JUnit to include updated timeout value.
Increase long poll DMaaP timeout value to 60 seconds

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

added license headers to various java files

added license headers to various java files

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

Remove resources that are not neutron networks

Remove resources that are not neutron networks

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

add DSL endpoint support to A&AI Client

add DSL endpoint support to A&AI Client

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

Merge "provide correct custIds"

Merge "populate modelinfovfmodule from catalog db"

Add junit tests for AdapterRestClient

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

mso is not updating the vip in aai for the vnf

Update JUnits for multiple invocation argument verifications
Implement delete mechanism for network policies and vnf adapter params
in AAI.
Fix a JUnit for getNetworkPolicy().
Implement creation of network policies and setting of
openstack-provided params for vf module in AAI
CreateNetworkPolicies in AAI functionality update
Add the plumbing for create network policy in AAI.
Enable VnfAdapterBB to return fqdn and management ip info

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

Merge "store raw distribution notification in db"

store raw distribution notification in db

updated the test case to verify ASDC notification sent is the same as
one persisted in the DB and verified it
updated the test case to verify ASDC notification sent is the same as
one persisted in the DB
Added unit test case for WatchdogServiceModVerIdLookup changes
Updated test resources schema.sql and data.sql to include changes to
watchdog_service_mod_ver_id_lookup table
Added length to consumer id column with WatchdogServiceModVerIdLookup
class
ASDC Controller has been enhanced to include the notification message
and consumer id as part of existing table
watchdog_service_mod_ver_id_lookup.

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

improve vf module logic

remove vf module from list when calculating index

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

Merge "logic to handle null values for vf module index"

populate modelinfovfmodule from catalog db

add in a junit test to check if all vfModules model info get populated
populate all vf modules for the vnf not just the orchestrated

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

provide correct custIds

correct variable name and use bean comparison
included object assertion in junit on catalog client
update test name to reflect return value
fix cvnfc query by 2 customization uuid relationships
provide customization ids to module query input

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

logic to handle null values for vf module index

added logic to handle null values for vf module index

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

Removed duplicate classes from api handler

removed duplicate classes

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

Merge "auto generated vf module index param"

Merge "Updated fields on sniro request"

Merge "Clean up Unit tests for audit"

Merge "Merge Casablanca"

cloud config migration from openstack to catdb

remove the wait and sleep code in migration
move cloud config migration from openstack to catdb

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

Updated fields on sniro request

Updated setter methods in avpn bonding classes.
Updated fields on sniro request to be idenitier instead of candidate

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

Clean up Unit tests for audit

Fix file names to adjust for case sensitivity
Clean up Unit tests for audit

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

Merge Casablanca

Issue-ID: SO-1460
Change-Id: I102ab0dddb8c26ca4001320ee8eb5b34ae9f0238
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "add maven build properties to spring actuator"

Merge "add in a retry function for cloud config migration"

Merge "compare resources from openstack to aai"

Merge "make sure the NetworkResourceUUID are correct."

compare resources from openstack to aai

Address Review comments on pull request
Update Unit test to have proper variable for testing
update logic inside bpmn to reflect proper variable
Update unit test to check property for audit
Add flag to turn on and off audit behavior
Add license headers, remove un-used files
Update Building Block Tests and Activity Names
Fix additional unit tests in so-bpmn-tasks layer
Fix broken Junit Test, and correct Bug in if statement
Add annotation to ignore class under test profile
Additional bug fixes and unit tests for classes
Add Additional JUNITS, add more sample data
Add Openstack Audit functionality to Adapter

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

add maven build properties to spring actuator

add maven build properties to spring actuator

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

auto generated vf module index param

added test for when this is the first vfmodule added
unit test no longer tests already created vnf
refactored test to be more clear about uuid grouping
Added javadoc for getLowestUnusedVfModIndex method.
Increment index by vf module model invariant uuid
Fix unnecessary whitespace delta for GenericVnf file
Fix GenericVnf and modify junit for setting vfmodules

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

add in a retry function for cloud config migration

add in a retry function into cloud config migration

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

make sure the NetworkResourceUUID are correct.

- Added test asserts to show that the NetworkResourceUUID was updated
correctly.
- Added a NetworkResourceCustomization Audit to make sure the
NetworkResourceUUID are correct.

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

Merge "Add setting of serviceId to productFamilyId"

Merge "Removed check for ServiceProxyCustomization"

Merge "added list of flows to execution for cockpit"

Removed check for ServiceProxyCustomization

- Added test records for the testServiceProxy testcase
- Removed check for existing ServiceProxyCustomization since multiple
services can associate to the same customization record.

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

Add setting of serviceId to productFamilyId

Add setting of serviceId to productFamilyId and provStatus to PREPROV
on GenericVnf creation

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

added list of flows to execution for cockpit

added list of flows to execution for cockpit readability

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

use OE from AAI with sdc

Updated the SDC API call with the ECOMP OE from AAI relationship
instead of VNF OE value.

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

Updated:wrong redirection of link & allignment.

Issue-ID: SO-1440

Change-Id: I10d7d466457db6c94299bee490fa8f7f091aa429
Signed-off-by: Manish Kumar <manishjpiet@gmail.com>

Update the SO release notes

Issue-ID: SO-1428

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

Merge "Merge 'origin/casablanca' into master"

Merge 'origin/casablanca' into master

Issue-ID: SO-1435
Change-Id: If065ef5c91e769452fd6701fa6c28a23b4bdf2b2
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "add custom object support to a ai client"

updated the SDC tosca to 1.4.63

Issue-ID: SDC-1955

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

pool is now serializable

add implements Serializable to openstack Pool object
Make Openstack Pool object serializable - for netwk

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

add custom object support to a ai client

enhanced custom support of objects in client
simplified the way objects are added to the type map
attempt to search all packages with ClasspathHelper
do not allow reflections to log so much
former enums are now serializable again
added the ability to create custom object types for A&AI client

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

support alloc pool from dhcpStart/end

Fix to include alloc pool from dhcpStart End on reqs

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