Merge "Added new policy types for native PDP policies"

Allow wild cards in supported type filter

PAP was rejecting deployment of policies whose supported type
contained a wild card.  This was due to the fact that it was
querying the Pdp Groups using a filter on the supported type,
which did not take wild cards into consideration.  Updated the
filtering.

Change-Id: I50b3202a00ac85ff09a9861d8bbe1efb6dd49ae3
Issue-ID: POLICY-1636
Signed-off-by: Jim Hahn <jrh3@att.com>

Added new policy types for native PDP policies

Issue-ID: POLICY-2212
Change-Id: Ib29ad66994aa64a73f23bc36eebbc1bc03b2ac02
Signed-off-by: Chenfei Gao <cgao@research.att.com>

Fix minor checkstyle issuse in models A&AI

Some minor checkstyle issues in the policy models Custom Query support
in A&AI fixed.

Issue-ID: POLICY-2125
Change-Id: I520caa044966acbe28d3cd864b53fb398df2f385
Signed-off-by: liamfallon <liam.fallon@est.tech>

Support wild cards in "supported policy types"

Refactored PfConceptKey, extracting most of it into PfKeyImpl.
Added PfSearchableKey as a subclass of PfKeyImpl.

Change-Id: I524f4ce9208fc9ba09e77db4cc7dde5a21e1d7fc
Issue-ID: POLICY-2224
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Removing Named Query."

Removing Named Query.

Removing all the related  files and necessary modifications for removing named query. The code now only supports Custom Query

Issue-ID: POLICY-2125
Change-Id: Ifa5ccb74142aa424359c0dea85b90bbcf70147cb
Signed-off-by: pramod.jamkhedkar <pramod@research.att.com>

Remove duplicate dao object

Code was creating the DAO object twice - removed the extra line.
Note: this properly doesn't fix the race condition, but it should fix
a memory leak.

Change-Id: I5f5c6002d42f9bb0525d1b5013b1300e80264645
Issue-ID: POLICY-1832
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix sonar issue affecting in drools-applications

Drools-applications has a serializable class that contains AaiCqResponse.
Sonar complained that the latter class must also be serializable, so
made the change in policy-models to fix that.  In addition, the latter
class contained a list of Object.  To ensure that those are also
serializable, the list was changed to a list of Serializable instead.

The latter change can be reverted, if it causes compilation issues
elsewhere.  However, it seems safer to enforce serializability in the
list rather than risk having a later change inject a non-serializable
object into the list.

Change-Id: Ia741a1a96ec4efcc63451e9af31c19b4fe99dd7d
Issue-ID: POLICY-2225
Signed-off-by: Jim Hahn <jrh3@att.com>

Use lombok annotations for objects in models-interaction/models-impl module

Issue-ID: POLICY-1896
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: I425afe25876cea91908391418582ef584e6fe6fc

Cleanup up unused actor.test maven module

Issue-ID: POLICY-1894
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Ib662aa6a28beb40c2a1e6814b103e95ebf4ee2e1

Merge "PNF support changes in policy/models"

PNF support changes in policy/models

AAI Enrichment method for PNF target-type and unit tests.
Addition of a hashmap field to ControlLoopEvent class to hold event specific parameters.
Update CDS actor to include additional event parameters

Issue-ID: POLICY-1187
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Ie0ceb320943531de6e6bc8675844b29a358dfb7e

Add SDNC Naming Policy Type and example policy

Will be used to create naming application for XACML PDP.

Issue-ID: POLICY-1740
Change-Id: I996537da16bdb386f4bcddd0008055b8ffac417e
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Add common Jersey version

Add a common version of Jersey for all Policy Framework components.

Issue-ID: POLICY-2209
Change-Id: Ic1d2ea808bb41156f776d18b875e44de22fbf06b
Signed-off-by: liamfallon <liam.fallon@est.tech>

Merge "Change field names in policy notifications to match wiki"

Fix Checkstyle issues in policy-models

Some minor checkstyle issues were introduced into policy-models by
recent reviews.

Issue-ID: POLICY-2088
Change-Id: I84af19926432081d72e2054af788cdc35850a99c
Signed-off-by: liamfallon <liam.fallon@est.tech>

Change field names in policy notifications to match wiki

Field names looked like "policyType.name".  Changed them to look like
"policy-type", instead, to match the wiki for the notifications to be
sent to DCAE.

Change-Id: Ib3cef1712fd12a44db8acdcff2245d800b21137a
Issue-ID: POLICY-2190
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Changes in preparation for the m2 model (part 2)"

Changes in preparation for the m2 model (part 2)

This change was missed earlier: convert 'ResourceType' to a 'String',
so application-specific types can be added.

Issue-ID: POLICY-1948
Change-Id: Ie8a35e7fb0746d8c8e3a6db11c09ca31dc4763cc
Signed-off-by: Straubs, Ralph (rs8887) <rs8887@att.com>

Merge "Refactor Appc LCM Model"

Refactor Appc LCM Model

Added model classes for APPC LCM Input and APPC LCM Output
and removed adding the input/output tag to the JSON via
the APPC LCM custom serializer.

Issue-ID: POLICY-2043
Signed-off-by: Daniel Cruz <dc443y@att.com>
Change-Id: I54e1b5f589a4b29357cf1a45dbdf2173610870ce

Merge "Changes in preparation for the m2 model"

Changes in preparation for the m2 model

1) Replace enum 'ControlLoopTargetType' with 'String' -- symbols such as
   'ControlLoopTargetType.VM' still work, but now expand into a String.
   This gives the ability to add new application-specific types.

2) Move 'ControlLoopEvent.payload' to 'VirtualControlLoopEvent'. This
   symbol isn't common to all 'ControlLoopEvent' types, and this
   definition collides with a 'payload' defined in our application.

3) Add 'RestManager.patch(...)', which performs a REST patch.

4) Add lombok getters and setters

5) Remove trailing spaces

6) Fix order of arguments in 'assertEquals' in 'ControlLoopTargetTypeTest'

7) Update Junit tests in 'RestTest' to include 'RestManager.patch(...)'

Issue-ID: POLICY-1948
Signed-off-by: Straubs, Ralph (rs8887) <rs8887@att.com>
Change-Id: I08e04ea3cbcf368c760b630bcfe23a4370cf94dc

Changed the serialized name of policy notifications

Changed the serialized name of the policy identifier from "policy"
to "policy-id".

Change-Id: I7188b65428728b65fef38c70e9fae99202ab0eea
Issue-ID: POLICY-1841
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Flatten the CDS grpc request payload"

Flatten the CDS grpc request payload

Before:
{
    "vfw-modify-config-request":{
        "resolution-key":"1234567890",
        "aai-properties":{
            "service-instance.service-instance-id":"1234",
            "generic-vnf.vnf-id":"5678"
        },
        "policy-payload":{
            "data":"{\"active-streams\":\"5\"}"
        }
    }
}

After:
{
    "vfw-modify-config-request":{
        "resolution-key":"1234567890",
        "vfw-modify-config-properties":{
            "service-instance.service-instance-id":"1234",
            "generic-vnf.vnf-id":"5678",
            "data":"{\"active-streams\":\"5\"}"
        }
    }
}

Issue-ID: POLICY-2089
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: I139a7ca3e1e2d0cd50bbd86d0cdf499f215be74f

Update optimization policies to have version

These should be version 1.0.0. Add in a no version test.

Issue-ID: POLICY-2066
Change-Id: Ie113c02c37e95b86dbb31247bf7a9fda09f724b1
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Add PolicyNotification.isEmpty() method"

Add PolicyNotification.isEmpty() method

Change-Id: I5cf5f21b968975018bd1f18daa1653289b837d2e
Issue-ID: POLICY-1841
Signed-off-by: Jim Hahn <jrh3@att.com>

Remove the CDS blueprint name and version from its payload.

This information is redundant since it is included in the request header.
Found during integration testing

Issue-ID: POLICY-2089
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Icdb4753ccae687f85318f5efeed09d5e13a6853b

Merge "Remove actor and recipe checks from ControlLoopCompiler.java"

Remove actor and recipe checks from ControlLoopCompiler.java

Remove actor and recipe checks from ControlLoopCompiler.java
and adjusted associated JUNIT tests for the changes that were
implented. Removed some redundant variables, replaced null check
in actor/recipe with blank string, which checks for null as
well as empty/blank strings. drools-applications still builds
correctly.

Issue-ID: POLICY-2128
Change-Id: I83d9e3b1152d87c9c8d2c90586300aebdd53b8e2
Signed-off-by: dg5762 <dg5762@att.com>

Add PAP Policy Notification classes

Added classes that PAP will use to publish to DMaaP to indicate policy
status changes (i.e., added/removed from PDPs).
Modified serialized name of the policy identified to match other uses
(e.g., PAP REST API calls).
Added policy type, per review comments.

Change-Id: I66131fa9ae0b63081f239e7f402b7fea65c434a5
Issue-ID: POLICY-1841
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix remaining fixable Sonar issues on policy models

This patch fixes the remaining  easy to fix Sonar issues on policy
models. The remaining issues are
- methods that do not follow Java naming conventions and fixing those
  will probably break policies that use those methods
- TODO statements to do with functionality we didn't get done in Dublin
  and will address in Frankfurt

Issue-ID: POLICY-2165
Change-Id: I7f604bc2335388bdc0dd9286ae17a5ab76517665
Signed-off-by: liamfallon <liam.fallon@est.tech>

Clean up minor things in policy-models

Cleam up of some minor eclipse, checkstyle, and Sonar issues in
policy-models

Issue-ID: POLICY-2165
Change-Id: Id5682fd60e1a6cebbf716d9c258453aa53ca2482
Signed-off-by: liamfallon <liam.fallon@est.tech>

Merge "Clean up SONAR issues in CDS actor"

Fix dmaap simulator junit

Modified the code to leave the topics open through-out the test cases,
and simply clear the queues between each test case.  Hopefully that
will fix the EndToEndTest junit in the dmaap simulator.

Change-Id: I3658df4dbc70cd47fec1cfb84c7a2365133105ff
Issue-ID: POLICY-2144
Signed-off-by: Jim Hahn <jrh3@att.com>

Clean up SONAR issues in CDS actor

Issue-ID: POLICY-2165
Change-Id: I2ccfb32193c85958d525af19a7744a74c97a3485
Signed-off-by: liamfallon <liam.fallon@est.tech>

Merge "Added new policy example to support junit test for new simplified API"

Added new policy example to support junit test for new simplified API

Issue-ID: POLICY-1991
Change-Id: Ib52bffe7895dc0625c77da7aa954bb71ed16187d
Signed-off-by: Chenfei Gao <cgao@research.att.com>

Added ModelType to SO Req.

Added ModelType to SO Request and set it to vfModule

Issue-ID: POLICY-2156
Change-Id: Icfd824ff1f521ec32a711506d01283566d2a7ed9
Signed-off-by: pramod.jamkhedkar <pramod@research.att.com>

Correct the CDS basic auth header in  the request

Issue-ID: POLICY-2088
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Ibcdfdbd1de66fcf193e52fcc14a1220c7520f140

Merge "Flesh out DMaaP simulator"

Flesh out DMaaP simulator

Fleshed out the dmaap simulator with additional media types.
Added more junit tests.
Added buildDmaapSim() to simulators Util.

Change-Id: I39acd3df8e8d0ded21228e56fa1ef919cafc3772
Issue-ID: POLICY-2144
Signed-off-by: Jim Hahn <jrh3@att.com>

Changes in model to integrate cds actor

* Create class for CDS response

Issue-ID: POLICY-2088
Change-Id: I813a310f7d5123fac4bb1c3880d108391096250f
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>

Update CDS grpc request generation in CDS Actor service provider

- CDS payload requires resolution-key (random UUID)
- Update the CDS request object to rename the payload child node key name from config-deploy-properties to policy-payload.
- Generate dynamic childnode key name from the policy recipe-name aka CDS action-name. action-name itself is never included in serialization as is.

Below is a sample:
{
    "commonHeader": {
        "subRequestId": "$subReqId",
        "requestId": "$reqId",
        "originatorId": "POLICY"
    },
    "actionIdentifiers": {
        "mode": "sync",
        "blueprintName": "$bpName",
        "blueprintVersion": "$bpVer",
        "actionName": "$recipeName"
    },
    "payload": {
        "$recipeName-request": {
            "resolution-key": "{{$guid}}",
            "aai-properties": {
                "$key": "$value",
                ...
            },
            "policy-payload": {
               "$clamp-payload"
            }
        }
    }
}

Issue-ID: POLICY-2120
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Ia7f0551f94755bbe73b48ae387b4c53842729a81

Change policy/common version in models

Junits in policy/api were breaking.  Tracked it down to the fact the
policy/models was still referencing an old version of policy/common.

Change-Id: I1aee40336382741d62cabbcac0f6268d2ceab263
Issue-ID: POLICY-2101
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix derivedFrom field

TOSCA does not have a version on derivedFrom fields on policies and
policy types. THerefore, we use the default value of 0.0.0 for versions
on these fields in the database.

When reading back out derivedFrom fields, we need to just take the name
of the field and not the ID of the field.

Issue-ID: POLICY-2075
Change-Id: I55e9c479047d6c1a9376a1713f4ddb2536476b29
Signed-off-by: liamfallon <liam.fallon@est.tech>

Upgrade io.netty : netty-codec-http2 and netty-common to 4.1.39.Final

These insecure dependencies are brought in by io.grpc:grpc-netty:1.17.1.
Override the io.netty dependencies and add exclusions where version is mandated.

Resulting dependencies in policy/models/models-interactions on master*
$ mvn dependency:tree -Dverbose| grep io.netty
[INFO] +- io.netty:netty-codec-http2:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-common:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-buffer:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-transport:jar:4.1.39.Final:compile
[INFO] |  |  \- io.netty:netty-resolver:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-codec:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-handler:jar:4.1.39.Final:compile
[INFO] |  \- io.netty:netty-codec-http:jar:4.1.39.Final:compile
[INFO] +- io.netty:netty-handler-proxy:jar:4.1.39.Final:compile
[INFO] |  \- io.netty:netty-codec-socks:jar:4.1.39.Final:compile
[INFO] |  +- io.netty:netty-codec-http2:jar:4.1.39.Final:provided
[INFO] |  |  +- io.netty:netty-common:jar:4.1.39.Final:provided
[INFO] |  |  +- io.netty:netty-buffer:jar:4.1.39.Final:provided
[INFO] |  |  +- io.netty:netty-transport:jar:4.1.39.Final:provided
[INFO] |  |  |  \- io.netty:netty-resolver:jar:4.1.39.Final:provided
[INFO] |  |  +- io.netty:netty-codec:jar:4.1.39.Final:provided
[INFO] |  |  +- io.netty:netty-handler:jar:4.1.39.Final:provided
[INFO] |  |  \- io.netty:netty-codec-http:jar:4.1.39.Final:provided
[INFO] |  +- io.netty:netty-handler-proxy:jar:4.1.39.Final:provided
[INFO] |  |  \- io.netty:netty-codec-socks:jar:4.1.39.Final:provided

Issue-ID: POLICY-2072
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Ib3706e1c67e33fe73ebbfc80a6b474021b4a2679

Bump versions for models master

Change-Id: I351d1a0ebd3c7cfa76d57297ce44ef6a6fa8c753
Issue-ID: POLICY-2045
Signed-off-by: Jim Hahn <jrh3@att.com>

Add release yaml file to models

Change-Id: I5053071a63c67770986181fe034d7371ac98ddca
Issue-ID: POLICY-2045
Signed-off-by: Jim Hahn <jrh3@att.com>

Add metadata to properties

Properties should support metadata - adding it in.
Added junit tests for the new metadata field.

Issue-ID: POLICY-2060
Change-Id: I2e1933ca4260fe5989f36a098108893a366f657a
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Signed-off-by: Jim Hahn <jrh3@att.com>

Use released versions in models pom

Updated models pom.xml to use released version of parent and
common artifacts.

Change-Id: I21bf9911d1dcd3ea9cc0f065dc80f65e153da885
Issue-ID: POLICY-2045
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "CDS Actor service-provider implemntation"

CDS Actor service-provider implemntation

https://wiki.onap.org/display/DW/CDS+actor+support+in+Policy

Issue-ID: POLICY-1763
Signed-off-by: Rashmi Pujar <rashmi.pujar@bell.ca>
Change-Id: Idcb6e9168b949745cc644e97ba77c479573a8bf5

Repair policy types in TOSCA service template

Changed policy types from a list of maps to a map.
Changed data types from a list of maps to a map.
Made all knock on changes to adapters and tests.

Updated Issue ID.

Added
onap.policies.controlloop.guard.coordination.FirstBlocksSecond.yaml
policy type that is required to be preloaded in policy-api.

Issue-ID: POLICY-2047
Change-Id: Ic8e809eab248fd7242f0e565fa2e3da75bcfbdeb
Signed-off-by: liamfallon <liam.fallon@est.tech>

Merge "Added VFModule count"

Added VFModule count

Added the function VFModuleCount to AaiCqResponse class. It counts the
number of vf modules. Also changed the same custom query response in
AaiSimulator.

Issue-ID: POLICY-1858
Signed-off-by: pramod.jamkhedkar <pramod@research.att.com>
Change-Id: If08de84f74d2bd9501b501b881cdd761089614dc

Replace copyTo methods with copy constructors

Deleted the copyTo() method from PfConcepts and replaced uses
with deep-copy constructors.  Also added mapMap() and makeCopy()
methods to PfUtils to facilitate.

Change-Id: Id6391bb806ef0dfab6c1089278bf2b514c7e303e
Issue-ID: POLICY-1600
Signed-off-by: Jim Hahn <jrh3@att.com>

update keystore

Issue-ID: POLICY-1988
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: I565d8ca294750b748ba2709eedab032cd5eb0936
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Review maven dependency for snakeyaml

Remove explicit version for snakeyaml maven dependency and inherit it
from integration parent artifact.

Upgrade snakeyaml to latest 1.24 version.

Issue-ID: POLICY-1603
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>
Change-Id: Id575085f220807a574758dd11fd6fef44fd2ffc8

replace hashCode function with compareTo

Issue-ID: POLICY-1600
Change-Id: If660cbe84424e492b6db3ab11ef97b37f046b3aa
Signed-off-by: Henry.Sun <henry.a.sun@est.tech>

Bump version in policy/models

Change-Id: Ie8761ccbd05edc80af51e0ae5e8c90a4dfaf7715
Issue-ID: POLICY-1965
Signed-off-by: Jim Hahn <jrh3@att.com>

Add release file for 2.1.2

Issue-ID: POLICY-1953
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: I09cb51c15330d76a746a2671133ef7145d776b28

Bump versions to 2.1.2

self-release process need to generate partial
releases with some jars missing.

Workaround is to bump versions an additional
patch number.

Issue-ID: POLICY-1953
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: I4b8fa38f082d8f06c7fe56a26f8c55db91cd1d9e
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Add self-release 2.1.1 file

Issue-ID: POLICY-1953
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: I8cc6a79507817690139db04ac8bd1808b90da9d1
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>

Point to policy/common 1.5.1 released version

Issue-ID: POLICY-1953
Signed-off-by: jhh <jorge.hernandez-herrero@att.com>
Change-Id: I4c2739dac8d9a0648b419635c6198c7dfa3fb720

Use new addTopic() method in models

Replaced ParameterUtils with the use of the new
addTopic(TopicParameterGroup) method.
Used separate addTopicXxx() methods per review comment.

Change-Id: I755b75f986a5c42f01bcad5a22472a46766d192d
Issue-ID: POLICY-1953
Signed-off-by: Jim Hahn <jrh3@att.com>

Removed all mention of www.example.org from tests
Replaced those tests with test using a local/temp webserver to exercise
the RestManager
Updated with Jim/Pam's suggestions.
Updated following SONAR changed to policy/common

Change-Id: Ia2551fbcafd2cd30d74a881237b003216e8caec7
Issue-ID: POLICY-1665
Signed-off-by: JohnKeeney <John.Keeney@est.tech>

Fix models due to sonar changes in common

Fixed breakages due to changes made in policy/common to satisfy
sonar.

Change-Id: I7de701e743664afb5963a454c59478d1a1c83e87
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Remove topic.properties and incorporate into overall config file for pdp-simulator

Change-Id: Iac882c7ffb28a80ef474d15a90307cd68b78d417
Issue-ID: POLICY-1744
Signed-off-by: a.sreekumar <ajith.sreekumar@est.tech>

Use ToStringTester from utils-test

Changing ToStringTester from utils to utils-test
Bump up common to 1.5.1-SNAPSHOT

Change-Id: I7427673c73bd1cc52d5f24a0542e89eb33a39ae8
Issue-ID: POLICY-1746
Signed-off-by: ramverma <ram.krishna.verma@est.tech>

Bump models to 2.1.1 SNAPSHOT

Released 2.1.0

Issue-ID: POLICY-1737
Change-Id: I2bc12dd4a9cdad7a867e3d710bb427493de6c7f4
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Upgrade to release common 1.5.0

Released

Issue-ID: POLICY-1737
Change-Id: I9e448a340643a5cc3a9fade226e6e40fcc63e1d9
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Upgrade to release 3.0.0 parent"

Upgrade to release 3.0.0 parent

Policy/parent released

Issue-ID: POLICY-1737
Change-Id: I34a4af77caf8e371089480b467833119d07ae937
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Adding pdps class for pdpgroup healthcheck

Adding Pdps class to hold the response for PdpGroup Heatlcheck REST API
of PAP.

Change-Id: I69b4979c8fdaefd3451fa0e9286dace2485cdc73
Issue-ID: POLICY-1770
Signed-off-by: ramverma <ram.krishna.verma@est.tech>

Merge "Change getCanonicalName to getName in models"

Change getCanonicalName to getName in models

Per javadocs, getName() should generally be used instead of
Class.getCanonicalName(). This change only applies to classes; it
does not apply to File objects.

Note: this impacts the output from PfValidationMessage, as it will
now list the names of embedded classes with a "$" separator instead
of a ".".

Change-Id: I2cd64bc0281dd324bf80eac57335a7299a516447
Issue-ID: POLICY-1646
Signed-off-by: Jim Hahn <jrh3@att.com>

Removed unused imports from models

Change-Id: I1f64127e1d783a962304c0add1524a57d6c24e4b
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Fix simple sonar issues in models: errors to sim-pdp"

Merge "Fix the bug of cannot return multiple versions of particular tosca policy and policy type"

Merge "Fix more sonar issues in models: yaml to dao"

Merge "Fix simple sonar issues in models-tosca"

Fix simple sonar issues in models: errors to sim-pdp

models-errors
models-pdp
models-provider
models-sim-pdp

Also had to work around this checkstyle issue:

src/test/java/org/onap/policy/models/sim/pdp/comm/
TestPdpStateChangeListener.java:[77,32] (javadoc) JavadocMethod:
Unable to get class information for @throws tag
'PdpSimulatorException'.

The error appears to be bogus, as PdpSimulatorException is on the
"throws" line thus indicating that the class IS accessible to the
above java file.

Change-Id: Iaca58457a32b00121000fc0bab12a8be4cb19bac
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Fix sonar issues in models: sdc to vfc"

Fix the bug of cannot return multiple versions of particular tosca policy and policy type

Issue-ID: POLICY-1807
Change-Id: I831c720993930a911d301a82b59a74cda7cb0c50
Signed-off-by: Chenfei Gao <cgao@research.att.com>

Fix more sonar issues in models: yaml to dao

Extracted common Strings into constants.
Reduced "cyclomatic complexity" in some return statements.
Used assertj to eliminate "log or rethrow" messages in junit tests.

models-yaml
models-base
models-dao

Change-Id: I20548d4cf5e67d085245e0d54df8ba0116ec86ec
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix sonar issues in models: sdc to vfc

sdc
sdnc
snr
so
vfc

Still haven't done the "rest" project yet, as still awaiting removal
of www.examples.org.

Change-Id: Icc4948799e2423bd4568232ee5629b88b7ecb996
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix simple sonar issues in models-tosca

Change-Id: I36a91fbbd95df7aff2656a97dd09b778cdbb798f
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Fix simple sonar issues in models

Added @FunctionalInterface where needed.
Replaced anonymous classes with lambda expressions.
Replaced duplicate strings with a constant.
Removed unused BeforeClass & AfterClass test methods.
Removed some trailing spaces.

Fixed:
aai
actor.appc
actor.appclcm
actor.sdnc
actor.sdnr
actor.so
actor.vfc
actorServiceProvider
appc
appclcm
cds
events

Change-Id: I0e21cbb10db6d1217bbd0e00e6dd4fac3eb84e31
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>

Merge "Allow multiple versions of entities to be returned"

Merge "Remove isNullVersion from identifier class"

Fix copyright

Missed on earlier reviews

Issue-ID: POLICY-1855
Change-Id: Ib172059e105e2940dce43f5da79b81dbd80f527d
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Allow multiple versions of entities to be returned

Fix .gitreviw file to point at "master" rather than "dublin"

Allow return of multiple versions of policy types and data types in
TOSCA service templates and multiple policies in TOSCA topology
templates.

Because the return type is a list of singleton maps, utility methods
were added to return flat maps of poliicy types, data types, and
policies keyed by a compound ToscaEntityKey name/version key.

Issue-ID: POLICY-1807
Change-Id: I355038aaca26f41064d0e3cb3b45b1de2294cf5f
Signed-off-by: liamfallon <liam.fallon@est.tech>

Fix fieldname for guard

The correct name is 'clname'.

Issue-ID: POLICY-1855
Change-Id: I27314576bb84615ccf225f0b57bfd710e2015e60
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
(cherry picked from commit f2b9a074416f814f8b4c071296a0abaf373db1b9)

Merge "Fixed SO request creation"

Fixed SO request creation

Fixed the SO creation from aai custom query.

Issue-ID: POLICY-1856
Change-Id: I5842a4dccbd61c774d84c8e897a876eae76d3a79
Signed-off-by: pramod.jamkhedkar <pramod@research.att.com>

Remove isNullVersion from identifier class

ToscaPolicyIdentifierOptVersion had an isNullVersion() method
which is not being used.  Removed this version, thus eliminating
the need to include a jackson @JsonIgnore annotation.

Change-Id: I03309099a1173591736f69b82b549ebce7994169
Issue-ID: POLICY-1653
Signed-off-by: Jim Hahn <jrh3@att.com>