Policy client fixes

Some fixes on policy, new exception, new params for BRMS.

Change-Id: Ia30bf68ec492c448249c1634f3790712a27631e4
Issue-ID: CLAMP-62
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "Fix the bad request issue for operational policy"

Fix the bad request issue for operational policy

Change-Id: Ie3275adf6ed549fe4a8488f84ca3ff5bd429069b
Signed-off-by: xg353y <xg353y@intl.att.com>
ISSUE-Id:CLAMP-62

Merge "Update TCA config json send to Policy"

Update TCA config json send to Policy

Update the format of the TCA configuration send to Policy

Issue-ID: CLAMP-62
Change-Id: I53cbfcb1f8f8772856b645e8aec0db2a8a94d15e
Signed-off-by: xg353y <xg353y@intl.att.com>

Merge "Add architecture and release notes structure"

Merge "Remove datatable lib"

Add architecture and release notes structure

Add release-notes template
Move Clamp in ONAP Architecture description a separate file
Add seperator lines on list items

Issue-ID: DOC-110
Change-Id: Ibe803f4de3d4641b54ff4c2df54c5c3af62e4afc
Signed-off-by: Rich Bennett <rb2745@att.com>

Remove datatable lib

Remove datatable lib and its associated designer UI code

Change-Id: I328bad242d478a43e3b77716491aa921ade37a64
Issue-ID: CLAMP-63
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Corrections to pass doc8 and minor corrections

Change-Id: I4ee1613a446ca02b9dc94506f5cf455dd161beb0
Issue-ID: DOC-94
Signed-off-by: Eric Debeau <eric.debeau@orange.com>

Add config folder to be able to change config

Change-Id: Icd0fe1a261370f21ee391aaf3294e84b2b33deb7
Signed-off-by: eh552t <eh552t@intl.att.com>
Issue-ID: CLAMP-61

Add Policy dependency

Add policy deppendency to solve NoClassFoundException issue.

Issue-ID:CLAMP-61
Change-Id: Iaf20852aff0313d3f503773468222113888940e9
Signed-off-by: xg353y <xg353y@intl.att.com>

Merge "Update the TCA blueprint format"

Merge "Inverted patch commited"

Update the TCA blueprint format

Change-Id: Ib5b6fe1b46a509bea6c40ebc8ec9058990e5f64d
Signed-off-by: xg353y <xg353y@intl.att.com>
Issue-ID: CLAMP-59

Documentation update

Change-Id: I74f093304ebfc5f0c77bd880eefd6cfc63c654f7
Signed-off-by: martial <gn422w@intl.att.com>
Issue-ID: CLAMP-16

Inverted patch commited

The previous patch on this file has been commited
with a not up to date codebase and previously made
changes were overwritten.

Change-Id: I0731a4a228e8bbdff46c09dee757d076eebcc799
Issue-ID: CLAMP-54
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Fix the bad test

This test fails when pnfType is set in the yaml

Change-Id: I7ec3996ca0bbf40334063011d9d745155b1ba6e8
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove useless method

This method was private and not used so it has been removed

Change-Id: I8d4799dfd499fcc6b046ff59b449d79abd28dc40
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Restoring try catch for policy request

This exception log was not useful so it has been removed and replaced
by an easier log to understand

Change-Id: Ie25426d3b8b44b55a3fd2dd3807620d98fad5bbf
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Fixing typo in parameter names

Fix formated to formatted parameter names

Change-Id: I01e78b99eabea8b098d9c9dfca9f77116e0dd62a
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Increase the test portability

This test was testing the class name provided in the yaml,
if it changes the test is dead. So it's more generic now

Change-Id: I72ba7ac087da137b0e43628c99b5d92a1696078a
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove useless code

This code was always returning an empty map so it has been removed

Change-Id: I7691d2a52b2ea50d4cb8586b2e66b6ecfdd6fe13
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Set Cache Timeout to 30

Reduce the expiration timeout of the sdc model cache to 30

Change-Id: If44852f807bd65e5494a95d64e90985967e55eef
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove useless method

One method one private and not used anymore in the code so it
has been removed

Change-Id: Id6754eb446a781b691184ace07a68ce26c8ed684
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Update Holmes related feature

No blueprint need to be send to SDC; Add configuration policy name in
holmes UI; Update the config body sent to Policy.

Change-Id: I4bf5b0c32d64f4acf28ac6f24128e463e28e5779
Signed-off-by: xg353y <xg353y@intl.att.com>
Issue-ID: CLAMP-27

Merge "Remove useless plugin"

Remove useless plugin

Remove the gpg plugin in main pom as not used at all

Change-Id: Ic86cb285012da0cdbceca78c4750a5365e9fc6a0
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "Removed redundant code"

Merge "Fixed major sonar issues"

Merge "Removal of useless test"

Merge "Change SDC address to openecomp"

Merge "Fix the Checkstyle issues"

Removal of useless test

This file is deleted as not used at all

Change-Id: I257c7410d5864890ed4d0258baa90f3061e294b6
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Change SDC address to openecomp

Change SDC address to openecomp one as required for integration tests

Change-Id: Ib8661f0755df6af5b07fc0f693ee5e6f2dc61d11
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "TCA Window initialization not possible anymore"

Fix the Checkstyle issues

Fix all checkstyle issues in all unit tests + rename all IT
and remove useless tests

Change-Id: I8d1244c26d4d2ca490ba8ffcf2236f484c783394
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

TCA Window initialization not possible anymore

Change-Id: I56750cebf5345f89eb11158c0cbd6325d83a283e
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Merge "Set localhost in config"

Set localhost in config

Set to localhost in clds config file

Change-Id: Ic8595e0fcfd7a4481464b1facc8bcb4cbeac92fb
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "Fix spelling mistakes"

Merge "Change the model to fix TCA"

Merge "new method for policy client"

Merge "Fix the a log in Model Properties"

Merge "Removal of useless tests"

Merge "Add an exception for new class TcaRequestFormatter"

Merge "New Files added for future unit tests"

Fix spelling mistakes

Spelling mistakes fixed with formater word

Change-Id: I2e5e44e3de6ac1c8f2a5a6aee0ec7ec713cf139f
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Change the model to fix TCA

Fix TCA model and JSON model for TCA + rework the JUNIT

Change-Id: I35099633e89f9f95f1957afeffa915feda4045f3
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

new method for policy client

A new method to send JSON in other type to policy

Change-Id: I24b2d36464fa3c9786007ea3772c6a2c4b7c7b66
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Fix the a log in Model Properties

One of the log did not contain the stacktrace for Model properties

Change-Id: I5f269bb1744343ef9646cf818babfd77ea237b76
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Removal of useless tests

These IT are not useful anymore as covered by others test classes

Change-Id: I6691c944f3ec98d4685229ce4758ba79c6bfa0ae
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Add an exception for new class TcaRequestFormatter

This new exception will be used in a new class that will manage TCA

Change-Id: I4ae2c33929c81a141b59dabe6e2d1db371cfc188
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

New Files added for future unit tests

New files that will uses by new junit on TCA code

Change-Id: Icfd991da3210cc2a9534815ec5d1671fcd72a0d8
Issue-ID: CLAMP-59
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Fix fields corruption on save

Change-Id: I02c8fe662698ce943ce3eed8cd1e0b78b1311cc8
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Fix fields not restored on tca open

Change-Id: I18bb36d7df0aa982f941c9153a86ab17a48f4295
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Remove debugger + fix missing info

Change-Id: Ic6e016eef81fbad70d3ac61c435cd47feb23cf3b
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Fixed major sonar issues

Fix major sonar issues in clamp module
https://sonar.onap.org/component_issues?id=org.onap.clamp%3Aclds#resolved=false|severities=MAJOR|rules=squid%3AS1132%2Csquid%3AS1858

"get" returns a string, there's no need to call "toString()".

Issue-Id:CLAMP-54
Change-Id: I36cadab8e60122e1b4e244768d345cf04568d134
Signed-off-by: shashikanth.vh <shashikanth.vh@huawei.com>

More flexible styling + fix cannot save tca

Change-Id: Ice937ce4a56218981976e6ad777b3e4e71963680
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Remove tab header

Change-Id: I6c0fd3e46b3e7062521b0274cadae620719f0d4b
Issue-ID: CLAMP-58
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Removed redundant code

Fix major sonar issues in clamp module
https://sonar.onap.org/component_issues?id=org.onap.clamp%3Aclds#resolved=false|severities=MAJOR|rules=squid%3AS1132%2Csquid%3AS1858

Move the "csv" string literal on the left side of this string comparison.

Issue-Id:CLAMP-54
Change-Id: If24d56fd5f539718e12241d4c1d4d83d8a079c0e
Signed-off-by: shashikanth.vh <shashikanth.vh@huawei.com>

Code Improvement

Change-Id: I703aabf234cb656b1faea3b5c0776d034d8ec807
Signed-off-by: xg353y <xg353y@intl.att.com>
Issue-ID: CLAMP-54

Restore version to SNAPSHOT

The version has been changed to 1.1.0 and should be 1.1.0-SNAPSHOT

Change-Id: I268f050c381a8ae884f0c5615e78288be0390329
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Fix bug at runtime

There was a missing class due to policy dependency changed

Change-Id: Id8d85a8925c1ac2164528333672a552cb55ca5ca
Issue-ID: CLAMP-48
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Rework the indentation

Remove the tabs and replace by Spaces

Change-Id: Id9e1e5e434789ec7ebf5ba0d3e25a8ccf9765852
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove the Policy Dep Snapshot

The policy dependencies have been changed to refer to
non snapshot packages. Some java packages have been updated too.

Change-Id: Id70a0247b859368c642a73fa59bbda93bf9904ae
Issue-ID: CLAMP-55
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove HashMap definition

Change HashMap to Map for variable declaration

Change-Id: Ia352ab40c087e26ed43a2e69ad5ef60b6bb68a87
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Add the target resource ID

Add target resource ID in operational policy code as requested.
It changes the UI, the Policy code, the model and the tests.

Change-Id: Ie72badde55e7f8b5c2e5234099a09b95958205cf
Issue-ID: CLAMP-55
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove useless code in test

Some variables were created and not used, this has been removed

Change-Id: I40f28f4a991989b3759b2adf8842cebcfd9ac207
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove the mariadb pull if no test

The build was pulling mariadb even when the flag skipITs was set

Change-Id: I447af0de537bd55666ce93958451b160c72c6234
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Rename a test class

Rename a test class by using the name of the class tested

Change-Id: Ic2dc993ec874872b3acc9ef95fdfdb540ff61742
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Remove Camunda auto conf for Junit

Camunda autoconf classes removed for SPring junit IT tests

Change-Id: Ia71574520bd88144efaf95c3a22fcb74d91e6fb9
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Add different log in AbstractModelElement

Log changed when the Json attribute value is null or empty

Change-Id: Ie0321efbbcc7f9c637f1bd8ea44001e5036987d3
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Removal of useless files & bugfix

Removal of DCAE useless files, and bugfixes SDC calls(+CldsDAO bug), and
insertion of the search engine for Model Properties window in UI

Change-Id: Ia68117bb917807eaf8e8e317f391f90b6d948135
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Reformat the JSON and fix checkstyle issue

All Json used in the tests have been reformatted.
Also some checkstyle bugs have been fixed

Change-Id: I48373d9d53d39244c62b9b88b1cf62c15c766cb3
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Add new tests to CLAMP

New tests added on ModelProperties, CldsModel and ITs (CldsService
tests)

Change-Id: I75a29c11ec66e0eef7c80f76cf90c30b2fc29972
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "Add new tests for DAO"

Merge "Some rework around DAO classes"

Add new tests for DAO

New test to validate some methods of the DAO and attempt to disable
Camunda in IT when it was not required to have it

Change-Id: I231230a7a4818f4bfd7887319a321625f66ffcae
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Some rework around DAO classes

Remove the useless return parameter in setModel, remove the
runtimeService in cldsTemplateService

Change-Id: Ia6d82cc3167f2e82b0da9cdb14d93c64ccee19c4
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Wait for database with mariadb connector

Need to add sequential in connection url to be
able to retry

Change-Id: Ib5343de3c1ecad317ae1628590b1766a31ed1f15
Signed-off-by: eh552t <eh552t@intl.att.com>
Issue-ID: CLAMP-42

Wait for database with mariadb connector

With the change from Mysql to MariaDB connector
maxReconnect parameters doesn't work anymore.
We then need to use retriesAllDown instead but
this is doing retries as fast as possible with
no wait between tries so need an arbitrary high
value. With maxReconnect it was waiting a time
exponentially growing so a low value was enough.

Change-Id: I7d03baa69bbee46d82a403f573d9f492f9720a4c
Signed-off-by: eh552t <eh552t@intl.att.com>
Issue-ID: CLAMP-42

Wait for database with mariadb connector

With the change from Mysql to MariaDB connector
maxReconnect parameter doesn't work anymore.
We then need to use retriesAllDown instead but
this is doing retries as fast as possible with
no wait between tries so need an arbitrary high
value. With maxReconnect it was waiting a time
exponentially growing so a low value was enough.

Change-Id: Id1429759aa4bdcfc9af1934fd6f8b848ed879e91
Signed-off-by: eh552t <eh552t@intl.att.com>
Issue-ID: CLAMP-42

Merge "Restore the history log in camunda"

Merge "Add test to Holmes"

Merge "Clean unnecessary files using camunda"

Merge "Rework the datasource access"

Merge "Rework the comment of the constructor"

Improve doc

Update to reflect readme.md details

Change-Id: If8ee14add971ededa2725424e161c46f782378d1
Issue-ID: CLAMP-15
Signed-off-by: ChrisC <cc697w@intl.att.com>

Policy and StringMatch GUI need refresh

A reload of GUI for Policy and StringMatch was required to see changes
done in some select fields. This patch add automatic updates of fields
that needs it.

Change-Id: Ida7ad705218b1881868dcdec936c809b19df69bd
Issue-Id: CLAMP-53
Signed-off-by: Anaël Closson <ac2550@intl.att.com>

Restore the history log in camunda

Restore the history log in camunda as it has been disabled but the DB
SQL files set that already

Change-Id: Iba24a4cefdfb533046eed92e7ce1424a06d1c0fb
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Add test to Holmes

New test for holmes and refactor of the application.java class according
to Sonar reports

Change-Id: I2117b403c18e2cb2cd902018db951025a1db9bae
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Clean unnecessary files using camunda

The camunda examples have been removed as not used by anyone

Change-Id: Ib659ea0d68aff613620fce6bf25d4d4a1e1acac4
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Rework the datasource access

Make a call to Spring datasource instead calling the method directly

Change-Id: If9464f8591cd140079d09dc33c83a8b4380bf5ae
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Rework the comment of the constructor

Refactor the parameters name of the constructor

Change-Id: Ie4314d0384a875da1e8bf3abc1092cd826347929
Issue-ID: CLAMP-54
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Merge "Change to mariadb connector"

Merge "Change mysql to mariadb connector"

Change to mariadb connector

The tests application.properties had to be changed as well

Change-Id: Ie2a4ca5e136aef3f2ae7ba55cbee705661374273
Issue-Id: CLAMP-52
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Change mysql to mariadb connector

Replace the mysql connector used by Spring to mariadb
+ remove the Mysql connector from the clamp.jar

Change-Id: If7241e111d07ad98eba3b0fcb612b0f4711d683d
Issue-Id: CLAMP-52
Signed-off-by: Determe, Sebastien (sd378r) <sd378r@intl.att.com>

Fix: Policy GUI not responding on opening

A call to the backend method getSdcPropertiesByServiceUUIDForRefresh
is failing. This is due to invalid handling of Sdc messages introduced
with revision 13966ab288578be392c5d2c64d5a9d7b6f600327.

Change-Id: I1ade99f50a177d7db6042938c4f0734eba0071fd
Issue-Id: CLAMP-51
Signed-off-by: Anaël Closson <ac2550@intl.att.com>