Merge "converted test to dynamic port"

Merge "Do not log stack trace when http not found occurs"

Merge "Created Ref Data tbl for controllerSelection"

converted test to dynamic port

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

Merge "retired flows no longer in use"

Merge "correct return values of asBean"

Do not log stack trace when http not found occurs

do not log stack trace when http not found occurs

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

retired flows no longer in use

corrected spelling for migration sql file
delete 1802-style flow, MSO to create l3network in AAI
Revert "retire 1802-style flow, MSO to create l3network in AAI"
retire 1802-style flow, MSO to create l3network in AAI

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

correct return values of asBean

More adds of Optional<> to other ServiceInstance objects.
Change to add the Optional<> to the ServiceInstance object.

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

aairesultwrapper is now actually serializable

added in a proper serialization and deserialization test
addressed any comments from pull
added in a serialization test for AAIResultWrapper and updated one of
the null returns
change jsonBody in AAIResultWrapper back to String

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

Rename asdc-controller docker image

OOM would like it to be called onap/so/sdc-controller.

Change-Id: Ifa37b4fa62f214a1ff56e31c4f4f4eca5b4566fc
Issue-ID: SO-897
Signed-off-by: Rob Daugherty <rd472p@att.com>

Created Ref Data tbl for controllerSelection

RD: I rebased this and fixed the unit tests.

Issue-ID: SO-763
Change-Id: Ie5f6a708cffbc70a9481afd35db3047fe44cb088
Signed-off-by: Prema Bhatt <pb6115@att.com>
Signed-off-by: Rob Daugherty <rd472p@att.com>

Prod fixes August 21st

changes required for unit tests to pass again
Changes sniro homing BB license demand to be a named array per sniro
defect.
boolean checks must now account for null
fixed compilation issues in code
updated tests and removed primatives from bbobjects
upgraded to A&AI dependency with Boolean

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

Merge "write update status of BBs progress"

Merge "Do not serialize AAIResultWrapper"

Merge "removed classes no longer in use"

Merge "Changed RequestFactory for Catalog DB client"

Merge "created generic HttpLookupUri class"

Merge "Renamed migration from version 4.2 to 4.3"

Merge "Prod fixes August 18th"

Merge "get service type and customer id from uri keys"

Merge "fixed filter of configuration flow calls"

Merge "update field in service decomp object"

removed classes no longer in use

Change create configuration to use yang service instance and removed
AAIServiceInstance pojo.

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

Do not serialize AAIResultWrapper

fixed a broken building block bpmn test
removed serialization of result wrapper

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

Changed RequestFactory for Catalog DB client

- Missed some files, the repository isn't required now that we
dont fetch the identity service directly
- Fixing review comments and removing the unused import
- Fixing the build :) Now that cloudSite has a 1-1 relationship with
cloudidentity, there shouldn't be a need to query the identity on its
own unless the site is missing Refactored some code to use site to
fetch identity as a composition
- Changing the catalogDb client to use
HttpComponentsClientHttpRequestFactory instead of
SimpleClientHttpRequestFactory, the later would lead to a IOException
(FileNotFoundException specifically) whenever a 40X type of a response
code is returned from the spring rest endpoint. Adding some tests for
the catalogDbClient which earlier got missed in handover. Also fixing
another issue with cloudSite repository which was not fetching data
correctly. Fixed a repository which was added initially but later the
same went missing (Not sure how)

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

Renamed migration from version 4.2 to 4.3

renamed migration from 4.2 to 4.3 to avoid collision

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

write update status of BBs progress

added in the missing bpmn file change
add implementation to update status of BBs progress

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

get service type and customer id from uri keys

Added logic to get service type and customer id from uri keys for sdnc request.

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

fixed filter of configuration flow calls

fix filtering of configuration flow calls

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

update field in service decomp object

removed duplicate json test file and updated existing
update ServiceDecompoition serviceNetworks field

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

fix sonar critcal issues in R_CloudConfigMigration

added specific excpetions instead of generic ones

Issue-ID: SO-886
Change-Id: Ib0f2e655a6e58b8dc142233679a53c15b373f117
Signed-off-by: Sandeep J <sandeejh@in.ibm.com>

fixed sonar issue in CatalogQueryExceptionCommon

fixed sonar issue: removed unused import

Issue-ID: SO-883
Change-Id: Ie9117601c7d64974067b5ab81ccb8a58d4a2fdc5
Signed-off-by: Sandeep J <sandeejh@in.ibm.com>

Merge "fixed sonar issues in MsoNetworkAdapterAsyncImpl"

Merge "resolved sonar issue in QueryServiceMacroHolder"

fixed sonar issues in MsoNetworkAdapterAsyncImpl

removed useless assignment of local variable "sMap" at lines
646,664,684. And, also removed keyset with entryset at line 650

Issue-ID: SO-882
Change-Id: I31a1a0c2c7aa4d65466125bda6948765326994eb
Signed-off-by: Sandeep J <sandeejh@in.ibm.com>

resolved sonar issue in QueryServiceMacroHolder

removed the commented lines of code as per issue raised by SONAR

Issue-ID: SO-881
Change-Id: Ic25a7418df06c7b7da2c60bec9dc4adbc97d902b
Signed-off-by: Sandeep J <sandeejh@in.ibm.com>

added unit test for class

Adding unit tests for the infraactiverepositorycustomcontroller
and fixing an issue found while adding the test

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

Merge "removed problematic swagger dependency"

Merge "updated ServiceInstance POJO"

Prod fixes August 18th

fixed compilation issues and all unit tests now pass
fix condition that checks for vnf in list to set homing flags
fix resource name variable being null
Skip volume group search for alacarte VF Module creation, since that group is expected to be specified on the input
Remove temporary change to JUnit for creation of volume group.
Add mapping of isBaseVfModule setting to AAIObjectMapper for vfModule.
adding a unit test for hte issue which was fixed
Explicitly return heatStackId variable from VnfAdapter subprocess
Fixing an issue with the workflow processor/asynresource which may lead to
apihandler leaving the active_Request table record in_progress while the bpmn quits the processing due to the error.

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

removed problematic swagger dependency

removed swagger dependency which was causing issues

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

Merge "update install and configure document"

update install and configure document

Issue-ID: SO-675

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

Merge "sonar try-with-resources"

Merge "openstack_adapter container fails to start"

openstack_adapter container fails to start

The reported problem occurs because the openstack-adapters app
is attempting to do a migration on top of a migration already
performed by the catalog-db app.

I'm adding these options to flyway in openstack-adapters:
  baseline-on-migrate: true
  validate-on-migrate: false

Another issue is that the java-based CloudConfig migration class
was looking for CloudConfig data only in the application.yaml
file in the classpath.  It was not looking for CloudConfig data
in the override file.  I've changed this logic to look in the
override file first and then in application.yaml.  Ideally, these
sources would be merged, but I don't see a reasonable way to do it.

Change-Id: I7ba07c1f8f00b4c628e825393ee31502950fe592
Issue-ID: SO-868
Signed-off-by: Rob Daugherty <rd472p@att.com>

updated ServiceInstance POJO

added new fields to pojo for groovy compatibility

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

created generic HttpLookupUri class

created generic HttpLookupUri abstract class

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

prod bug fixes august 12th

Set generalBuildingBlock variable on execution to
pass heatStackId updates to the calling process.

Pass CloudParams to SDNC on Create request

Pass cloudParams to VNF Adapter on VF Module
creation too.

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

fixed sonar issue in WebSecurityConfigImpl.java

fixed the sonar issue in the file at line 39

Issue-ID: SO-874
Change-Id: Ic187fa42e81a503e99ff71ab9ab971c21aa82cd3
Signed-off-by: Sandeep J <sandeejh@in.ibm.com>

Merge "Fix the structure in the doc for bpmn"

Fix the structure in the doc for bpmn

Issue-ID: SO-675

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

Merge "Update the structure of the documentation"

Update the structure of the documentation

Issue-ID: SO-675

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

Add device resource bpmn process for ccvpn uc

Bug fix for ccvpn uc

Change-Id: Idaa55084f5ecb0dd3636c232cebc14fa70000013
Issue-ID: SO-683
Signed-off-by: Yulian Han <elaine.hanyulian@huawei.com>

sonar try-with-resources

Issue-ID: SO-871
Change-Id: I151ea890755c0a9dd9e7f5df1ed807a660fb0c2f
Signed-off-by: Arindam Mondal <arind.mondal@samsung.com>

Docker naming convention

SO is not following the naming convention.  Docker image names
should have an "onap" prefix.  The image names are changed as
follows:

so/catalog-db-adapter --> onap/so/catalog-db-adapter
so/request-db-adapter --> onap/so/request-db-adapter
so/sdnc-adapter --> onap/so/sdnc-adapter
so/openstack-adapter --> onap/so/openstack-adapter
so/vfc-adapter --> onap/so/vfc-adapter
so/asdc-controller --> onap/so/asdc-controller
so/bpmn-infra --> onap/so/bpmn-infra
so/api-handler-infra --> onap/so/api-handler-infra

A corresponding change is needed in INT's docker-manifest.csv.

Change-Id: I024e42607d5fc57a712ad67709f241ee8720a670
Issue-ID: SO-870
Signed-off-by: Rob Daugherty <rd472p@att.com>

Merge "Site service process with location"

Fix VfModuleCustomizationToVduMapperTest

- Moved VfModuleCustomizationToVduMapperTest to mso-openstack-adapters
module, where the tested implementation actually is.
- Fixed and reimplemented tests

Change-Id: If5b4a29be22b7cbb59642eff78e471f294a512a7
Issue-ID: SO-865
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>

Site service process with location

Delete real existing resources

Change-Id: Idaa55084f5ecb0dd3636c232cebc14fa70000006
Issue-ID: SO-683
Signed-off-by: Yulian Han <elaine.hanyulian@huawei.com>

Merge "Cloud Config Updates"

Cloud Config Updates

Update Cloud Config to be writable over REST API.  This allows easier
environment setup, so sql scripts are not required

Issue-ID: SO-862
Change-Id: I70931f09ff2600a15094199f46fc8d9ac36403f4

Change-Id: I70931f09ff2600a15094199f46fc8d9ac36403f4
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "Added vfModule scale out endpoint and validation"

Merge "Fix Dependency"

Added vfModule scale out endpoint and validation

Issue-ID: SO-764
Change-Id: I6fc983ed6bc9d9c410406db1f2355c27e201b4f4
Signed-off-by: Brittany Plummer (bp896r) <bp896r@att.com>

Fix Dependency

Fix missing dependency in bpmn infra

Issue-ID: SO-862

Change-Id: I72ceb0050e38610262192df980330341bc91e89a
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "Fix for NullPointerException in Pnf"

Merge "ParameterizedTypeReference now passed in"

Merge "Update CatalogDB Logging"

Merge "Sonar issue Major"

Update CatalogDB Logging

Update CatalogDB Logging to remove MSO Logger, remove useless log lines,
clean up minor sonar issues while working in the file

Issue-ID: SO-847

Change-Id: Icab5d8cf2885d2193bb55f495b3230dde0f5af54

Change-Id: I2d2b2fa919074c439b6c9039aa7ed5e45a792781
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Cloud config database table support

added cloud config database entities
added cloud config repository
converted existing cloud config class
converted all code interacting with previous cloud config object
created migration to automatically load cloud config properties from
application.yaml

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

Merge "made required pom changes for cxf"

Merge "Casablana bugfixes"

Fix for NullPointerException in Pnf

- Fix: Since lambda expressions are evaluated lazily,
Camunda flow results in an NPE being thrown if RuntimeService
is not acquired as part of invoking thread.
- Modified integration test CreateAndActivatePnfResourceTest
to verify the fix.

Change-Id: I683a8400c81b2edd9d55f2a1d4121a4fdbdf820c
Issue-ID: SO-722
Signed-off-by: Michal Kabaj <michal.kabaj@nokia.com>

Sonar issue Major

Issue-ID: SO-852
Change-Id: Ia0d58c52884abdd1f752dc6671d315b8516812e6
Signed-off-by: Arindam Mondal <arind.mondal@samsung.com>

Merge "added edge label support to A&AI client"

made required pom changes for cxf

cxf 3.2.5 also requires us to move to javax.ws.rs-api 2.1
javax.ws.rs-api 2.1 is fully backwards compatible with 2.0
it will not be a problem with our other dependencies

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

Handle special aai case better

corrected error handling in aai client for service instances
updated the exception logic for si's that are not found
adhere to mapNotFoundToEmpty in special case

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

added edge label support to A&AI client

you can now specify an edge label when connecting two objects

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

Casablana bugfixes

updated mock in unassignnetworkbbtest instead of base
added mock to extract pojos method in unassignnetwork
Return gBuildingBlockExecution from calls to VnfAdapter subprocess
Correct JUnit data to expect modelInfoVfModule
Enable Spring Security for logging of unauthorized/bad credentials
errors
Fix handling of a case of no VF Modules in a VNF resource.
fix populating lookupKeyMap with resource ids
removed unnecessary test resources

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

Merge "Fix Error Scenario"

ParameterizedTypeReference now passed in

An argument must be added other wise we cannot return anything other
than a linkedhashmap because of type erasure.

https://stackoverflow.com/questions/21987295/using-spring-resttemplate-in-generic-method-with-generic-parameter

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

Merge "Improve Logging"

Merge "Fix Session Time"

Merge "GR-API flow, VF Module, SDNC GR API Fixes"

Fix Error Scenario

If Camunda returns a non-auth error, or invalid JSON. APIH was not
logging the error or properly updating the request database record

Issue-ID: SO-818
Change-Id: I8b164002132ca6c25f95e1fa2b194e6cb4c3be03

Change-Id: If6e06de0fb8a01a00ef06ade915dd7c13623047b
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Improve Logging

Add Stack trace to be logged, currently only a generic error message is
logged

Issue-ID: SO-846
Change-Id: I8479247227cf08de25bece5077003ee83c08472d

Change-Id: I8479247227cf08de25bece5077003ee83c08472d
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Fix Session Time

Fix session times taking too long to create, which is causing fallout in
heat environments

Issue-ID: SO-845
Change-Id: I6655507bd062ed42c0f2062edd4edfb580dca194

Change-Id: I6655507bd062ed42c0f2062edd4edfb580dca194
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Downgrade CXF

The Upgrade of CXF broke the A&AI client, downgrading it for now to
unblock integration testing.  We will investigate in more detail around
the upgrade of the version.

Issue-ID: SO-823

Change-Id: I15d761737b8c62db6de74ce5e5ee9262016120e0

Change-Id: I15d761737b8c62db6de74ce5e5ee9262016120e0
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

GR-API flow, VF Module, SDNC GR API Fixes

Fixes the new GR-API flow
Various fixes in the create VF Module area
Fixes issues parsing the SDNC GR API response
Fixes new resource type in ASDC controller
Fixes AppC mismatch in JSON marshaling

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

Merge "Groovy Compile"

Groovy Compile

Groovy was not compiling as the maven plugin was off.  I had to move
some files around, and add some import statements to make it compile
again.  I also had to rename some packages to onap.so. A few files were
deleted that did not compile but were not in use.

Issue-ID: SO-823

Change-Id: I99d04b4f25709c86d397d72700c237d81362d8b3
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Fix Logging

Add more logging information for Java.Lang Exceptions

Issue-ID: SO-822

Change-Id: I5eb25adc087c9fb495829af2c9d0d1be7374f4a3

Change-Id: I5eb25adc087c9fb495829af2c9d0d1be7374f4a3
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "Fix Ref Data"

Merge "Refactored references to AaiUtils"

Fix Ref Data

Fix recipe reference data, add initial data setup script

Issue-ID: SO-812

Change-Id: I9cbc45f7788fe7c5a74e15d45a9b82a55ddfe299
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Remove Property

Remove BPEL properties that are no longer used

Issue-ID: SO-817
Change-Id: Ie654562dabaddfcff68de3dfddeb85eed478b0a5

Change-Id: I6a418c73608d08f4cf12833f893e671ffa77beb0
Signed-off-by: Smokowski, Steve (ss835w) <ss835w@us.att.com>

Merge "Fix Invalid BPMN"