Handle duplicate targets NcmpOut event

- Accpted , Rejected and Pending targets in the NcmpOut response to have
  unique values hence changing the datastructre to Set.

Issue-ID: CPS-2338
Change-Id: I24a109fad4c854eff1b052df38947cc121445bb9
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Add k6 for legacy async batch passthrough read operation"

Add k6 for legacy async batch passthrough read operation

- add kafka support to k6 codebase
- add two new tests: data operation batch passthrough read
   and data operation async batch read (from kafka)
- calculate the events/second via Trend
- add kafka ui support to docker-compose

Note: Before this commit being merged, the k6 executable
should have been compiled with the kafka extension of xk6-kafka

Issue-ID: CPS-2268

Change-Id: Ib7777b7bc9f15b210ea36d3541cba0e0c943f883
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Uplift liquibase-core dependency"

Fix ordering issue in QueryServiceIntegrationSpec

A test intermittently fails due to comparing unordered list.
Fix is to compare as a set.

Issue-ID: CPS-2337
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I84bc4f7d9ff01ca7b36ecb56835387dbea6b24d1

Uplift liquibase-core dependency

- upgraded from 4.21.0 to 4.28.0
- updated the release notes

Issue-ID: CPS-2326
Change-Id: I8734fd8005438ec682e8b2f458ec84bbabb4be56
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Delete Subscription method"

Merge "[k6] Fix response code check for passthrough write test"

Delete Subscription method

Issue-ID: CPS-2253
Change-Id: I30768dc431e13d0bac0dcc7b9b16cb0854f06702
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>

Merge "Added missing authorization header to DataJobService"

Added missing authorization header to DataJobService

- added auth header to the interface & updated testware

Issue-ID: CPS-2330
Change-Id: I091fc5c3c3fa323024f55055129cc7240732479a
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Integration test to validate DMI service URL

- New groovy test is written to validate if dmi service url is encoded
  correctly.

Issue-ID: CPS-2317
Change-Id: I75023a0bd2c41c832177edf062c694bb0d9b14b8
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

[k6] Fix response code check for passthrough write test

The POST operation returns 201, not 200.
This causes checks to fail when running k6 manually.

Issue-ID: CPS-2267
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I3c0e771b89aa09e41c9dbdf1c532599e1ab5440d

Get data job status from DMI Plugin

    - added java interface
    - extended DmiRestClient to get data job status
    - added testware

Issue-ID: CPS-2296
Change-Id: If9006bba06397724c15bdc3bdf1bd52a0188f002
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Release notes update for CPS-2306 and CPS-2319

Issue-ID: CPS-2319
Change-Id: I2ab58437c133521fdce10ade61b16e34cfdec0bc
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>

Integration test for the writeDataJob

    - added Stub controller
    - created integration test for writeDataJob

Issue-ID: CPS-2236
Change-Id: I067a11dc5bfe629d50128cf303b2a81abc75a348
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Merge "Fix "Create a node" and "Add List Elements" APIs response code"

Merge "Fix NCMP CSITs by using new TLS certificates"

Merge "Fix dataProducerId in SubJobWriteRequest"

Fix NCMP CSITs by using new TLS certificates

TLS certs for SDNC and PNF simulators have expired, causing CSITs
to fail. The new certificates are valid until 22/07/2034

- Add new certificates, valid for 10 years
- Re-enable NCMP CSITs

Issue-ID: CPS-2327
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I6c4361da9f8aaf1449a2ba22e701f1c0ec33a34b

Fix timing issue for data sync CSIT

The default data sync watchdog sleep time is 30 seconds, while
the CSIT only waits 10 seconds for data sync to run. This is
leading to intermittent failures.

- increase CSIT time out to 40 seconds for data sync operation
- adjust log levels for data sync so that issues are easier to debug

Issue-ID: CPS-2313
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I3ceb9097bf357f47e5d49f84c70b1c711b52171a

Fix dataProducerId in SubJobWriteRequest

Issue-ID: CPS-2322
Change-Id: I7bfdfd05ffacb085ac056707d477f348cb5bd64f
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Update response message for data validation failure

- Make the response message more informative and cleaner in case of data
  validation failure.
- Updated message body will not contain the entire jsonData as part of
  response sent back to the user

Issue-ID: CPS-2306
Change-Id: I28aab1450779af98ccdd11701e78bdfcdbc18cf0
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>

Fix "Create a node" and "Add List Elements" APIs response code

Create a node and Add list elements APIs would return a 500 response
code when a JSON payload mismatching the Schema was sent as part of
the request body.

Updated the code to return response code 400.

Issue-ID: CPS-2319
Change-Id: I86d6af446966790d9e45045b330f0c411fcd0f1d
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>

Disable NCMP CSITs

- comment the CSITs temporarily until the SDNC certs are corrected.

Issue-ID: CPS-2325
Change-Id: I335b17e4f148ff838301fb511656da1068d124b4
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Integration test to validate DMI service URL"

[k6] Measure NCMP overhead for passthrough operations

Measure NCMP overhead by subtracting DMI delay from request duration.

Issue-ID: CPS-2269
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ieb4758e9f7cab119da4c28e4a3febfa1361b2613

Integration test to validate DMI service URL

- New groovy test is written to validate if dmi service url is encoded
  correctly.

Issue-ID: CPS-2317
Change-Id: I048bf37dd3ba4f37df7dace927f55bf8dd899922
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "CPS NCMP: Add state tags to cps.ncmp.lcm.events.publish metrics"

Merge "XML content support on replace a node with descendants"

Merge "Report throughput for passthrough read operation"

Separate dmi delay for passthrough operations

- set dmi delay to 670 ms for passthrough write (CUD)
- set dmi delay to 300 ms for passthrough read

Issue-ID:CPS-2285

Change-Id: I7a4294d3ed04c89aeb5988a9975516135317af12
Signed-off-by: halil.cakal <halil.cakal@est.tech>

CPS NCMP: Add state tags to cps.ncmp.lcm.events.publish metrics

- Added cm handle state into micrometer Timer as an extra tags while capturing total time taken to publish a LCM events for publishLcmEvent method.

Issue-ID: CPS-2123
Change-Id: I6357c628496700fee7448dbc381a7e0e62d8b540
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

XML content support on replace a node with descendants

Issue-ID: CPS-2282
Change-Id: Ibb7ffb65ccbb03703266712c6d5c2eade0e7ab4b
Signed-off-by: Rudrangi Anupriya <ra00745022@techmahindra.com>

Report throughput for passthrough read operation

- change threshold from avg duration to req per second
- the delay for dmi has been hanlded in seprate commit
   in dmi repository
- add delays through docker-compose env variables

Issue-ID:CPS-2285

Change-Id: I35978b21c208fdb4209b1610160dc5545ee22965
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Add passthrough write (CUD) operations to k6"

Mapper to group Subscription Details for DMI

- introduced a mapper to group a collection of subscription
  keys(datastore cmhandle and xpath ) based on datastore and xpath
- the information will be used to send out to the dmi plugin

Issue-ID: CPS-2312
Change-Id: I99f509eb5b4a8c3a7fb1078ae86490d894f22b6c
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Bump CPS to 3.5.2 Snapshot

- bump CPS to 3.5.2-SNAPSHOT
- updated the release notes and prepared for the next release

Issue-ID: CPS-2318
Change-Id: I584faf6218e60ad62e32ef8c3cb0deb4047149a2
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

container yaml for CPS 3.5.1

- adding container yaml for CPS release 3.5.1

Issue-ID: CPS-2318
Change-Id: Iba39b49c46c30f917e526cdf716b963dc1e139d4
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

maven stage yaml for 3.5.1

- adding maven stage yaml for CPS 3.5.1 release

Issue-ID: CPS-2318
Change-Id: I14f4f8a890fbab80497696907fde4fd22601a562
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Release notes CPS 3.5.1"

Release notes CPS 3.5.1

- Added release notes and syncd the openapi files.

Issue-ID: CPS-2318
Change-Id: I1982e31f3bcb9337ed9814cb55a4db885b157158
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Print DMI stub logs in CSITs

Issue-ID: CPS-2313
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I4ecb5ce1713666b5e90b90eba3c98b737fd91896

Add passthrough write (CUD) operations to k6

- the delay for dmi has been hanlded in seprate commit
- for simplicity only the POST request is considered

Issue-ID: CPS-2267

Change-Id: I62ede04dd8e2744fc723566742396df31c42ea5d
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Print docker logs in CSITs for 2 cps-and-ncmp instances

Following introduction of 2 instances of cps-and-ncmp,
CSIT logs show an error "No such container: cps-and-ncmp".
This commit prints docker logs for each cps-and-ncmp container.

Issue-ID: CPS-2038
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: If3ed4590cef0f5b59d74e62c84f6c6e72edfd411

Merge "Policy Executor Feature Toggle"

Policy Executor Feature Toggle

- defined config parameters for feature toggle and server details
- log request details when feature enabled
- improved JavaDoc in Controller
- improved configuration properties checks in HttpClientConfigurationSpec

Issue-ID: CPS-2311
Change-Id: I1ff4bd3592ce2570ac74f9ec6e62b75001cb611a
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Fix timing issue in CmHandleCreateSpec"

Fix timing issue in CmHandleCreateSpec

The test for module sync retry on LOCKED handles has a timing issue,
due the ordering of the steps. After a LOCKED handle gets reset to
ADVISED state (but DMI is not yet set to available), the Module Sync
Watchdog may wake up and put it back into LOCKED state again.

This fix is to first make the DMI available before waiting for the
LOCKED handle to reset.

Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I9d33b0eeea1e0a2e163489130c928b4d03182774

Documentation updated in docs/ncmp-inventory-querying.rst

- Existing API docs in docs/ncmp-cmhandle-querying.rst is updated where
  CPS Path is now supported as ancestor axis.

Issue-ID: CPS-2289
Change-Id: I065179fe7c294588580448a8467765e9d116d33a
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "NCMP inventory cm handle search should support cps path query"

Remove nginx logs that not being used anywhere

Issue-ID: CPS-2038

Change-Id: Iba8950db4f3fcfc13f32bf915189f5fcb2ccee90
Signed-off-by: halil.cakal <halil.cakal@est.tech>

NCMP inventory cm handle search should support cps path query

- Added cps path query conditions for inventory search api

Issue-ID: CPS-2289
Change-Id: I0bcc9a45af65ad9d41c4d4cf79b35bfb95f50cff
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Split integration tests into cps and ncmp packages

Issue-ID: CPS-2309
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ie97c1430d1121c4c68b3fc6dacbc2466d48e1017

Policy Executor API, make Authorization header optional

Issue-ID: CPS-22291
Change-Id: I2b8f9b97c92dfc640c8b40b7053c609bbdcb2f6e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

NCMP: NCMP search API fixed to handle cps path that is ancestor axis as
well

- Cps path builder is fixed to handle ancestor axis that is target as well while quering.

Issue-ID: CPS-2308
Change-Id: Iaf215851ada17d21516ae83fa142ac77ff1c6c19
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Refactor remaining classes package structure

-Introduced org.onap.cps.ncmp.config for all config type classes like http, kafka etc.
-Introduced org.onap.cps.ncmp.impl.dmi for dmi specif classes like DmiRestClient etc.
-Moved most exception classes to org.onap.cps.ncmp.api.exceptions
(only two were 'internal' i.e. NOT used in REST so left in org.onap.cps.ncmp.exceptions)
-Deleted unused DateNodeHelper class (and related testware)
-Moved some classes to org.onap.cps.ncmp.impl.utils
-Moved messaging test base classes to org.onap.cps.ncmp.impl.utils.events
-All imports were reorganised automatically by IntelliJ
-did NOT update any copyright dates (seems incorrect to me anyway)

Issue-ID: CPS-2257
Change-Id: Ie28b1777ea278064b46c6a1c3a28f4d0696565e2
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Increase module sync wait time in tests to 10 seconds

Too small timeout is causing intermittant test failures.

Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ib6a25cb2f44e9a65b8f7d2d98e8d5e7dfe1f9ceb

Merge "Create PolicyExecutor Stub"

Merge "Setup automated tests with two replicas of cps-and-ncmp"

Merge "[k6] Measure CM-handle (de)registration in CM-handles/sec"

Merge "Remove forced module sync from NCMP integration tests"

Merge "Refactor integration tests so only one DmiDispatcher is used"

Create PolicyExecutor Stub

- Generate interface from OpenApi in RTD docs
- Fixed Content Type in OpenAPI
- Fixed Paths in OpenAPI
- Made Authorization header compulsory in OpenAPI
- All 'enum' values in OpenAPI lowercase (in linr with CPS/NCMP conventions)
- Added impl with some basic functionality
- Added testware
- Pom includes docker image creation
- Docker compose updated to deploy stub

Issue-ID: CPS-2301
Change-Id: I462ad5c70474b2813fc04005c0d20a1b15b574ec
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "Remove Mappers and Events Facade"

[BUG] Fix handling of special characters in moduleSetTag

During CM-handle registration, moduleSetTag is sent as part of the
JSON request body when getting modules from DMI. However, it
incorrectly parses the moduleSetTag as JSON, instead of encoding to
JSON. This results in parsing errors when moduleSetTag includes
characters such as ':'

Issue-ID: CPS-2302
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ie38008e0effac2423a4d069a91587e47d8747d66

Remove Mappers and Events Facade

- Removing Mappers and Events Facade and introducing the normal classes
  that this facade was wrapping
- Corresponding changes in the testware

Issue-ID: CPS-2298
Change-Id: I380d73ff3a9b3bf79633cd91580d6a4dee00a662
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

[k6] Measure CM-handle (de)registration in CM-handles/sec

As per characteristics requirements document:
- measure registration in CM-handles/second
- measure deregistration in CM-handles/second
- summary table includes test case number, description,
  units of measurement, actual value and limit.

Issue-ID: CPS-2269
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I838004da1c230ab722f49c2adacf34e730d7ac79

Merge "refactor cmsubscription code"

Remove forced module sync from NCMP integration tests

- remove forced module sync in tests
- set module sync timers to 1 second in test application.yml
- have tests wait up to 2 seconds for module sync to run

Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I727894dc1b4b9cee1ab82d05a9c31d1e3e067cf3

Refactor integration tests so only one DmiDispatcher is used

Issue-ID: CPS-2303
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I4d04fb9b7693a769e9059d3b2b7257a714da0532

refactor cmsubscription code

- Moved cmsubscription to the impl package instead of api.
- Below packages are renamed or newly created for code and tests
  - moved ..api.cmsubsription to ..impl.cmsubscription
  - renamed ..api.cmsubscription.mappers to ..impl.cmsubscription.ncmp
    or dmi as per the functionality
  - renamed ..api.cmsubscription.model to ..impl.cmsubscription.models
  - removed ..api.cmsubscription.producer/consumer and classes moved to  ..impl.cmsubscription.ncmp/dmi
  - new package ..impl.cmsubscription.cache to hold cache config and
    cache operations
  - new package ..impl.cmsubscription.avc to hold the CmAvcEventConsumer
    for the notifications emitted as a result of the subscriptions
  - Removed the prefix CmNotificationSubscription for majority of the classes , not removed for Config and model classes
  - Renamed the schemas and package structure as per the code refactoring

Note: *EventsFacade and *MappersFacade to be removed in the next
patchset

Issue-ID: CPS-2298
Change-Id: I1d788ab745d65965570e28beaefa511cbe4a8547
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Release notes for Web Client prometheus metrics

- Release notes updated as we enabled http client prometheus metrics.

Issue-ID: CPS-2121
Change-Id: I6918da7cd4e4523c1ec0ab6060bdc1686c141a9d
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Merge "Add Policy Executor Open API Interface definition"

Merge " CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url"

 CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url

 - Used autoconfigured web client builder for http_client_requests_* prometheus metrics.
 - Refactored dmi service url builder to create url template and its
   variables.
 - Web client is modified to use uri(urlTemplate, urlvars) version.
 - Deleted InvalidDmiResourceUrlException that no longer needed.
 - Used DmiServiceUrlBuilder to build dmi health check service url.
 - Created a new pkg url.builder into utils to have all related classes
   and record.

Issue-ID: CPS-2121
Change-Id: Id67e0f0d4e640bb8f9eea0b6c2db1dba3468e1d7
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Add Policy Executor Open API Interface definition

- OPenApi in RTD only (for now)
- Bear authentication included in OpenApi
- Placeholder RTD page (to be completed upon feature delivery)
- For consistency (and color coding in editors) I put all Description and Example strings inside double qoutes "..."

Issue-ID: CPS-2291
Change-Id: I7f16cdce2a06cb011997005fa8b86b54c044304a
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Fix sonar issue in performance test ResourceMeter

Replace busy loop with await().until as suggested by sonar

Issue-ID: CPS-89
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ic6c870c419a67708082ceb05ef3ccba884ae375e

Setup automated tests with two replicas of cps-and-ncmp

- this change for two test suits: CSIT and K6
- increase cps-and-ncmp replicas to 2
- introduce nginx as load-balancer
- set proxies inside nginx for the cps-and-ncmp URI paths
- upgrade hazelcast-spring to 5.3.7

Issue-ID: CPS-2038

Change-Id: If1b6c1e1218af16b24cc061507ea9b25d277d143
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Improve handling of Invalid Operation exception"

Merge "Fix incorrect use of getAnchors(dataspace, schemasets)"

Improve handling of Invalid Operation exception

- MOve to API package
- Handle in REST Controller to return BAD REQUEST

Issue-ID: CPS-2256
Change-Id: I4f96519aaf2e7f781729c8b8671ea02630a2e0a8
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

repackage 'data' feature

- moved relevant classes into these (new) packages:
api.data.exceptions
api.data.models
impl.data
impl.data.async
impl.data.exceptions
impl.data.models
impl.data.utils
utils.events
- removed old unused event class
- moves some missed inventory related class to the right place

Issue-ID: CPS-2256
Change-Id: I75563e063acc0054769d8f2b13146e6c1d1c6054
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

move lcm/common events classes

-moved api.impl.events.lcm to impl.inventory.sync.lcm
-moved most api.impl.events class to utils.events
(remaining classes in api.impl.events belong to cmsubscrption and Priyank wil take care of those)

Issue-ID: CPS-2258
Change-Id: I1818194e708aa0d9c713b470e878997c2f26fb13
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Simplify package structure cps-ncmp-rest"

Simplify package structure cps-ncmp-rest

Issue-ID: CPS-2294
Change-Id: Idc3605c1b877be13a8d344875612da6bef791fc7
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Move TrustLevel Feature"

Move TrustLevel Feature

-Moved TrustLevel impl from Controller to Inventory Facade
-Applied new package name convention using impl\inventory\trustlevel for this (sub)feature
-Renamed some trustlevel related classes to better describe their role
-Disabled failing architecture test (agreed this is tech. debt and created CPS-2293)
-Refactored YangDataConverter method names and signatures (moved Spec to correct place)
-Cleaned up names and javadoc of class now called DeviceTrustLevelMessageConsumer
-Applied conventions to modified test methods

Issue-ID: CPS-2255
Change-Id: Iccd20541488e6b61444bb2c7406c90e61a34124d
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Disable the java debug port permanently"

Disable the java debug port permanently

Issue-ID: CPS-1864

Change-Id: I0366b126920cf0bc706545bc5b4e469d3faa2486
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Merge "Clean up the k6 test suite"

Fix incorrect use of getAnchors(dataspace, schemasets)

There is an issue in a method deleteDataNodes(dataspace, anchors)
where it doesn't send data update events. This is because it fetches
anchors using a method getAnchors(dataspace, schemaSetNames), when it
needs to fetch anchors by name: getAnchors(dataspace, anchorNames)

Changes:
- Rename getAnchors method using schemasets to getAnchorsBySchemaSetNames
- Add a method getAnchors(dataspace, anchorNames)
- Update test of deleteDataNodes method using getAnchors so it checks
  that data update events are sent

Issue-ID: CPS-2254
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I55fe853f0a9278a66a5724bf4cf2723b0e2fbc8b

Repackage Inventory Feature

- decided that cm handle queries is part of 'inventory' feature
- a lot of files (~100 total) have changes because of some files have been moved.
- The reviewers should probably just focus on the MOVED/renamed (~30) files
- New packages (top level only):
-- org.onap.cps.ncmp.api.inventory
-- org.onap.cps.ncmp.impl.inventory
-- org.onap.cps.ncmp.impl.models  (common models NOT used in API, this was unforeseen)
-- org.onap.cps.ncmp.impl.utils
- I am NOT updating copyright years as part of this commit since there are no real code changes and it is simply NOT worth the effort
TODO:
- extend scope to 'TrustLevel' as an 'internal feature' (TBD)

Issue-ID: CPS-2255
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I40b4efc8326068ab9ae1f31b77f46b2fe8579431

Clean up the k6 test suite

- Remove the tests that are not in the Characteristics
- Set .625 seconds/operation for cm handle id search with module filter
- Set 13 seconds/operation for cm handle search with module filter

Issue-ID: CPS-2286

Change-Id: I88d5773abadb29ce46914029332384ae8e55e032
Signed-off-by: halil.cakal <halil.cakal@est.tech>

Fix compile issue after facade introduction

Due to automatic rebase, a method got renamed in one commit but not
the other.

Issue-ID: CPS-2263
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Id0fa8eba79cb98bb24419bfddc0c9b9e8c9a42d3

Merge "Introducing NCMP Facades"

Merge "Spliting a data-job into sub-jobs for DMI Plugin"

Introducing NCMP Facades

- NetworkCmProxyDataService was polluted with many non-data operations
- it is now split into:
1) NetworkCmProxyFacade, a single thin facade for the (main) controler to redirect calls to the correct services
2) CmHandleRegistrationService(Impl), methods related to registration only
- introduced NetworkCmProxyInventoryFacade for the invenoty controller
- renamed some services for consitency and clarification
- Use facade to acces ncmp data request handlers (instead of direct from controller)
- remove unnecesarry wrappings between request handlers and facade
- split facades according to names: data & inventory
(the REST controllers are not split properly so I think one rest controller will end up needing both facades)

Issue-ID: CPS-2263
Change-Id: I250732aa16ec28b43ff642d2adf10ba36f67290e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Bump CPS version to 3.5.1-SNAPSHOT

- bump cps version to 3.5.1-SNAPSHOT
- updated release notes to make it ready for the next release

Issue-ID: CPS-2283
Change-Id: I242cae7c95a87c1a9e225cce3eb7b36097f1e5ba
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Maven container yaml for release

- added maven container yaml to release cps 3.5.0 version

Issue-ID: CPS-2283
Change-Id: I45ce5b440d8ca2f6ff3999971202f9fc3bd4a8fa
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>