Bug: Add check from alternateId cache before saving state

- ensured the the alternate ids/cm handle ids were removed from
  cache as part of the remove cm handle process
- upon saving a cm handle state , a check to the cache is
  performed first

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

Bump lfit/releng-reusable-workflows from 0.2.21 to 0.2.22

Bumps [lfit/releng-reusable-workflows](https://github.com/lfit/releng-reusable-workflows) from 0.2.21 to 0.2.22.
- [Release notes](https://github.com/lfit/releng-reusable-workflows/releases)
- [Commits](https://github.com/lfit/releng-reusable-workflows/compare/4231fda07b06d356c904fe39a868522393dee4bb...353a06e737c950cb297b3138784729ebd24e4861)

updated-dependencies:
- dependency-name: lfit/releng-reusable-workflows
  dependency-version: 0.2.22
  dependency-type: direct:production
  update-type: version-update:semver-patch

CIMAN-33

Change-Id: I8f2bc63a8062731d679060b73eeca0fd2cf63ff4
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: onap.gh2gerrit <releng+onap-gh2gerrit@linuxfoundation.org>

Fix: Correct import statement for ignoring delta performance tests

Changed the import statement to spock.lang.Ignore to ignore delta
performance tests correctly.

Issue-ID: CPS-2997
Change-Id: I1876ad229bd7c858820ab86a097d246b7cfcc135
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Ignore CPS Delta performance tests

Patch adds @Ignore annotation to skip Delta Report related performance
tests.
This is to test the stability of other tests that are impacted by delta
report performance tests.

Issue-ID: CPS-2997
Change-Id: I944b3941e92c2e74a05c1cd51803acf915b236d2
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Merge "Fix YangModelCmHandle SonarQube error"

Update performance test parameters for the Query API

- Performance tests for Delta report needed addition of a new anchor to the test data.

- This resulted in the failure of performance tests for Query across anchors API and slight increase in timings of respective test.

- The task modifies the test parameters to ensure compatibility with new anchor configuration.

Issue-ID: CPS-2997
Change-Id: I573f0f414dba6324e91b62bd9a06053bed406c13
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Fix YangModelCmHandle SonarQube error

- Changed YangModelCmHandle.toYangModelCmHandle to use DmiPluginRegistration object instead of 3 inputs
- Changed tests and YangDataConverter to reflect change

Issue-ID: CPS-3002

Change-Id: I8aab2ff8593d88b8b1655ea85dc509e062996392
Signed-off-by: egernug <gerard.nugent@est.tech>

Merge "Refactored large functions into smallar, modular blocks (Split-3)"

Refactored large functions into smallar, modular blocks (Split-3)

- extract meta-data into a separate config file for summary report
- extract trend names into a separate config file for trend declarations
- refactor large functions into readable blocks
- added pullPolicy (Always) for cps-deployment
- fix an issue with adding a result column: if duration tests results
  are zero, it is currently marked as passed.

Issue-ID: CPS-2967

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

Merge "Added extra buffer to Delta Performace Tests"

Added extra buffer to Delta Performace Tests

The existing buffer time on delta report related performance tests was
not sufficient.

Added 4-5 seconds of extra buffer time across all tests.

Removed the method resetTestAnchorData as it was causing Query API
related tests to fail.

Implemented an alternative approach to test Apply Delta Report
performance test, where the delta report is applied to a separate anchor
named openroadm-modified1 instead of openroadm1.

Issue-ID: CPS-2996
Change-Id: Ib063e0470b487105e31dc3fc1211f630a1392d8a
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Bump lfit/releng-reusable-workflows from 0.2.19 to 0.2.21

Bumps [lfit/releng-reusable-workflows](https://github.com/lfit/releng-reusable-workflows) from 0.2.19 to 0.2.21.
- [Release notes](https://github.com/lfit/releng-reusable-workflows/releases)
- [Commits](https://github.com/lfit/releng-reusable-workflows/compare/f318aaac9f4fce73a8059fe95ba9080833e018c2...4231fda07b06d356c904fe39a868522393dee4bb)

updated-dependencies:
- dependency-name: lfit/releng-reusable-workflows
  dependency-version: 0.2.21
  dependency-type: direct:production
  update-type: version-update:semver-patch

CIMAN-33

Change-Id: Ib1cba0852c145006031464a1cbea0536309d115e
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: onap.gh2gerrit <releng+onap-gh2gerrit@linuxfoundation.org>

Merge "Include a dmi-properties value at the top level"

Merge "Performance tests for CPS Delta APIs"

Merge "Cleanup k6 test templatters (Split-2)"

Performance tests for CPS Delta APIs

- Performance tests for two Delta APIs
- Delta Between 2 anchors
- Delta between an anchor and JSON payload
- Tests are inclusive of old and new delta report formats.
- Added a method to generate anchors with modified openroadm data
- Added a method to reset test data for an anchor

Issue-ID: CPS-2839
Change-Id: I8ec3a265c5fa02b8cd9060b65fee79a6ca9366e0
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Bump CPS to 3.7.2

Issue-ID: CPS-2994

Change-Id: Ib61e34e148c4575b3a18eaeb694a2f1f1d34e863
Signed-off-by: egernug <gerard.nugent@est.tech>

Add docker stage container 3.7.1

Issue-ID: CPS-2994

Change-Id: I9912bdc8418b791becaa3c2186f601ac87461fe5
Signed-off-by: egernug <gerard.nugent@est.tech>

Add maven stage yaml 3.7.1

Issue-ID: CPS-2994

Change-Id: Idc935ef985d0135a4eb4f0eedcff33cbd0e280fd
Signed-off-by: egernug <gerard.nugent@est.tech>

Update RElease Notes for release 3.7.1

Issue-ID: CPS-2994

Change-Id: I1416e44bcce33f1713a9531c5687fb37164fd547
Signed-off-by: egernug <gerard.nugent@est.tech>

Cleanup k6 test templatters (Split-2)

- the current template files make the development of k6 test complex and
  difficult to manage unnecessarily, need to be removed
- so that new ports and variables for the kubernetes cluster can be
  provieded easily for Split-3
- simplify the algorith to generate a human readeable summary report
- remove internal nodePort that is a remained action from commit
  (Split-1) that already merged
- total lines of code removed approx: 600

Issue-ID: CPS-2967

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

Merge "Refactor EventsProducer class to remove Generic type for legacyevent"

Include a dmi-properties value at the top level

- Added dmi-properties to YangModelCmHandle
- Amended tests to include new property

Issue-ID: CPS-2987

Change-Id: Ib1b8bc99389afff55d4ab93f2a435fb86769e43e
Signed-off-by: egernug <gerard.nugent@est.tech>

Merge "Change parent node xPath for saving subscription"

Refactor EventsProducer class to remove Generic type for legacyevent

- EventsProducer refactored to not use generic type and instead use
  LegacyEvent type for all the events that still uses non-cloud events.
- Events are implementing LegacyEvent interface now just to
  differentiate b/w cloud and non-cloud events when creating a
  kafkatemplate to send events

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

Merge "Control Model loader execution Order"

Merge "Local k8s env. running k6 test (Split-1)"

Change parent node xPath for saving subscription

- wrong parent node xPath was used by calling wrong method ,
  should be /dataJob and not '/'

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

Control Model loader execution Order

- Use @Order annotation to execute model loader sequentially instead of parallel
- Removed duplicated steps between model loaders (just one)
- Improved and consistent logging for each model loader starting and stopping

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

Merge "Manual Implementation of ProvMnS controller"

Merge "Consume 'ACCEPTED' create response event from DMI"

Consume 'ACCEPTED' create response event from DMI

- added agreed schema for dmi out event response
- added method to consume event from DMI for create response
- added method to update subscription status where status is
  ACCEPTED

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

Merge "Include a cm-handle-state at the top level"

Merge "Uplift dependency versions to eliminate vulnerabilities"

Manual Implementation of ProvMnS controller

- Implementation of ProvMnS using spring

Issue-ID: CPS-2966
Change-Id: Ia767b2779aa7b741230f295886f898246daf025c
Signed-off-by: seanbeirne <sean.beirne@est.tech>

Uplift dependency versions to eliminate vulnerabilities

-Springboot to 3.5.6
-Postgres to 42.7.8

Issue-ID: CPS-2747

Change-Id: I3161ba870ad9a16d0f11be2b019c8ae1fab393c4
Signed-off-by: egernug <gerard.nugent@est.tech>

Include a cm-handle-state at the top level

- Added cm-handle-state to YangModelCmHandle
- Amended tests to include new property

Issue-ID: CPS-2922

Change-Id: I82333eb69b4b5d77a76cb15868d7e7e6865de9e2
Signed-off-by: egernug <gerard.nugent@est.tech>

Bump lfit/releng-reusable-workflows from 0.2.18 to 0.2.19

Bumps [lfit/releng-reusable-workflows](https://github.com/lfit/releng-reusable-workflows) from 0.2.18 to 0.2.19.
- [Release notes](https://github.com/lfit/releng-reusable-workflows/releases)
- [Commits](https://github.com/lfit/releng-reusable-workflows/compare/8c3cf221da0e47955647647c9a254c1f807081ce...f318aaac9f4fce73a8059fe95ba9080833e018c2)

updated-dependencies:
- dependency-name: lfit/releng-reusable-workflows
  dependency-version: 0.2.19
  dependency-type: direct:production
  update-type: version-update:semver-patch

CIMAN-33

Change-Id: I6903eace93b637407f106b25592e8b1cd5adbec7
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: onap.gh2gerrit <releng+onap-gh2gerrit@linuxfoundation.org>

CI: Add Github2Gerrit workflow

Issue-ID: CIMAN-33
Change-Id: Ieffd064d98784d237ca19314cb1b3712691db2b6
Signed-off-by: Kevin Sandi <ksandi@contractor.linuxfoundation.org>

Local k8s env. running k6 test (Split-1)

- add a new service for kafka to provide broker external connectivity
- update resources with remedy values (hardware resources will be align
  with stakeholder requirements later)
- update kafka broker address to correct one for dmi-stub
- update licence headers for readme.md

Issue-ID: CPS-2967

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

Merge "Bug: Update operation executes on a data node that is not present in the database"

Bug: Update operation executes on a data node that is not present in the database

Failing test where an attempt is made to update a data node that does not exist
in database.
The test shows that no exception is thrown when such datanode is
updated.

Issue-ID: CPS-2980
Change-Id: Ia42b66016be4e880e6207916c4f98b9aa3069dad
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Merge "Remove CpsDataService dependency from DeltaReportExecutor"

Remove CpsDataService dependency from DeltaReportExecutor

CpsDataService is injected as bean in DeltaReportExecutor, This is
causing a cyclic dependency impacting CPS-2523(in progress).

The cyclic dependency forming is as follows:
CpsDataServiceImpl->CpsDeltaServiceImpl->DeltaReportExecutor->CpsDataService

The solution implemented is for DeltaReportExecutor to directly call
CpsDataPersistenceService

Since the cps delta service does not depend on CpsDataService, the
data validation steps which were present in CpsDataService have been
implemented in CpsDeltaServiceImpl and DeltaReportExecutor.

Issue-ID: CPS-2964
Change-Id: I1eecee22f67139eda111948d8aafad62c17340e2
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Subscription CREATE flow

- cover scenarios of the following: add subscription for non
  existing details in database, add subscription for overlapping
  details
- method first persists subscription and then send relevant
  details to DMIs
- remove classes, files and configuration that was from legacy
  subscription implementation

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

Merge "Update JEX Parser method"

Update JEX Parser method

-JEX parser needs to be updated to accommodate consumer and service methods in NCMP
-updated NcmpInEventConsumer and NcmpInEventConsumerSpec class
-Ensure REGEX for XPaths is safe and performant

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

Last inspect on all modules

- Mainly focussed on spelling mistakes (e.g. schema set instead of schemaset)
- Removed some unused code (overloaded but unused interface methods)
- spelling issues resolve in many timed method (names) by adding full stops.
  I think this only affects Graphana dashboards etc and should be OK
- Changed timeformat descriptions to say just "ISO standard format"
- Fixed latest SonarQube warnings
  (removed test from MultiPartFileUtilSpec as the scenarios are correctly unit-tested in ZipFileSizeValidatorSpec)

Issue-ID:CPS-2941

Change-Id: Ia654be16881b65b21522a554d6a70a3b38a6379e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Add test coverage for last missed branch in cps-service"

Merge "Make original delta report format consistent with delta report new format"

Merge "More fixes suggested by Project-wide Inspect Code..."

Add test coverage for last missed branch in cps-service

- Add test for user-order leaf-list using test-tree model
 (see https://en.wikipedia.org/wiki/The_Rattlin%27_Bog)

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

Updated onboardOrUpgradeModel in InventoryModelLoader to check if module revision already installed for all scenario

 - Simplified conditional flow by removing duplicate revision checks.

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

More fixes suggested by Project-wide Inspect Code...

- Removed some code duplication highlighted by Inspect
- Refactored some logging duplication
- Corrected suspected duplicate logging by correcting the message (copy&paste error)
- Added 1 or 2 null checks as suggested by Inspect (might need test for coverage)

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

Merge "update and use cm data job subscription model"

Merge "Fix issue with package name cmnotificationsubscription"

Merge "Add ReadinessHealthIndicator to expose migration readiness status"

Fix issue with package name cmnotificationsubscription

- removed _1_0_0 postfix (inconsistent with others)
- renamed/moved to ../datajobs/subscriptions

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

Add ReadinessHealthIndicator to expose migration readiness status

- Reports READY when migrations are complete.
- Reports NOT READY when a migration or rollback is in progress.
- This will allow Helm (Kubernetes) and other monitoring tools to detect application readiness during migrations and act accordingly.

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

update and use cm data job subscription model

Issue-ID: CPS-2971
Change-Id: I68368d5253e4e8aacfc4279527202acf91caa6e2
Signed-off-by: shikha0203 <shivani.khare@est.tech>

Merge "Refactor(deployment): make environment variables explicit for cps container"

Merge "Code Inspection WHOLE project"

Refactor(deployment): make environment variables explicit for cps container

 - Replaced dynamic env loop with explicit environment variable definitions
  (DB_HOST, DB_USERNAME, DB_PASSWORD, DMI_USERNAME, DMI_PASSWORD, KAFKA_BOOTSTRAP_SERVER).
 - Made dmi stub env. variable from string to integer for ms

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

Merge "update readme.md for cps charts"

Merge "Inconsistency With JSON Response(List Items) Using ReplaceANode API"

Merge "Add requestID and clientID in cps core logs"

Code Inspection WHOLE project

- applied most of the simple/safe stuff
- Many @suppress unchecked-warnings
- Many @Serial annotations
- A lot of copyrights updates!

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

Updated the k8s objects with correct labels

- added the label for app=cps-and-ncmp to all the kubernetes objects
- also added the relevant components label based on the type of
  kubernetes object
- added a custom service account cps-hazelcast-service-account to just
  provide the least amount of priviledges
- added cps-hazelcast-rbac file so that the hazelcast members are able
  to discover using the k8s discovery service

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

Modified Inventory loader to handle module install or upgrade based on flag

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

Inconsistency With JSON Response(List Items) Using ReplaceANode API

Jira - https://lf-onap.atlassian.net/browse/CPS-2800
Documentation - https://lf-onap.atlassian.net/wiki/x/gIDMAQ

- Updated DataRestController.java to return 201 for new nodes
- CpsDataService.java Changed method signature to return boolean
- Added logic to detect new nodes in CpsDataServiceImpl.java
- Implements the new logic for detecting and inserting new nodes in CpsDataPersistenceServiceImpl.java
- Added testcases for above changes

Issue-ID: CPS-2800
Change-Id: Ibc9bbb88ccbb07302355321c6d5c2eade0e7e5fb
Signed-off-by: Rudrangi Anupriya <ra00745022@techmahindra.com>
Signed-off-by: GM001016278 <gourav.malviya@techmahindra.com>

Fix more spelling mistakes

- Remove time stamp formatting from comments (its OK in code)
  (also reused existing formatter utility)
- added . (full stops) separators in counter names to make them acceptable words

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

Add requestID and clientID in cps core logs

- This enhancment is to signicantly improve the traceability and debugging capabilities for specific request flowing through our system.

- Create new class MdcServletFilter.java and added the dofilter to handle Mapped Diagnostic Context setup for capture and link requestID and clientID.

- Added testcases MdcServletFilterSpec.groovy for above changes.

Issue-ID: CPS-2796
Change-Id: I56c34400dc73c71b936a51260efd060505baabdc
Signed-off-by: GM001016278 <gourav.malviya@techmahindra.com>

update readme.md for cps charts

- update release name to cps for helm installation,
   as per requirement it will support only one release which is cps
- update port forwarding commands that are now uses the deployed
  services: cps-cps-and-ncmp-cps and cps-cps-and-ncmp-dmi-stub
- update get all command with correct label

Issue-ID: CPS-2956

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

Make original delta report format consistent with delta report new format

The old delta report only provides information about the leaf nodes that
changed.
The details about the data node are not provided as part of the delta
report.
The change adds node name and JSON sructure of the data node to the
original delta report format.

Issue-ID: CPS-2860
Change-Id: Iac51925f0d0a8ddf1958b47ad372090fbf0d75d9
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

updating NcmpInEventConsumer to only log the event information

-event handling will be done in next commit
-rebased

Issue-ID: CPS-2893
Change-Id: I6fe334ce66ef2b4f9e6f7916083821a73a5f7e8f
Signed-off-by: shikha0203 <shivani.khare@est.tech>

PoC Code Cleanup WHOLE Project

Check Nordix for comparing IntelliJ Clean Code Differences
(https://gerrit.onap.org/r/c/cps/+/141947)

Patch #1 : Community Edition
Patch #2 : Ultimate Edition
Patch #3 : Community Edition, Manually undo 2 incorrect updates
Patch #4 : #3 plus checkstyle (indentation) and copyright fixes
Patch #5 : #4 plus SonarQube warning fixes & refactoring

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

Merge "Data producer id field during cm handle create/update"

Data producer id field during cm handle create/update

- updated the CmHandleCreate test to prove that the LcmEvent is not
  triggered during the Cm Handle Create Flow
- added new test for CmHandleUpdate to prove the LcmEvent is triggered
  when updating the data producer id field during the Cm Handle Update
  flow
- refactored the common code and moved it to CpsIntegrationBase spec

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

Merge "IntelliJ Code Inspection Updates (last NB comment)"

IntelliJ Code Inspection Updates (last NB comment)

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

Merge "IntelliJ Code Inspection Updates"

IntelliJ Code Inspection Updates

- Use Lombok where possible
- Removed unused (interface) methods
- Removed parameters where method was called with only 1 (always same) value.
    This lead to so manual refactoring, more specific method names etc.
    For data job read I logged some details to get over this as the implementation is currently on hold/blocked
- Use 'record' (instead of class) where possible
- Apply @serial annotation
- Fixed unused result from matcher.find() We used to catch exceptions instead which is very bad :-(
- Some unused constant/return values needed for consistency, used annotation to ignore thee warning
- Removed Enum private value that was identical to Enum itself
- Uncheck casting, just added annotation to prevent warning I think they are all safe because we know and control the model
- Use {} for logging instead on string concatenation
- Use <a></a> for links in Javadoc
- Added missed null-checks
- Manually updated groovy test classes that were affected by some of above
- Spelling and grammar, original about 200 errors
     using 'Id' (with uppercase) I was flagged as a grammar issue and I like the idea of consistently using 'id' lower case in descriptions
     most typos where actually fixed by adding some work to the dictionary like cmhandle, onap, etsi etc.
     about 10% were genuine errors which also 2 or 3 variables affected which I have corrected now
- Security
     flagged some (http) links with suppress as we cannot change them
     some dependency issues remaining, need to check with Ger, looks like correct early warning without need for any CIO job or other tools!
- CODE ANALYZER ON WHOLE PROJECT
      only 4 java updates! inserted 2 finals (1 reverted as it was needed for test!), removed unnecessary casting in 2 places
      incorrectly add = sing in a cert.properties file for socker which I reverted

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

Update data producer id and send LCM event

- Logic updated for setting the data producer id
   * null to value is allowed
   * some-value to other-value allowed
   * other-value to null is not allowed and ignored
   * some-value to some-value is not allowed and ignored
- LCM event is raised only when the data producer identifier is updated
- added testware for the same

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

Disabling NCMP CSIT execution

- temporarily disabling the NCMP CSIT execution as they are unstable at
  the moment

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

Merge "REST operations using Delta Report"

REST operations using Delta Report

- Feature that uses the delta report to perform REST operations.
- The code uses underlying CPS data java interface logic to perform the
  REST operations.
- The REST operations are executed based on the 'action' identified in the
  delta report.
- The supported operations are: create, delete and update.
- Fixed indentation error in CpsDataServiceImpl

Issue-ID: CPS-2614
Change-Id: Ic1c9f5d2320afd8e4fb49b638e6636291124ea75
Signed-off-by: Arpit Singh <AS00745003@techmahindra.com>

Merge "Modify methods in CmSubscriptionPersistenceService"

Merge "JEX expression parser"

JEX expression parser

-Splits a multiline selector string into individual lines
-Resolves the nearest alternate ID from a single selector
-Returns a list of alternate IDs

Issue-ID: CPS-2892
Change-Id: I067c99c81d5ad8da9c582537e7fcf2d8d22e7dc9
Signed-off-by: shikha0203 <shivani.khare@est.tech>

Modify methods in CmSubscriptionPersistenceService

- existing methods of CmSubscriptionPersistenceService to fit the new cm subscription model
- unit tests updated, not used tests removed
- DmiCacheHandler, DmiCacheHandlerSpec, CmSubscriptionComparator, CmSubscriptionComparatorSpec was only modified to pass tests due to changes on persistence service methods - these classes will be changed as part of CPS-2166 development

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

Merge "Knowledge Sharing: Groovy/Spock best practices refactoring"

Knowledge Sharing: Groovy/Spock best practices refactoring

This commit shows many bad (before)/good (after) practises with Spock/Groovy Unit test like
- Mocking behavior and checking invocation (don't do both normally)
- When to use the word 'some' or 'my' for test variables/strings
- Remove scenarios that only apply to helper classes i.e. no branching in object under test
- Groovy 'magic' instead of java code
- Correct use of GString
- Update descriptions to refer to current (helper) classes an not old ones
- Use of assert keyword
- Consistent check of status code using 3pp constants
- Use 1 line when it is not too long (I don't mind up to 100-120 characters but it is just personal preference)
- Use Static imports where it does not reduce readability

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

Chore: Add dependabot config

Issue-ID: CIMAN-33
Change-Id: I09a0df4e370c636ea8b26e85a5cdd00c8e5363d0
Signed-off-by: Kevin Sandi <ksandi@contractor.linuxfoundation.org>

Merge "RTD update for CM stack use case"

Chore: Add dependabot config

Issue-ID: CIMAN-33
Change-Id: Ie682a0ba6b80dfa0c5f18a1141e95a3c113a7233
Signed-off-by: Kevin Sandi <ksandi@contractor.linuxfoundation.org>

Merge "Update schema for NCMP to DMI on datajob subscription"

RTD update for CM stack use case

- added support for operational datastore in CPS
- note to process the CM Data Notifications from ONAP DMI PLUGIN which
  is configured for data synchronization

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

Update schema for NCMP to DMI on datajob subscription

- based on agreed with E// outlined in https://lf-onap.atlassian.net/wiki/spaces/DW/pages/87984624/CPS-2166+Forwarding+CM+Data+Notification+to+Topic+Defined+in+Subscription#Schema-Definitions
- added target folder structure by adding 'from_to_destination'
  prefix on the new schema and existing datajob operation in
  event schema

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

Split the data sync request using module and root references

- Introduced a method to get concatenated module and root node for a
  given cm handle(using already synched modules) to use options field
- The request is splitted as some devices dont support fetching the data
  using root url and the devices that support that can in turn return
  huge amount of data in one go.
- Updated the data sync logic to make multiple calls for the initial
  data sync
- The failing requests are ignored and the algorithm works on best
  effort basis
- Testware to support all the above use cases

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

Merge "Use new model on subscription model loader"