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>

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>

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"

Merge "SONARQUBE ISSUES: 'equals' is deprecated"

Use new model on subscription model loader

- remove integration test (notification related) for subscription as it is now invalid
  due to development
- modify unit tests with use of new model on onboarding
- onboard new model
- remove old model from repo

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

Merge "New schema for creation/deletion datajob subscription event"

SONARQUBE ISSUES: 'equals' is deprecated

- equals by StringUtils is deprecated (see https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#equals(java.lang.CharSequence,java.lang.CharSequence)
- replaced deprecated based on recommended (See https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/Strings.html#equals(java.lang.CharSequence,java.lang.CharSequence)
- modified import org.apache.logging.log4j.util.Strings to
  import org.apache.commons.lang3.StringUtils for use of
  .isNotBlank to avoid conflict

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

Allow get operation on the non-passthrough datastore

- Now that the data can be cached when the data sync is enabled , this
  patch allows to query that data using ncmp-passthrough-operational
  datastore
- Added testware to support the above feature
- Updated outstanding copyright from the last patch

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

Bump CPS to 3.7.1

Issue-ID: CPS-2917

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

Added 3.7.0-container.yaml

Issue-ID: CPS-2917

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

Add maven stage yaml 3.7.0

Issue-ID: CPS-2917

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

Update Release Notes for Release 3.7.0

Issue-ID: CPS-2917

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

Merge "Process Cm Avc Event in NCMP"

Process Cm Avc Event in NCMP

- process the cm avc event when the source system is set as ONAP DMI
  Plugin in the header
- handle the create , update , patch and delete operation to the CPS
  Cache.
- Testware to support the write operation
- Usage of the RestConf to Cps Path utility in this patch

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

Ignore SonarQube Deprecation Warnings

- Supressed warning for DeprecationHelper#mapOldConditionProperties
- Actual removal will be done later as part of CPS-2914

Issue-Id: CPS-2914
Change-Id: I3ca9a6f1754af5c34e7925ab76484aaceb482474
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

New schema for creation/deletion datajob subscription event

- schema is based on analysis and proposal in https://lf-onap.atlassian.net/wiki/spaces/DW/pages/87984624/CPS-2166+Forwarding+CM+Data+Notification+to+Topic+Defined+in+Subscription#Comparison-Subscription-Legacy-Create-Event-and-R1-DataJob-Event

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

Merge "Replaced parameter includeCmHandlePropertiesInQuery by outputDmiProperties"

Uplift Springboot to version 3.5.4

- Moved to Springboot 3.5.4 to eliminate vulnerability in netty

Issue-ID: CPS-2747

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

Replaced parameter includeCmHandlePropertiesInQuery by outputDmiProperties

Replaced includeCmHandlePropertiesInQuery query parameter in search CM  handle api by outputDmiProperties

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

Merge "Remove old subscription documentation"

Remove old subscription documentation

- removing old subscription documentation as part of new
  implementation of datajob subscriptions

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

Merge "Utility to convert restconf style path to cps path"

Utility to convert restconf style path to cps path

- resource-identifier in passthrough request has restconf style path for
  the devices
- when listening cm avc event on successfull write operation , ncmp has
  to update the cache, hence the path needs to be converted to cps path
  to reuse cps-core services
- added testware to support this
- actual usage of this utility is in the next patch
- for simplicity, utility supports just one key in the path

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

CPS local Helm charts

  - added helm chart
  - added required services (cps, dmi etc)
  - added headless service to access hazelcast service
  - added README

Issue-Id: CPS-2783
Change-Id: I11f0b07b80625a80b414490d630407a262495867
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Create new data job subscription model"

Merge "Make Groovy Performance Test Stable"

Fix Vulnerabities

- Upgraded commons-lang to 3.18.0

Issue-ID: CPS-2747

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

Merge "Part 3: Refactor CPS Delta code to utility class"

Make Groovy Performance Test Stable

- Overload assert method to apply custom margin
- Increase margin to 3x (+200%) for tests
  Chosen based on last 2 failures and graph data

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

Merge "SonarQube Violation: fields() deprecated"

Part 3: Refactor CPS Delta code to utility class

- Condensed Delta Report format refactored to a utils class

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

Merge "Part 2: Refactor CPS Delta code to utility class"

test RTD change to verify python and os upgrade

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

Merge "Upgrade OS, Python and Sphinx for RTD build Config"

Upgrade OS, Python and Sphinx for RTD build Config

- Updated .readthedocs.yaml (build config) to use
OS: ubuntu-24.04
Python 3.12 (as this is what Ubunut-24 includes)
- tox.ini basepython to 3 (will use whatever release OS has)
- tox.ini Removed all python dependency constraints as they prevented using newer sphinx version)
- requirement-docs: Sphinx to version 8.2.0 (to work with python 3.12 or higher)
- conf.py add required extension and exclusion patterns for unused files

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

Merge "Create controller for ProvMnS interface on NCMP"

Part 2: Refactor CPS Delta code to utility class

- Refactoring code for original delta report format to a utility class

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

SonarQube Violation: fields() deprecated

- change use of fields() to properties() as documented https://www.javadoc.io/doc/com.fasterxml.jackson.core/jackson-databind/latest/com/fasterxml/jackson/databind/JsonNode.html#fields--

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

Merge "Refactor K6 test script to use template based code generation"

Merge "Upgrade SDNC image"

Create new data job subscription model

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

Refactor K6 test script to use template based code generation

- Introduced templates for kpi config and scenario config.
- Used these templates to generate thresholds and trends.

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

Merge "Part 1: Refactor CPS Delta code to utility class"

Upgrade SDNC image

- upgrade SDNC image from 2.2.3 to 3.0.2
- upgraded SDNC image doesnt have support of application/yang.patch+json
  so converted the request to equivalent application/json patch which is
  supported
- updates to healthcheck url and node mount url as per the upgraded
  version

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

Create controller for ProvMnS interface on NCMP

- New endpoints return 501 Not Implemented. Full implementation to be
  completed in later patch
- Parameter descriptions pulled from 3GPP documentation (ref to wiki)
- TODO: implement southbound interface forwarding in NCMP

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

Update CM Handle API Query Performance Test Title and Expectation

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

Part 1: Refactor CPS Delta code to utility class

 - Refactoring Delta code to separate utility class for a more granular
   and manageble code.
 - Patch 1 refactors code for 'update' action as the code is shared
   between old and new delta report formats.

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

Merge "Fix: Update OpenSSF Scorecard to RelEng reusable"

Fix: Update OpenSSF Scorecard to RelEng reusable

Change-Id: I99dbf9a6f6f439ada445d96ae3a63fdb4237cef5
Signed-off-by: Matthew Watkins <mwatkins@linuxfoundation.org>

Revert RTD (docs) tooling to Python 3.10, Sphinx 4.2.0

- Also restored dependency constraints
- Temporary, will request LNF IT to support new toolchain

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

Update RTD Cm Handle Queries, Upgrade Sphinx build tools

- Combined all cm handle query documentation into 1 page (for both interfaces) because of overlap
- Upgraded basepython to 3.12.3
- Upgraded sphinx to version 8.2.0 (to work with python 3.12.3
- Removed all python dependency constraints as the prevented using newer sphinx version)

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

Merge "Fix endurance failures"

Merge "Uplift Springboot to 3.5.3"

Fix endurance failures

- endurance test failed since it exited with 'Unsucessful' termination
  because teardown.sh  was not able to remove CPS images (cps-and-nmcp,
policy-executor-stub). The tag 'latest' was used for both pipelines at
the same time: Regular KPI and Endurance
- teardown algorithm changed: it only remove cps images having 'Snapshot' tag.
- dmi-stub removed from the image list since it should always use the
  Snapshot version in the deployment!

Issue-ID: CPS-2870

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

Uplift Springboot to 3.5.3

-Uplifted Springboot to 3.5.3
-Uplifted JAckson to 2.19.1

Issue-ID: CPS-2871

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

Clearly separate North- and Southbound interface in Postman collection

- Move relevant test into new North- and Southbound subfolders
- Add more Cm Handle (id) Query examples
- Cleaned up example names (more concise and consistent)

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

Add Groovy Perf. Tests CM Handle Query REST API

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

Update documentation

- after merging commit for clean-up, documentation should be updated to
  reflect docker-compose.yml changes

Issue-ID: CPS-2861

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

Clean up setup and teardown processes for k6 tests

- rename cps-base.yml to docker-compose.yml to use minimum
  docker-compose commands efficiently
- rename confusing file names:
  - run-k6-tests.sh to k6-main.sh
  - run-all-tests.sh to execute-k6-scenarios.sh
  - ncmp-test-runner.js to scenarios-config.js
- cleanup teardown.sh
  - not only endurance but also kpi should remove all ONAP docker images
    since setup function always builds them!
  - Don't remove any other images (they don't belong ONAP) because it
    bring extra pulling time during the setup, and make the pipeline
slower.
  - combine relevant methods into one
- add cm avc event and v3 data node get metrics to metrics.csv file to
  prevent them commit always
- add missing import for crypto

Issue-ID: CPS-2861

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

Merge "Refactor WriteSubJobSpec to make use of test REST endpoint of write job"

Fix validation northbound cm handle searches

- Renamed related methods and Enum using terms north (client facing) and southbound (dmi) for more clarity
- Northbound Cm Handle search was using the wrong Enum for validation (this caused the issue)
- Grouped and ordered renamed methods in facade impl
- Enums also had the wrong values: unsupported values in one and missing values in the other
  (southbound can/should support all possible queries, no reason not too)
TODO (separate subtasks and commits)
- Update RTD documentation to properly describe options in each ifc
- Update Postman collection to have cleared separation between north and southbound interfaces

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

Merge "Add dm-service-name cm-handle searches to  integration tests"

Merge "Refactor: Improve PolicyExecutor WebClient error logging"

Refactor WriteSubJobSpec to make use of test REST endpoint of write job

- Replaced internal service-layer test with REST-based integration test for writeDataJob

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

Add dm-service-name cm-handle searches to  integration tests

- Split cm handle search tests in North- and Southbound test classes
- Added test variations with and without (additional) cm handle properties
- Too many functional NCMP test: grouped them in folder by feature names
  (semi-automatically updated copyright to all moved files even if not strictly necessary
- Reformatted and cleaned up (descriptions) northbound tests
- Minor reformatting of production code
- Increase margin on performance timer accuracy test (unrelated but caused test to fail sometimes)

Issue-ID:CPS-1872

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

Remove private properties from the response whem the value is null

- Fixed openapi and unit test

Issue-ID:CPS-1872
Change-Id: I6c4465fd79249861814a56c6fb02e4d5cf854928
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>

Refactor: Improve PolicyExecutor WebClient error logging

- Improved logging and ensured consistency with processFallbackResponse().

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

Merge "Unify dmi/private/additional/cmhandle properties naming to additional properties"

Unify dmi/private/additional/cmhandle properties naming to additional
properties

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

Merge "Fix vulnerabilities"

Fix K6 KPI Legacy Batch Reporting issues II

- Changed legacy batch consume to ran once per second (constant-arrival-rate) rather then only 1 big loop
- Updated k6 install to version 1.0.0 ! (old version cause most of the issue this patch took so long)
- Removed import of crypto as it is now part of k6 standard install
- No more checks on total messages, just recording message consumed every second
- Fixed 'timeout' configuration parameter, it is 'maxWait' !
- Fixed some incorrect casing on variables
- Added (now debug level) logging for troubleshooting produce/consume legacy batch events
- Improved name of config parameter: containerUpTime to containerCoolDownTime
- Reduce cool down time for kpi to 10s (its really only needed for endurance)
- Improved several variable names (consistency!)
- Reduced scope of data be passed on (to what is used) e.g. "data.metrics" instead of "data"
- Some formatting changes
TODO remove small test config

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

Fix vulnerabilities

-Upgraded postgres to 42.7.7
-Upgraded springboot to 3.4.7
-Upgraded httpclient to 5.4.4

Issue-ID: CPS-89

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

Ensure teardown.sh is alwasy executed

- Currently, on_exit() is advised to run by only 'EXIT' signal that is
  for gracefull termination
- Thus, on_exit() function might not run in cases where job terminates
  abruptly
- To guarantee on_exit() alwasy being called, then add other common
  termination signals: SIGINT, SIGTERM, and SIGQUIT
- This will allow the trap command to catch these signals and execute
  on_exit() before it terminates.

Definition of the signals:
- SIGINT: Interrupt signal
- SIGTERM: Termination signal, a generic signal used to request a
  process to terminate.
- SIGQUIT: Quit signal

Issue-ID: CPS-2821

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