leventecsanyi [Wed, 17 Jul 2024 14:48:15 +0000 (16:48 +0200)]
Fix dataProducerId in SubJobWriteRequest
Issue-ID: CPS-2322
Change-Id: I7bfdfd05ffacb085ac056707d477f348cb5bd64f
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
mpriyank [Wed, 24 Jul 2024 08:30:14 +0000 (09:30 +0100)]
Disable NCMP CSITs
- comment the CSITs temporarily until the SDNC certs are corrected.
Issue-ID: CPS-2325
Change-Id: I335b17e4f148ff838301fb511656da1068d124b4
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
Daniel Hanrahan [Fri, 19 Jul 2024 10:02:29 +0000 (10:02 +0000)]
Merge "Integration test to validate DMI service URL"
danielhanrahan [Tue, 16 Jul 2024 10:47:24 +0000 (11:47 +0100)]
[k6] Measure NCMP overhead for passthrough operations
Measure NCMP overhead by subtracting DMI delay from request duration.
Issue-ID: CPS-2269
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ieb4758e9f7cab119da4c28e4a3febfa1361b2613
sourabh_sourabh [Tue, 16 Jul 2024 16:21:45 +0000 (17:21 +0100)]
Integration test to validate DMI service URL
- New groovy test is written to validate if dmi service url is encoded
correctly.
Issue-ID: CPS-2317
Change-Id: I048bf37dd3ba4f37df7dace927f55bf8dd899922
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Daniel Hanrahan [Wed, 17 Jul 2024 12:10:47 +0000 (12:10 +0000)]
Merge "CPS NCMP: Add state tags to cps.ncmp.lcm.events.publish metrics"
Daniel Hanrahan [Tue, 16 Jul 2024 19:26:48 +0000 (19:26 +0000)]
Merge "XML content support on replace a node with descendants"
Daniel Hanrahan [Tue, 16 Jul 2024 13:57:32 +0000 (13:57 +0000)]
Merge "Report throughput for passthrough read operation"
halil.cakal [Tue, 16 Jul 2024 12:53:05 +0000 (13:53 +0100)]
Separate dmi delay for passthrough operations
- set dmi delay to 670 ms for passthrough write (CUD)
- set dmi delay to 300 ms for passthrough read
Issue-ID:CPS-2285
Change-Id: I7a4294d3ed04c89aeb5988a9975516135317af12
Signed-off-by: halil.cakal <halil.cakal@est.tech>
sourabh_sourabh [Mon, 8 Jul 2024 16:54:11 +0000 (17:54 +0100)]
CPS NCMP: Add state tags to cps.ncmp.lcm.events.publish metrics
- Added cm handle state into micrometer Timer as an extra tags while capturing total time taken to publish a LCM events for publishLcmEvent method.
Issue-ID: CPS-2123
Change-Id: I6357c628496700fee7448dbc381a7e0e62d8b540
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Rudrangi Anupriya [Thu, 11 Jul 2024 16:26:24 +0000 (21:56 +0530)]
XML content support on replace a node with descendants
Issue-ID: CPS-2282
Change-Id: Ibb7ffb65ccbb03703266712c6d5c2eade0e7ab4b
Signed-off-by: Rudrangi Anupriya <ra00745022@techmahindra.com>
halil.cakal [Tue, 9 Jul 2024 16:43:25 +0000 (17:43 +0100)]
Report throughput for passthrough read operation
- change threshold from avg duration to req per second
- the delay for dmi has been hanlded in seprate commit
in dmi repository
- add delays through docker-compose env variables
Issue-ID:CPS-2285
Change-Id: I35978b21c208fdb4209b1610160dc5545ee22965
Signed-off-by: halil.cakal <halil.cakal@est.tech>
Daniel Hanrahan [Tue, 16 Jul 2024 09:24:18 +0000 (09:24 +0000)]
Merge "Add passthrough write (CUD) operations to k6"
mpriyank [Wed, 10 Jul 2024 13:46:46 +0000 (14:46 +0100)]
Mapper to group Subscription Details for DMI
- introduced a mapper to group a collection of subscription
keys(datastore cmhandle and xpath ) based on datastore and xpath
- the information will be used to send out to the dmi plugin
Issue-ID: CPS-2312
Change-Id: I99f509eb5b4a8c3a7fb1078ae86490d894f22b6c
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
mpriyank [Mon, 15 Jul 2024 11:25:15 +0000 (12:25 +0100)]
Bump CPS to 3.5.2 Snapshot
- bump CPS to 3.5.2-SNAPSHOT
- updated the release notes and prepared for the next release
Issue-ID: CPS-2318
Change-Id: I584faf6218e60ad62e32ef8c3cb0deb4047149a2
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
mpriyank [Mon, 15 Jul 2024 10:59:56 +0000 (11:59 +0100)]
container yaml for CPS 3.5.1
- adding container yaml for CPS release 3.5.1
Issue-ID: CPS-2318
Change-Id: Iba39b49c46c30f917e526cdf716b963dc1e139d4
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
mpriyank [Mon, 15 Jul 2024 10:21:57 +0000 (11:21 +0100)]
maven stage yaml for 3.5.1
- adding maven stage yaml for CPS 3.5.1 release
Issue-ID: CPS-2318
Change-Id: I14f4f8a890fbab80497696907fde4fd22601a562
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
Daniel Hanrahan [Mon, 15 Jul 2024 10:05:06 +0000 (10:05 +0000)]
Merge "Release notes CPS 3.5.1"
mpriyank [Mon, 15 Jul 2024 09:48:49 +0000 (10:48 +0100)]
Release notes CPS 3.5.1
- Added release notes and syncd the openapi files.
Issue-ID: CPS-2318
Change-Id: I1982e31f3bcb9337ed9814cb55a4db885b157158
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
danielhanrahan [Fri, 12 Jul 2024 14:05:15 +0000 (15:05 +0100)]
Print DMI stub logs in CSITs
Issue-ID: CPS-2313
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I4ecb5ce1713666b5e90b90eba3c98b737fd91896
halil.cakal [Thu, 11 Jul 2024 13:54:57 +0000 (14:54 +0100)]
Add passthrough write (CUD) operations to k6
- the delay for dmi has been hanlded in seprate commit
- for simplicity only the POST request is considered
Issue-ID: CPS-2267
Change-Id: I62ede04dd8e2744fc723566742396df31c42ea5d
Signed-off-by: halil.cakal <halil.cakal@est.tech>
danielhanrahan [Fri, 12 Jul 2024 10:58:40 +0000 (11:58 +0100)]
Print docker logs in CSITs for 2 cps-and-ncmp instances
Following introduction of 2 instances of cps-and-ncmp,
CSIT logs show an error "No such container: cps-and-ncmp".
This commit prints docker logs for each cps-and-ncmp container.
Issue-ID: CPS-2038
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: If3ed4590cef0f5b59d74e62c84f6c6e72edfd411
Sourabh Sourabh [Thu, 11 Jul 2024 16:11:34 +0000 (16:11 +0000)]
Merge "Policy Executor Feature Toggle"
ToineSiebelink [Wed, 10 Jul 2024 16:34:13 +0000 (17:34 +0100)]
Policy Executor Feature Toggle
- defined config parameters for feature toggle and server details
- log request details when feature enabled
- improved JavaDoc in Controller
- improved configuration properties checks in HttpClientConfigurationSpec
Issue-ID: CPS-2311
Change-Id: I1ff4bd3592ce2570ac74f9ec6e62b75001cb611a
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Priyank Maheshwari [Thu, 11 Jul 2024 13:29:59 +0000 (13:29 +0000)]
Merge "Fix timing issue in CmHandleCreateSpec"
danielhanrahan [Thu, 11 Jul 2024 11:16:43 +0000 (12:16 +0100)]
Fix timing issue in CmHandleCreateSpec
The test for module sync retry on LOCKED handles has a timing issue,
due the ordering of the steps. After a LOCKED handle gets reset to
ADVISED state (but DMI is not yet set to available), the Module Sync
Watchdog may wake up and put it back into LOCKED state again.
This fix is to first make the DMI available before waiting for the
LOCKED handle to reset.
Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I9d33b0eeea1e0a2e163489130c928b4d03182774
sourabh_sourabh [Thu, 11 Jul 2024 09:45:45 +0000 (10:45 +0100)]
Documentation updated in docs/ncmp-inventory-querying.rst
- Existing API docs in docs/ncmp-cmhandle-querying.rst is updated where
CPS Path is now supported as ancestor axis.
Issue-ID: CPS-2289
Change-Id: I065179fe7c294588580448a8467765e9d116d33a
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Toine Siebelink [Wed, 10 Jul 2024 16:43:03 +0000 (16:43 +0000)]
Merge "NCMP inventory cm handle search should support cps path query"
halil.cakal [Wed, 10 Jul 2024 13:58:26 +0000 (14:58 +0100)]
Remove nginx logs that not being used anywhere
Issue-ID: CPS-2038
Change-Id: Iba8950db4f3fcfc13f32bf915189f5fcb2ccee90
Signed-off-by: halil.cakal <halil.cakal@est.tech>
sourabh_sourabh [Wed, 10 Jul 2024 11:51:31 +0000 (12:51 +0100)]
NCMP inventory cm handle search should support cps path query
- Added cps path query conditions for inventory search api
Issue-ID: CPS-2289
Change-Id: I0bcc9a45af65ad9d41c4d4cf79b35bfb95f50cff
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
danielhanrahan [Tue, 9 Jul 2024 11:06:20 +0000 (12:06 +0100)]
Split integration tests into cps and ncmp packages
Issue-ID: CPS-2309
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ie97c1430d1121c4c68b3fc6dacbc2466d48e1017
ToineSiebelink [Wed, 10 Jul 2024 08:24:22 +0000 (09:24 +0100)]
Policy Executor API, make Authorization header optional
Issue-ID: CPS-22291
Change-Id: I2b8f9b97c92dfc640c8b40b7053c609bbdcb2f6e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
sourabh_sourabh [Mon, 8 Jul 2024 16:40:38 +0000 (17:40 +0100)]
NCMP: NCMP search API fixed to handle cps path that is ancestor axis as
well
- Cps path builder is fixed to handle ancestor axis that is target as well while quering.
Issue-ID: CPS-2308
Change-Id: Iaf215851ada17d21516ae83fa142ac77ff1c6c19
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
ToineSiebelink [Tue, 9 Jul 2024 11:41:59 +0000 (12:41 +0100)]
Refactor remaining classes package structure
-Introduced org.onap.cps.ncmp.config for all config type classes like http, kafka etc.
-Introduced org.onap.cps.ncmp.impl.dmi for dmi specif classes like DmiRestClient etc.
-Moved most exception classes to org.onap.cps.ncmp.api.exceptions
(only two were 'internal' i.e. NOT used in REST so left in org.onap.cps.ncmp.exceptions)
-Deleted unused DateNodeHelper class (and related testware)
-Moved some classes to org.onap.cps.ncmp.impl.utils
-Moved messaging test base classes to org.onap.cps.ncmp.impl.utils.events
-All imports were reorganised automatically by IntelliJ
-did NOT update any copyright dates (seems incorrect to me anyway)
Issue-ID: CPS-2257
Change-Id: Ie28b1777ea278064b46c6a1c3a28f4d0696565e2
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
danielhanrahan [Tue, 9 Jul 2024 14:21:25 +0000 (15:21 +0100)]
Increase module sync wait time in tests to 10 seconds
Too small timeout is causing intermittant test failures.
Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ib6a25cb2f44e9a65b8f7d2d98e8d5e7dfe1f9ceb
Sourabh Sourabh [Tue, 9 Jul 2024 13:30:26 +0000 (13:30 +0000)]
Merge "Create PolicyExecutor Stub"
Daniel Hanrahan [Tue, 9 Jul 2024 12:31:10 +0000 (12:31 +0000)]
Merge "Setup automated tests with two replicas of cps-and-ncmp"
Sourabh Sourabh [Tue, 9 Jul 2024 09:38:47 +0000 (09:38 +0000)]
Merge "[k6] Measure CM-handle (de)registration in CM-handles/sec"
Lee Anjella Macabuhay [Tue, 9 Jul 2024 09:24:08 +0000 (09:24 +0000)]
Merge "Remove forced module sync from NCMP integration tests"
Lee Anjella Macabuhay [Tue, 9 Jul 2024 09:23:51 +0000 (09:23 +0000)]
Merge "Refactor integration tests so only one DmiDispatcher is used"
ToineSiebelink [Thu, 4 Jul 2024 14:15:36 +0000 (15:15 +0100)]
Create PolicyExecutor Stub
- Generate interface from OpenApi in RTD docs
- Fixed Content Type in OpenAPI
- Fixed Paths in OpenAPI
- Made Authorization header compulsory in OpenAPI
- All 'enum' values in OpenAPI lowercase (in linr with CPS/NCMP conventions)
- Added impl with some basic functionality
- Added testware
- Pom includes docker image creation
- Docker compose updated to deploy stub
Issue-ID: CPS-2301
Change-Id: I462ad5c70474b2813fc04005c0d20a1b15b574ec
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Lee Anjella Macabuhay [Mon, 8 Jul 2024 14:06:48 +0000 (14:06 +0000)]
Merge "Remove Mappers and Events Facade"
danielhanrahan [Fri, 5 Jul 2024 12:20:31 +0000 (13:20 +0100)]
[BUG] Fix handling of special characters in moduleSetTag
During CM-handle registration, moduleSetTag is sent as part of the
JSON request body when getting modules from DMI. However, it
incorrectly parses the moduleSetTag as JSON, instead of encoding to
JSON. This results in parsing errors when moduleSetTag includes
characters such as ':'
Issue-ID: CPS-2302
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ie38008e0effac2423a4d069a91587e47d8747d66
mpriyank [Fri, 5 Jul 2024 11:33:06 +0000 (12:33 +0100)]
Remove Mappers and Events Facade
- Removing Mappers and Events Facade and introducing the normal classes
that this facade was wrapping
- Corresponding changes in the testware
Issue-ID: CPS-2298
Change-Id: I380d73ff3a9b3bf79633cd91580d6a4dee00a662
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
danielhanrahan [Fri, 28 Jun 2024 12:43:35 +0000 (13:43 +0100)]
[k6] Measure CM-handle (de)registration in CM-handles/sec
As per characteristics requirements document:
- measure registration in CM-handles/second
- measure deregistration in CM-handles/second
- summary table includes test case number, description,
units of measurement, actual value and limit.
Issue-ID: CPS-2269
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I838004da1c230ab722f49c2adacf34e730d7ac79
Daniel Hanrahan [Fri, 5 Jul 2024 10:23:48 +0000 (10:23 +0000)]
Merge "refactor cmsubscription code"
danielhanrahan [Thu, 4 Jul 2024 17:01:28 +0000 (18:01 +0100)]
Remove forced module sync from NCMP integration tests
- remove forced module sync in tests
- set module sync timers to 1 second in test application.yml
- have tests wait up to 2 seconds for module sync to run
Issue-ID: CPS-2304
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I727894dc1b4b9cee1ab82d05a9c31d1e3e067cf3
danielhanrahan [Thu, 4 Jul 2024 16:03:44 +0000 (17:03 +0100)]
Refactor integration tests so only one DmiDispatcher is used
Issue-ID: CPS-2303
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I4d04fb9b7693a769e9059d3b2b7257a714da0532
mpriyank [Mon, 1 Jul 2024 11:32:27 +0000 (12:32 +0100)]
refactor cmsubscription code
- Moved cmsubscription to the impl package instead of api.
- Below packages are renamed or newly created for code and tests
- moved ..api.cmsubsription to ..impl.cmsubscription
- renamed ..api.cmsubscription.mappers to ..impl.cmsubscription.ncmp
or dmi as per the functionality
- renamed ..api.cmsubscription.model to ..impl.cmsubscription.models
- removed ..api.cmsubscription.producer/consumer and classes moved to ..impl.cmsubscription.ncmp/dmi
- new package ..impl.cmsubscription.cache to hold cache config and
cache operations
- new package ..impl.cmsubscription.avc to hold the CmAvcEventConsumer
for the notifications emitted as a result of the subscriptions
- Removed the prefix CmNotificationSubscription for majority of the classes , not removed for Config and model classes
- Renamed the schemas and package structure as per the code refactoring
Note: *EventsFacade and *MappersFacade to be removed in the next
patchset
Issue-ID: CPS-2298
Change-Id: I1d788ab745d65965570e28beaefa511cbe4a8547
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
sourabh_sourabh [Thu, 4 Jul 2024 08:46:24 +0000 (09:46 +0100)]
Release notes for Web Client prometheus metrics
- Release notes updated as we enabled http client prometheus metrics.
Issue-ID: CPS-2121
Change-Id: I6918da7cd4e4523c1ec0ab6060bdc1686c141a9d
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Sourabh Sourabh [Thu, 4 Jul 2024 08:34:42 +0000 (08:34 +0000)]
Merge "Add Policy Executor Open API Interface definition"
Toine Siebelink [Wed, 3 Jul 2024 16:57:48 +0000 (16:57 +0000)]
Merge " CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url"
sourabh_sourabh [Thu, 20 Jun 2024 16:48:47 +0000 (17:48 +0100)]
CPS NCMP: Resolved high cardinality of prometheus metrics for dmi service url
- Used autoconfigured web client builder for http_client_requests_* prometheus metrics.
- Refactored dmi service url builder to create url template and its
variables.
- Web client is modified to use uri(urlTemplate, urlvars) version.
- Deleted InvalidDmiResourceUrlException that no longer needed.
- Used DmiServiceUrlBuilder to build dmi health check service url.
- Created a new pkg url.builder into utils to have all related classes
and record.
Issue-ID: CPS-2121
Change-Id: Id67e0f0d4e640bb8f9eea0b6c2db1dba3468e1d7
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
ToineSiebelink [Mon, 1 Jul 2024 16:50:54 +0000 (17:50 +0100)]
Add Policy Executor Open API Interface definition
- OPenApi in RTD only (for now)
- Bear authentication included in OpenApi
- Placeholder RTD page (to be completed upon feature delivery)
- For consistency (and color coding in editors) I put all Description and Example strings inside double qoutes "..."
Issue-ID: CPS-2291
Change-Id: I7f16cdce2a06cb011997005fa8b86b54c044304a
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
danielhanrahan [Wed, 3 Jul 2024 14:41:40 +0000 (15:41 +0100)]
Fix sonar issue in performance test ResourceMeter
Replace busy loop with await().until as suggested by sonar
Issue-ID: CPS-89
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ic6c870c419a67708082ceb05ef3ccba884ae375e
halil.cakal [Mon, 17 Jun 2024 08:24:59 +0000 (09:24 +0100)]
Setup automated tests with two replicas of cps-and-ncmp
- this change for two test suits: CSIT and K6
- increase cps-and-ncmp replicas to 2
- introduce nginx as load-balancer
- set proxies inside nginx for the cps-and-ncmp URI paths
- upgrade hazelcast-spring to 5.3.7
Issue-ID: CPS-2038
Change-Id: If1b6c1e1218af16b24cc061507ea9b25d277d143
Signed-off-by: halil.cakal <halil.cakal@est.tech>
Sourabh Sourabh [Mon, 1 Jul 2024 14:08:27 +0000 (14:08 +0000)]
Merge "Improve handling of Invalid Operation exception"
Daniel Hanrahan [Mon, 1 Jul 2024 13:52:29 +0000 (13:52 +0000)]
Merge "Fix incorrect use of getAnchors(dataspace, schemasets)"
ToineSiebelink [Mon, 1 Jul 2024 10:24:49 +0000 (11:24 +0100)]
Improve handling of Invalid Operation exception
- MOve to API package
- Handle in REST Controller to return BAD REQUEST
Issue-ID: CPS-2256
Change-Id: I4f96519aaf2e7f781729c8b8671ea02630a2e0a8
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
ToineSiebelink [Thu, 27 Jun 2024 11:49:27 +0000 (12:49 +0100)]
repackage 'data' feature
- moved relevant classes into these (new) packages:
api.data.exceptions
api.data.models
impl.data
impl.data.async
impl.data.exceptions
impl.data.models
impl.data.utils
utils.events
- removed old unused event class
- moves some missed inventory related class to the right place
Issue-ID: CPS-2256
Change-Id: I75563e063acc0054769d8f2b13146e6c1d1c6054
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
ToineSiebelink [Thu, 27 Jun 2024 14:35:37 +0000 (15:35 +0100)]
move lcm/common events classes
-moved api.impl.events.lcm to impl.inventory.sync.lcm
-moved most api.impl.events class to utils.events
(remaining classes in api.impl.events belong to cmsubscrption and Priyank wil take care of those)
Issue-ID: CPS-2258
Change-Id: I1818194e708aa0d9c713b470e878997c2f26fb13
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Priyank Maheshwari [Thu, 27 Jun 2024 11:02:43 +0000 (11:02 +0000)]
Merge "Simplify package structure cps-ncmp-rest"
ToineSiebelink [Thu, 27 Jun 2024 08:06:31 +0000 (09:06 +0100)]
Simplify package structure cps-ncmp-rest
Issue-ID: CPS-2294
Change-Id: Idc3605c1b877be13a8d344875612da6bef791fc7
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Priyank Maheshwari [Wed, 26 Jun 2024 14:33:36 +0000 (14:33 +0000)]
Merge "Move TrustLevel Feature"
ToineSiebelink [Tue, 25 Jun 2024 13:40:40 +0000 (14:40 +0100)]
Move TrustLevel Feature
-Moved TrustLevel impl from Controller to Inventory Facade
-Applied new package name convention using impl\inventory\trustlevel for this (sub)feature
-Renamed some trustlevel related classes to better describe their role
-Disabled failing architecture test (agreed this is tech. debt and created CPS-2293)
-Refactored YangDataConverter method names and signatures (moved Spec to correct place)
-Cleaned up names and javadoc of class now called DeviceTrustLevelMessageConsumer
-Applied conventions to modified test methods
Issue-ID: CPS-2255
Change-Id: Iccd20541488e6b61444bb2c7406c90e61a34124d
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Sourabh Sourabh [Wed, 26 Jun 2024 11:36:00 +0000 (11:36 +0000)]
Merge "Disable the java debug port permanently"
halil.cakal [Wed, 26 Jun 2024 11:03:38 +0000 (12:03 +0100)]
Disable the java debug port permanently
Issue-ID: CPS-1864
Change-Id: I0366b126920cf0bc706545bc5b4e469d3faa2486
Signed-off-by: halil.cakal <halil.cakal@est.tech>
Daniel Hanrahan [Wed, 26 Jun 2024 08:09:46 +0000 (08:09 +0000)]
Merge "Clean up the k6 test suite"
danielhanrahan [Fri, 21 Jun 2024 10:12:45 +0000 (11:12 +0100)]
Fix incorrect use of getAnchors(dataspace, schemasets)
There is an issue in a method deleteDataNodes(dataspace, anchors)
where it doesn't send data update events. This is because it fetches
anchors using a method getAnchors(dataspace, schemaSetNames), when it
needs to fetch anchors by name: getAnchors(dataspace, anchorNames)
Changes:
- Rename getAnchors method using schemasets to getAnchorsBySchemaSetNames
- Add a method getAnchors(dataspace, anchorNames)
- Update test of deleteDataNodes method using getAnchors so it checks
that data update events are sent
Issue-ID: CPS-2254
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: I55fe853f0a9278a66a5724bf4cf2723b0e2fbc8b
ToineSiebelink [Mon, 24 Jun 2024 12:24:57 +0000 (13:24 +0100)]
Repackage Inventory Feature
- decided that cm handle queries is part of 'inventory' feature
- a lot of files (~100 total) have changes because of some files have been moved.
- The reviewers should probably just focus on the MOVED/renamed (~30) files
- New packages (top level only):
-- org.onap.cps.ncmp.api.inventory
-- org.onap.cps.ncmp.impl.inventory
-- org.onap.cps.ncmp.impl.models (common models NOT used in API, this was unforeseen)
-- org.onap.cps.ncmp.impl.utils
- I am NOT updating copyright years as part of this commit since there are no real code changes and it is simply NOT worth the effort
TODO:
- extend scope to 'TrustLevel' as an 'internal feature' (TBD)
Issue-ID: CPS-2255
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I40b4efc8326068ab9ae1f31b77f46b2fe8579431
halil.cakal [Fri, 21 Jun 2024 14:22:57 +0000 (15:22 +0100)]
Clean up the k6 test suite
- Remove the tests that are not in the Characteristics
- Set .625 seconds/operation for cm handle id search with module filter
- Set 13 seconds/operation for cm handle search with module filter
Issue-ID: CPS-2286
Change-Id: I88d5773abadb29ce46914029332384ae8e55e032
Signed-off-by: halil.cakal <halil.cakal@est.tech>
danielhanrahan [Fri, 21 Jun 2024 10:49:48 +0000 (11:49 +0100)]
Fix compile issue after facade introduction
Due to automatic rebase, a method got renamed in one commit but not
the other.
Issue-ID: CPS-2263
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Id0fa8eba79cb98bb24419bfddc0c9b9e8c9a42d3
Sourabh Sourabh [Fri, 21 Jun 2024 09:28:37 +0000 (09:28 +0000)]
Merge "Introducing NCMP Facades"
Sourabh Sourabh [Fri, 21 Jun 2024 08:50:46 +0000 (08:50 +0000)]
Merge "Spliting a data-job into sub-jobs for DMI Plugin"
ToineSiebelink [Thu, 13 Jun 2024 15:52:45 +0000 (16:52 +0100)]
Introducing NCMP Facades
- NetworkCmProxyDataService was polluted with many non-data operations
- it is now split into:
1) NetworkCmProxyFacade, a single thin facade for the (main) controler to redirect calls to the correct services
2) CmHandleRegistrationService(Impl), methods related to registration only
- introduced NetworkCmProxyInventoryFacade for the invenoty controller
- renamed some services for consitency and clarification
- Use facade to acces ncmp data request handlers (instead of direct from controller)
- remove unnecesarry wrappings between request handlers and facade
- split facades according to names: data & inventory
(the REST controllers are not split properly so I think one rest controller will end up needing both facades)
Issue-ID: CPS-2263
Change-Id: I250732aa16ec28b43ff642d2adf10ba36f67290e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
mpriyank [Thu, 20 Jun 2024 14:22:14 +0000 (15:22 +0100)]
Bump CPS version to 3.5.1-SNAPSHOT
- bump cps version to 3.5.1-SNAPSHOT
- updated release notes to make it ready for the next release
Issue-ID: CPS-2283
Change-Id: I242cae7c95a87c1a9e225cce3eb7b36097f1e5ba
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
mpriyank [Thu, 20 Jun 2024 13:32:31 +0000 (14:32 +0100)]
Maven container yaml for release
- added maven container yaml to release cps 3.5.0 version
Issue-ID: CPS-2283
Change-Id: I45ce5b440d8ca2f6ff3999971202f9fc3bd4a8fa
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
leventecsanyi [Mon, 17 Jun 2024 09:11:24 +0000 (11:11 +0200)]
Spliting a data-job into sub-jobs for DMI Plugin
- algorithm for create sub-job requests
- added new method to DmiServiceUrlBuilder to get the write job url
- created WriteOperationExaminer, DmiSubJobClient & testware
Issue-ID: CPS-2142
Change-Id: I258d334ef346cd388341a1deb4078d24d8bdb7cc
Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
mpriyank [Thu, 20 Jun 2024 13:00:56 +0000 (14:00 +0100)]
Maven yaml for release
- added maven yaml file to release cps 3.5.0
Issue-ID: CPS-2283
Change-Id: Ifc9cc9f33381d64bf573593949cf59303ff8d0fc
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
mpriyank [Thu, 20 Jun 2024 11:59:02 +0000 (12:59 +0100)]
Release notes and OpenApi sync
- Release notes updated and openapi syncd for CPS-Core and NCMP
Issue-ID: CPS-2283
Change-Id: Iafdebc65cbcec01bf0d6e6963bd9601fea5f5081
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
sourabh_sourabh [Wed, 19 Jun 2024 13:35:30 +0000 (14:35 +0100)]
Addressed an adaptability issue
- Defined a constant instead of duplicating literal "No data nodes 4 times.
Issue-ID: CPS-478
Change-Id: I2b980a228217e85424342d4c2b9e97c5cfa618d1
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
sourabh_sourabh [Mon, 17 Jun 2024 18:45:00 +0000 (19:45 +0100)]
Handle high frequency (faster then DMI) of passthrough request in NCMP
- Added pendingAcquireMaxCount property into appliaction.yml.
- Added findbugs google annotations dependency.
- Supressed "BC_UNCONFIRMED_CAST_OF_RETURN_VALUE" spotbugs low error for
reactor.netty.resources.ConnectionProvider.
Issue-ID: CPS-2262
Change-Id: Ie755e40282473933f2052fbe7654e7090bb9b337
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Priyank Maheshwari [Mon, 17 Jun 2024 16:01:24 +0000 (16:01 +0000)]
Merge "[k6] Add JVM warmup phase (#2)"
Priyank Maheshwari [Mon, 17 Jun 2024 13:13:24 +0000 (13:13 +0000)]
Merge "[k6] Refactoring k6 tests (#1)"
ToineSiebelink [Fri, 14 Jun 2024 09:01:56 +0000 (10:01 +0100)]
Refactor and Move NCMP Data Request Handlers
- Simplified NCMP Rest Request Handlers
- Moved responsEntity wrapping to Controller so most handler methods can have clear return types
- Moved NCMP Rest Request Handlers to Service Layer
- Moved related exceptions and utils to Service Layer
- Used Lombok for constructors
- Improved related testware
Issue-ID: CPS-2266
Change-Id: I0025fab1c92e0d613825093b6e4b43dae044c01a
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
danielhanrahan [Wed, 12 Jun 2024 15:50:07 +0000 (16:50 +0100)]
[k6] Add JVM warmup phase (#2)
To account for JVM warmup effect, the k6 test suite is run twice:
initially, the whole test suite is run without recording results.
- Added environment variables to allow setting total CM-handles and
batch size for registration. By reducing total CM-handles during
warm-up, many more operations will be run.
Issue-ID: CPS-2264
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ib1d1e76d31719d788a3e026797faa0a425b320a2
danielhanrahan [Mon, 10 Jun 2024 20:32:12 +0000 (21:32 +0100)]
[k6] Refactoring k6 tests (#1)
This commit moves all common request logic into a common folder.
It is needed to avoid duplication before adding JVM warmup phase.
- move registration-related code into common folder
- move passthrough operations into common folder
Issue-ID: CPS-2208
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: Ia9ebf61d21044b43063bde153f9c526e67d607c8
mpriyank [Fri, 14 Jun 2024 14:02:53 +0000 (15:02 +0100)]
Update ONAP DMI version in NCMP
- update the docker compose file to point to latest version on
onap-dmi-plugin. Same to be used to run the CSITs during build.
Issue-ID: CPS-2271
Change-Id: Ifbb2ee9c9548d54979a04595e19062d9192bfd1d
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
sourabh_sourabh [Tue, 11 Jun 2024 07:56:54 +0000 (08:56 +0100)]
#2: Used async version of web client for single cm handle read operation
- Code change is done to use async web client for single cm handle
read data operation.
- CpsNcmpTaskExecutor class and groovy test code is removed.
- Use of supplier code is removed from cm handle Query service.
Issue-ID:CPS-2259
Change-Id: I2b7a36e8a151a9224937afd876fceb30be6a24ce
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
sourabh_sourabh [Fri, 7 Jun 2024 18:10:45 +0000 (19:10 +0100)]
#1: Used async version of web client for batch read operation
- Exposed async version of post method into dmi rest client.
- Code change is done to use async web client for batch data
operation.
- Use of CpsNcmpTaskExecutor code is removed.
Issue-ID: CPS-2174
Change-Id: I7840fd8c6d9debe42e50c860f9cf39d64274df72
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Lee Anjella Macabuhay [Mon, 10 Jun 2024 08:31:59 +0000 (08:31 +0000)]
Merge "[k6] Refactor k6 tests for CM handle searches"
Lee Anjella Macabuhay [Thu, 6 Jun 2024 20:01:42 +0000 (20:01 +0000)]
Merge "Cm Subscription: Predicates optional now"
danielhanrahan [Thu, 6 Jun 2024 19:43:21 +0000 (20:43 +0100)]
[k6] Refactor k6 tests for CM handle searches
A cleanup of the k6 tests to avoid code duplication
Issue-ID: CPS-2208
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
Change-Id: If1e58ee87300cd28ed72b365d9a86e2709dc6c13
sourabh_sourabh [Mon, 27 May 2024 17:28:10 +0000 (18:28 +0100)]
#1: Dedicated web client instance is assigned for data, model and health services
- Switched web client instance based on dmi service type (like data or model)
- 3 diff. beans are configured for data, model and health dmi service.
- Added configurable properties for data and model.
- Hard coded properties are assigned for health service.
Issue-ID: CPS-2231
Change-Id: I39fb739c07c41430dae43509fe29ece5306b7d71
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
mpriyank [Thu, 6 Jun 2024 10:47:46 +0000 (11:47 +0100)]
Cm Subscription: Predicates optional now
- making the predicates sectional OPTIONAL as the schemas are common for
both create and delete use case and we dont have the need to send in
the predicates information for DELETE use case.
- Delete is done based on the incoming subscriptionId only
Issue-ID: CPS-2252
Change-Id: I43ec480821a6088c5fa161480ce2b29b660a2ba5
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
Arpit Singh [Thu, 7 Sep 2023 11:35:37 +0000 (17:05 +0530)]
CPS Delta API 2: Delta between anchor and payload
- Second API to get Delta between an anchor and JSON payload
- added new API getDeltaByDataspaceAnchorAndPayload
- added controller and service layer methods
getDeltaByDataspaceAnchorAndPayload
- Core Delta algorithm remains same as the first API.
getDeltaByDataspaceAnchorAndPayload will call getDeltaBetweenDataNodes
Issue-ID: CPS-1836
Signed-off-by: Arpit Singh <as00745003@techmahindra.com>
Change-Id: Id74cd930ce48e5cb414aa62c5381b79675788a37
Toine Siebelink [Thu, 6 Jun 2024 08:16:39 +0000 (08:16 +0000)]
Merge "Change ApplicationEvent for ModelLoading"
Rudrangi Anupriya [Wed, 5 Jun 2024 18:25:14 +0000 (23:55 +0530)]
XML content support on update node leaves
Issue-ID: CPS-2071
Change-Id: Ibe7f59fbfcbb03703626132c6d5c2afde0e7ab4b
Signed-off-by: Rudrangi Anupriya <ra00745022@techmahindra.com>
mpriyank [Tue, 7 May 2024 12:57:48 +0000 (13:57 +0100)]
Change ApplicationEvent for ModelLoading
- Application to use ApplicationStartedEvent as the trigger to start the
ModelLoading process instead of ApplicationReadyEvent
- Modified the testware to reflect the same
Issue-ID: CPS-2211
Change-Id: I3176b6373f585ec1915c3d43c553e233660225f5
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
Priyank Maheshwari [Wed, 5 Jun 2024 08:20:20 +0000 (08:20 +0000)]
Merge "CM SUBSCRIPTION: Add delete use case #2 (DMI-NCMP)"