Postman and Stub updates for testing ProvMns Error handling

- Aligned PolicyExecutor Stub simulations with DMI Stub
- Add Simulation for Policy Denial in Stub PE Controller
- Postman updates: Data (ProvMnS) folder, including:
  - get, put, del success scenarios
  - get, put, del error and deny scenarios
NOTE Enable Policy Executor (env variable) to test/demo deny scenarios!
- Also contains fix for double slash (will disappear once separate for commit for that is merged)
TODO
- PostMan Patch scenarios will be done in separate commit by Sean as it requires DMI Stub update too

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

Removed extra slash on outgoing fdn

- Extra preceding slash added for longest match search
- Now will be removed when forwarding to dmi

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

fix: 100% code coverage for ParameterMapper

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

Merge "Revert docker compose config to 2 instances"

Revert docker compose config to 2 instances

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

Merge "Adjust timing for Delta Report performance test and make OpenROADM test data deterministic"

Refactor ProvMnS error handling and simplify request parameters

This update covers several tickets: CPS-3078, CPS-3093, CPS-3095, CPS-3096, CPS-3099
- Consolidate error handling in ProvMnSRestExceptionHandler
  - changed reason to title
- Remove ErrorResponseBuilder and integrate functionality into exception handler
- Refactor ProvMnsController and centralize error handling into 1 common method. including
  - correct code for Policy Executor denial
  - handle timeouts
  - handle all other exceptions
  - handle #/attribute reference validation depending on content type
- Enhance ProvMnSException with additional error handling capabilities
  use ProvMnSException to store the information to create the correct error response
- Simplify OperationDetailsFactory implementation

TODO: Remove docker config updates for debugging
TODO Lee Anjella
   - Check if a mapping to type is available for every possible statuscode in provMnsExceptions
   - COVREAGE 100% MIsine exception in ParameterMapper.java

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

Merge "fix: Reduce high cardinality in ProvMnS API client metrics"

fix: Reduce high cardinality in ProvMnS API client metrics

- Add ClientRequestMetricsTagCustomizer to template FDN values in URI metrics
- Apply custom metrics handler to DMI WebClients
- Template URI from /ProvMnS/v1/SubNetwork=A/ManagedElement=E to /ProvMnS/v1/{fdn}
- Preserve query parameters as templates for operational visibility
- Fix existing tests to handle new WebClient configuration dependencies

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

Merge "Add dmi-properties to integration test"

Merge "Part 2: Add delta report in Kafka notification"

Adjust timing for Delta Report performance test and make OpenROADM test data deterministic

- Adjusted timing for Delta Report performance test to account for changes
  observed after isolating anchors within DeltaPerfTest.
- Change applies to Delta between 2 anchors test
- Modified approach for creating modified OpenROADM test data:
  - Replaced random generation with deterministic logic
  - Ensures consistent and reliable performance test results

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

Part 2: Add delta report in Kafka notification

Added support for delta report in kafka notifications for CPS core
endpoints.
The following methods are supported in this path:
 - saveData, saveListElements,
   updateNodeLeavesAndExistingDescendantLeaves, updateNodeLeaves,
   updateDataNodeAndDescendants, replaceListContent, deleteDataNode,
   deleteListOrListElement, deleteDataNodes.
The following methods are not supported in this patch:
 - updateDataNodesAndDescendants, deleteDataNodes
 As they need a separate method to generate delta report and then send
 the notifications, which can be done as a separate patch.

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

Merge "Bug: k6 logs not saving"

Add dmi-properties to integration test

Issue-ID: CPS-3061

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

Bug: k6 logs not saving

- renamed make-logs.sh to archive-logs.sh
- refactored archive-logs.sh for better readability
- implemented log file numbering with newest files first
- modified so only one zip folder for each test run; zip folder
  contains all services logs
- improve timestamp format for better readability
- moved log collection action from teardown to main script
  exit handler
- ensure logs are captured regardless of test outcome

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

Merge "Migrate cm-handles.additional-properties to cm-handles.dmi-properties"

Merge "Fix: update documentation for CPS-2428 notification subscription APIs"

Migrate cm-handles.additional-properties to cm-handles.dmi-properties

- Split bulkUpdateCmHandleStatesAndProperties and updateCmHandleFields to perform separate tasks
- updateCmHandleFields updates fields in previous use cases
- bulkUpdateCmHandleStatesAndProperties only updates fields for migration purposes
- Added dmiProperties to CmHandleStateAndDmiPropertiesUpdate
- Added convertAdditionalPropertiesToJson to convert additionalProperties to dmiProperties

Issue-ID: CPS-3063

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

Merge "Isolate Delta Report performance test data"

Isolate Delta Report performance test data

- Delta report performance tests shared openroadm anchor with other
  performance tests in CPS.
- The patch removes the use of openroadm anchors for Delta Report
  performance tests.
- Delta Report performance tests now use two independent anchors as
  source and target: delta-report-anchor and openroadm-modified.
- Both the anchors are deleted once the Delta Report tests finish
  executing.
- Removed @Ignore from first Delta Report test to check if there is any
  impact on other CPS performance tests.

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

Fix: update documentation for CPS-2428 notification subscription APIs

Three APIs were added to manage CPS notifications:
- Create notification subscription
- Get notification subscription
- Delete notification subscription

Swagger examples were outdated and did not reflect the correct fields.
Updated documentation to ensure accurate examples for successful API execution.

- Added missing Javadoc in notification subscription class for better clarity.

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

Fix Null Pointer Exception during LCM Event Metric Recording

- Add null check when recording metric for unknown states (initial create and delete use cases)
- Extend Create Integration test to verify the (number of) recording metrics
- Moved Instrumentation bean to integration testbase as it is now used in several test classes
- Update version of docker test container to latest (part of troubleshooting with Docker for windows integration)
- Clean up some legacy issues in related integration testware
  - unnecessary and distracting spacing
  - correct use of #testValue in descriptions

Issue-ID:CPS-3088

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

Merge "Update Patch Operation to make multiple calls to Policy Executor"

Update Patch Operation to make multiple calls to Policy Executor

  - refatored OperationDetailsFactory
  - updated testware

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

Accept single-element FDN in the ProvMnS interface

 - removed extra validation to accept simple FDNs

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

Merge "refactor: Remove LcmEventHeader parameter from sendLcmEvent method"

Merge "Update current expectation for k6 tests #3"

Update current expectation for k6 tests #3

- The new values (current expectation) is adjusted according to avarage
  value of test results after excluding the peaks (while other tests are
also running in parallel)

Issue-ID: CPS-2697

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

Bump lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml

Bumps [lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml](https://github.com/lfit/releng-reusable-workflows) from 0.2.26 to 0.2.28.
- [Release notes](https://github.com/lfit/releng-reusable-workflows/releases)
- [Commits](https://github.com/lfit/releng-reusable-workflows/compare/f8814d81051c7a2e4d7584657b4d283b62a56b20...b4c8bfdea114d034bfa6ed2fe2c9775bc091d5ca)

updated-dependencies:
- dependency-name: lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml
  dependency-version: 0.2.28
  dependency-type: direct:production
  update-type: version-update:semver-patch

CIMAN-33

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

Update current expectation (cps limit) for k6 tests #2

- More fine tuninng on current expectations

Issue-ID: CPS-2697

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

refactor: Remove LcmEventHeader parameter from sendLcmEvent method

- Remove LcmEventHeader parameter from LcmEventsProducer.sendLcmEvent()
- Extract headers directly from LcmEvent object using new extractHeadersAsMap() method
- Remove json definition of now unnecessary lcm event header object
- Fix bug(?) related to recording metrics when sending event failed
- Removed unnecessary null checks for status during metric record
- Refactored LcmEventHelper, LcmEventProducer and LcmEventProducerHelper into 3 classes with clear responsibilities
  LcmEventProducer               : can send events (depends on LcmObjectCreator)
  LcmEventObjectCreator          : create events(depends on CmHandlePropertyChangeDetector)
  CmHandlePropertyChangeDetector : detects the updates
- Cleaned up corresponding test classes
- Renamed cps/events/EventsProducer.java to cps/events/EventProducer (singular, consistent with other classes)
  (apologies, this caused updates for 3/4 of the files in this commit)
- Renamed dmi/EventProducer to dmi/DmiEventProducer to avoid clashes with main EventProducer (in some testware)

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

Merge "Fix Replace a Node API behaviour handling List items"

Merge "Added k8 specific tests for second dmi-stub in postman-collections"

Merge "Update current expectation (cps limit) for k6 tests"

Update current expectation (cps limit) for k6 tests

- After k6 started to use k8s cluster, current test expectations have
  been improved for some k6 tests, thus values updated

Issue-ID: CPS-2697

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

Fix Replace a Node API behaviour handling List items

Jira - [CPS-2708] Enhance Replace A Node with List Items
Documentation - https://lf-onap.atlassian.net/wiki/x/gIDMAQ

- Previously, Replace a node endpoint accepted list items and returned a
  200 OK response, but the data was not actually replaced when cross
  checked.
- This patch fixes the issue of phantom replace operations and ensures
  that the API correctly handles list items.

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

Merge "Fix response code returned by Get delta between 2 anchors API"

Merge "Integration test for state migration job"

Fix response code returned by Get delta between 2 anchors API

- Get delta between 2 anchors returned a 200 OK response in case an
  invalid xpath is provided with the request.
- The patch fixes that and now the API returns 400 BAD RERQUEST when an
  invalid xpath is used

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

Integration test for state migration job

- Added DataMigrationIntegrationSpec test class
- Changed migrateInventoryToModelRelease20250722 signature to take in batch size
- Changed InventoryModelLoaderSpec to call upgradeAndMigrateInventoryModel() directly
- Added schemaSetExists() check in ModuleUpgradeServiceIntegrationSpec to allow for suite and isolated execution

Issue-ID: CPS-3061

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

Merge "Move ConditionProperties from CPS to NCMP"

Move ConditionProperties from CPS to NCMP

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

Expand Integration test to verify LCM Event

- Fixed small bug: EventId and EventTime in header and event should be identical
- Check LCM event details for common fields
- Check Header value and corresponding event fields (duplicated!)
- Check old and new value for create and update use cases
- Removed unessary public method

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

Merge "Limit number of operations in a ProvMnS PATCH request"

Limit number of operations in a ProvMnS PATCH request

-New configurable stting in application.yml

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

Added k8 specific tests for second dmi-stub in postman-collections

- added new env. variable suit for k8s deployment
- new tests are added for k8s, all using the new env. suit:
  - cm handle registration for dmi-stub-1 and dmi-stub-2 each having two
    cm handles
  - cm handle get for dmi-stub-1 and dmi-stub-2
  - get resource data operation
  - delete cm handles for both dmis

Issue-ID: CPS-3039

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

Error Handling for unsupported ProvMnS PATCH operations

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

Merge "Migrate state data to new cm-handles.cm-handle-state"

Migrate state data to new cm-handles.cm-handle-state

- Added new DataMigration class to perform migration
- Amended InventoryModelLoader to use migration class
- Moved setAndUpdateCmHandleField to InventoryPersistence and made public
- Created bulk implementation of setAndUpdateCmHandleField

Issue-ID: CPS-3023

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

Merge "Improve mapping for Resource objects in PolicyExecutor"

Merge "Bump to version 3.7.4"

Bump to version 3.7.4

Issue-ID: CPS-3047

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

Merge "Add container 3.7.3"

Add container 3.7.3

Issue-ID: CPS-3047

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

Merge "Add maven stage yaml for release 3.7.3"

Add maven stage yaml for release 3.7.3

Issue-ID: CPS-3047

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

Merge "ProvMnS Get to support additional type in return"

Improve mapping for Resource objects in PolicyExecutor

  - added ResourceObjectDetails for improved mapping
  - added OperationDetailsFactory service and moved related code there from PolicyExecutor

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

Merge "Fix Sonarcube violations"

Revert commit (ncmp): for Connection refused exp. in DmiRestClient #2

- This is a manual revert for the commit : 142464: changed env. variable server.port for dmi-stub | https://gerrit.onap.org/r/c/cps/ncmp-dmi-plugin/+/142464
- There have been k6 failures after build # : 2637

Issue-ID: CPS-3056

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

Merge "Code Coverage gaps in CPS-NCMP-Service 100%"

ProvMnS Get to support additional type in return

- application/vnd.3gpp.object-tree-hierarchical+json now supported

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

Fix Sonarcube violations

- Extracted NOT READY to static variable
- Restructured Try catch blocks in controller
- Resolved minor code smells

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

Merge "changed env. variable to SERVER_PORT for pod dmi-stub-2"

Merge "Bump lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml"

Code Coverage gaps in CPS-NCMP-Service 100%

- Covered last 3 remaining gaps
- Removed Coverage setting from CPS NCMP Service module pom so its set by parent to 100% 😊
- Removed Coverage setting from CPS NCMP Rest module pom so its set by parent to 100% 😊

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

Adding @Ignore on delta performance tests while analysis

- Reverted the following:
- CPS-3031: https://gerrit.onap.org/r/c/cps/+/142353
- CPS-3032: https://gerrit.onap.org/r/c/cps/+/142359
- CPS-3033: https://gerrit.onap.org/r/c/cps/+/142363

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

Merge "Filling Code Coverage gaps in CPS-NCMP-Service"

Filling Code Coverage gaps in CPS-NCMP-Service

- Removed unused variation of ProvMnS exception
- Cover error scenarios on Inventory Facade Impl
- Refactored LCM Event sending to always use (batch) of CmHandleTransitionPair
  (this meant I could avoid a different variations that was hard to test, and simply removed redundant duplicated code
- Add tested for simple data object no used in any current test
- Added some additional (typically negative) scenarios in existing test
- Change LCM Event related methods signatures to use term 'current' and 'target' and in that order consistently
- Refactored (simplified) Lcm Event Helper had a lot of unnecessary method splitting ending up in bad names like populateX & create
- Simplified arg[] handle and some other groovy improvement in related tests

Issue-ID:CPS-475

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

Bump lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml

Bumps [lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml](https://github.com/lfit/releng-reusable-workflows) from 0.2.25 to 0.2.26.
- [Release notes](https://github.com/lfit/releng-reusable-workflows/releases)
- [Commits](https://github.com/lfit/releng-reusable-workflows/compare/21dee35da5cf13331dad63a2c81ae5bb6d023f95...f8814d81051c7a2e4d7584657b4d283b62a56b20)

updated-dependencies:
- dependency-name: lfit/releng-reusable-workflows/.github/workflows/reuse-openssf-scorecard.yaml
  dependency-version: 0.2.26
  dependency-type: direct:production
  update-type: version-update:semver-patch

CIMAN-33

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

changed env. variable to SERVER_PORT for pod dmi-stub-2

- minikube can not bind env. variable if it was DMI_STUB_SERVER_PORT for
  the pod dmi-stub-2, thus it changed to SERVER_PORT to solve the
problem

Issue-ID: CPS-3056

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

Delete cmSubscriptionConfigSpec groovy file

-Pointed to deleted java file

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

Merge "Bug: ProvMnS error handling returning all DMI error responses as 500"

Merge "Fix casting issue for Patch operation"

Merge "Add tests for coverage in subscription files"

Add tests for coverage in subscription files

- added tests to cover lines in subscription related classes
- removed unused subscription cache (legacy)

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

Fix casting issue for Patch operation

  - fixed casting issue for Resource
  - refactored mapping logic for patch item

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

Improve code coverage CPS-NCMP-Service

- Increase coverage threshold cps-ncmp-service to 99%
- Use real configuration classes for DMI Client Integration Test
  (fills big coverage gap in WebClientConfiguration)
- Filled testing gaps in CM AVC Events package
  (partly by removing redundant code!)
- Filled testing gaps in hazelcast cache configs
- Added missing test for Inventory Persistence service

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

Merge "Add Rejected scenario(s) for create subscription"

Add Rejected scenario(s) for create subscription

- added consuming REJECTED create request from DMI
- when REJECTED request is consumed, should update database and
  log details (dataNodeSelector, dataJobId, dmi details)
- added ignoring when dataJobId is not unique with test

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

Merge "added second dmi-stub to cps-charts"

Bug: ProvMnS error handling returning all DMI error responses as 500

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

Fix bug related to incorrect exception check when saving data nodes

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

Merge "Forwarding list parameters : attributes and field consistently"

Task 3: Get delta between an anchor and JSON payload with grouping enabled

Remove @Ignore from the third Delta Report performance test (Get delta between
an anchor and JSON payload with grouping enabled) Jenkins build remains stable
or fails due to this test.

Purpose:
This change is part of an investigation to identify which Delta Report performance
test is causing the build failures.

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

Forwarding list parameters : attributes and field consistently

- Treat 'fields' same as 'attributes' i.e. support empty value
- Refactored code (removed unnecessary null-check)
- Add test for all scenarios in Controller for best readability

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

Merge "Bug: Forward Empty attributes to DMI in Provmns GET"

Bug: Forward Empty attributes to DMI in Provmns GET

- Modified RestServiceUrlTemplateBuilder to accept empty attributes
- Updated test ware to reflect change

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

Merge "Replace sleep with polling in CmSubscriptionSpec"

Replace sleep with polling in CmSubscriptionSpec

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

Merge "Update Policy Executor mapping for ProvMnS inputs"

Update Policy Executor mapping for ProvMnS inputs

  - update mapping for Policy Executor for FDNs with a #
  - added extra testware

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

Merge "Implement PATCH for ProvMnS interface with bug fixes"

added second dmi-stub to cps-charts

- cps-charts should support two dmi-stub instances, each uses its own
  pod. Since each pod should return its own Servlet URI (dmi-stub
address)
- the services for dmi-stub instances should be defined seperately since
  a fixed dmi-stub address is required. (otherwise adress would be
dynamic on each deployment)
- env. variable DMI_STUB_SERVER_PORT is used to override ther server
  port for second dmi-stub (ports should be different on pod levels)
- for the latest architecture please visit the solution page:
   https://lf-onap.atlassian.net/wiki/spaces/DW/pages/347537434/CPS-2933+Kubernetes+container+management+system+for+CPS+and+NCMP
- updated Uri address for dmi stub services

Issue-ID: CPS-2934

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

Implement PATCH for ProvMnS interface with bug fixes

-Implement PATCH endpoint with error handling
-Missing # attribute update operation; will come in later patch
-Fixes error return bug where all errors defaulted to 500
-Fixes get call to dmi in put operation
-Moved Resource to body in fowarding

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

Fix SonarQube issues

- refactored cmDataJobSubscriptionPersistenceService to get URI
  from a customizable parameter
- defined constants for 'dataNodeSelector' , 'dataJobId' and 'status' leaf names

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

Merge "Change Implement Exactly Once Semantics logging change back to debug level"

Merge "Task 2: Merge Get delta between 2 anchors with grouping disabled"

Change Implement Exactly Once Semantics logging change back to debug level

Issue-ID:CPS-2984

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