Merge "Bump lfit/releng-reusable-workflows from 0.2.22 to 0.2.24"

Merge "Analyze the fluctuations seen in the Groovy based performance tests"

Introduce Model Loader Locking Mechanism

- Moved core hazelcast config class and parameters from ncmp to cps service
- Renamed CpsAndNcmp lock classes and config to 'CpsCommonLocks' for common use
- Moved (duplicated) test to correct test base class
- Introducing Model Loader Coordination classes
- Update lombok config in cps-service to include qualifier annotations
- Update model loaders to only execute when master

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

Analyze the fluctuations seen in the Groovy based performance tests

Testing the cause of fluctuations in QueryPerfTest by disabling the
DeltaPerfTests

This is done as the fluctuations started after Delta Performance Tests
were merged

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

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

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

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

CIMAN-33

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

Adjust timing for Creating Delta Report Test Data

The recorded time for creating a modified variant of openroadm anchor
is slightly more when it is being run on the Jenkins pipeline.

Fix is to adjust the timings by adding more buffer for the test to run successfully.

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

Merge "Isolate Delta Report performance test data"

Merge "Introduce GET operation in ProvMnS"

Merge "complete flow for subscription delete"

Merge "Add scripts that manage k8s orchestration (Split-4)"

Bump to version 3.7.3

Issue-ID: CPS-89

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

Add docker stage container 3.7.2

Issue-ID: CPS-89

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

Add maven stage yaml 3.7.2

Issue-ID: CPS-89

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

Update Release Notes for version 3.7.2

Issue-ID: CPS-89

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

Merge "Revert "(Bug) Ensure clean up process also removes orphaned data in fragmenttable""

Revert "(Bug) Ensure clean up process also removes orphaned data in fragmenttable"

This reverts commit 15e20fdf691556fd1c6a8d581dbd06716c02b06d.

Reason for revert: Previous revert patch makes this incorrect.

Issue-ID: CPS-3006
Change-Id: I366cc5fa23aa89295f6c3a5b88068d91478e6cb3
Signed-off-by: egernug <gerard.nugent@est.tech>

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

Merge "(BUG) Move adding to cache in registration"

(BUG) Move adding to cache in registration

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

complete flow for subscription delete

- Consume dataJobDeleted events and extract relevant details
- Query affected dataNodeSelectors and subscription details from DMI
- Remove target subscriber from arrays, keeping remaining subscribers intact
- If last subscriber is removed, delete entire entry
- Send the delete event to the relevant DMI
- Update tracking of subscription removal throughout the system

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

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

- This reverts commit 30d460ae43131d5fef3de041dcbcd2943702724a associated
  with CPS-2800.
- Due to logical errors in the previous patch, any data node added
  using Replace a node with descendants API gets added at the root level
  instead of the specific xpath.
- Suitable approach is to revert the patch and rework on a new patch set
  with corrected logic.

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

Merge "(Bug) Ensure clean up process also removes orphaned data in fragmenttable"

Merge "Bug: Add check from alternateId cache before saving state"

Introduce GET operation in ProvMnS

 - Moved code gen for ProvMns from ncmp-rest to ncmp-service package
 - Created new util class for provmns operations
 - Fixed legacy naming of methods

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

Merge "Fix the application readiness state during migration"

Add scripts that manage k8s orchestration (Split-4)

- updated bash scripts for installing charts, running k6 tests,
  uninstalling deployment, and storing pod logs
- added environment variable: deploymentType to differentiate deployment
  cluster and hosts
- added hosts that is related to k8s cluster deployment
- updated readme.md
- set timeout to 300s for the pod availability
- increase resources (cpu and memory) for kafka and zookeper services to
  let them start successfully on virtual machine
- increase resources (cpu and memory) for dbpostgresql service to
  support at least 30K cm handle for the virtual test server

Issue-ID: CPS-2967

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

(Bug) Ensure clean up process also removes orphaned data in fragmenttable

- Add method to remove orphaned data from fragment table to the
  endpoint /cps/api/{}/admin/dataspaces/{}/actions/clean
- the condition for removing fragment table data is based on the following:
- (a) if entity's xpath contains more than 1 '/' and the parent is
  null
- (b) the parent id is equal to the id of an entity that
  meets condition (a)

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

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>

Merge "Add integration test for create subscription"

Isolate Delta Report performance test data

- Performance tests for Delta report required creation of an anchor used
  only for performance tests of Delta Report
- Current approach made this anchor available to other performance tests
  causing them to fail.
- Updated approach isolates the test data for Delta Report to
  DeltaPerfTest class.
- removed addModifiedOpenRoadData() method as it is no longer needed to
  create anchor for Delta Report tests.

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

Fix the application readiness state during migration

- updating the readiness health indicator to just work on the readiness
  state and not on the overall state.
- before this fix, during migration the readiness, liveness and overall
  status is DOWN.
- after the fix, readiness and overall state would be DOWN, while the
  application will be live, hence liveness probe will show as UP
- ReadinessManager relies on ApplicationReadyEvent now instead of
  ApplicationStartedEvent
- Dynamically register the model loaders during Application startup
- removed the initial delay string from the watchdogs
- Added a new arch rule for the cps init package to be used in
  ncmp-service
- removed the module-sync-delayed package

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

Add integration test for create subscription

- add test for creating subscription and sending to multiple DMIs
- update status of a subscription based on dmi response event
- add test for creating subscription wherein it partailly overlaps with existing active subscriptions
- add test for creating subscription wherein it fully overlaps with existing active subscriptions

Issue-ID: CPS-2995
Change-Id: Iabbd54168ab8b9b241c357519e65e206440533dd
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>