summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2024-05-10Merge "Ncmp out event for REJECTED scenario"Daniel Hanrahan3-9/+36
2024-05-09Ncmp out event for REJECTED scenariompriyank3-9/+36
- Handling and publishing the event to client when dmi plugin rejects applying the subscription - test scenarios to support the same Issue-ID: CPS-2214 Change-Id: I2ac6d125a7262a7428a962d81a26a33594b570d9 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-05-09Fine tuning of the performance plotshalil.cakal6-37/+37
- Fine tune some of the performance plots which are not align to the limit Issue-ID: CPS-2200 Change-Id: I925443aa74a4eb814728d0642db0951c76f86186 Signed-off-by: halil.cakal <halil.cakal@est.tech>
2024-05-09Release note for cps-change-event togglempriyank1-0/+1
- Release notes to enable and disable the toggle functionality introduced. - If the CPS change events are of no interest to them then they can be disabled by the environment variable CPS_CHANGE_EVENT_NOTIFICATIONS_ENABLED Issue-ID: CPS-2211 Change-Id: If2fddb6a499e24194b3eeca2ecdf99b8fb5ec687 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-05-08Conditional cps change eventsmpriyank4-10/+30
- introduced a parameter to control the cps core change event notifications. we should be able to disable these notifications without affecting other notification flows. - fixed the LayeredArchitectureTest as we are accessing the Anchor model in the events package now. Issue-ID: CPS-2213 Change-Id: Id875925bc14de1cc6e8fa3193c0df470e09fe43f Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-05-07Merge "Schedule response to client"Lee Anjella Macabuhay2-7/+16
2024-05-03Merge "[BUG] Correctly parse observedTimestamp"Lee Anjella Macabuhay4-12/+23
2024-05-03[BUG] Correctly parse observedTimestampmpriyank4-12/+23
- Using the DateTimeUtility to parse the passed OffsetDateTime and using it for conversion to string - Moving to actual src/main/java from test side as it is to be used in the prod code as well and only keeping the methods in use. - Added test to fix the coverage issue Issue-ID: CPS-2211 Change-Id: I2b4325bb77a6da7fd470986b1da293f455650375 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-05-03Merge "Release notes is updated for error hanndling related to upgrade ↵Priyank Maheshwari1-0/+1
operation"
2024-05-03Release notes is updated for error hanndling related to upgradesourabh_sourabh1-0/+1
operation Issue-ID: CPS-2204 Change-Id: Iaa3f61c53e4120cdf19b7ad1c3313fc32f904866 Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-05-03Make NCMP integration tests use MockWebServersourabh_sourabh15-214/+253
Change from using MockRestServiceServer - which is only compatible with RestTemplate - to MockWebServer, which will allow tests to work using WebClient instead of RestTemplate. Issue-ID: CPS-2183 Change-Id: I7494fe17cba6e92f7df81f0fd0185e1d2b5a5541 Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-05-03Schedule response to clientmpriyank2-7/+16
- Scheduling sending the response by reading whatever is present in the cache for subscription after a pre-configured time. - Also added test validation for the same Issue-ID: CPS-2209 Change-Id: Ifacb0b12734a0529e671b0b63836189a243e453c Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-05-02Merge "Query operation path by alternate id"Daniel Hanrahan4-14/+117
2024-05-02Handle error in case of upgrade operationsourabh_sourabh2-11/+16
- Send an error response if cm handle not found for upgrade operation. - Restrict upgrade operation if it is already in progress. Issue-ID:CPS-2204 Change-Id: Iad63732c5b1d2215b06a09ddda1c0c604cd7648c Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-05-02Merge "Add Notification support in cps core"Priyank Maheshwari9-9/+365
2024-05-01Merge "CM Subscription: Link method to publish to DMI in service layer"Daniel Hanrahan10-36/+228
2024-05-01Bump CPS versiondanielhanrahan25-25/+52
- bump cps to 3.4.9-SNAPSHOT - also updated the release notes Issue-ID: CPS-2205 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ifd7feb413baac30e3cbbdf2aeaac5fbb5ae2e463
2024-05-01Container YAML for CPS 3.4.8danielhanrahan1-0/+8
- added container yaml for cps 3.4.8 Issue-ID: CPS-2205 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Id6ef50ab413bb910fa6ea866964ffe01d90b9b29
2024-05-01Maven stage yaml for 3.4.8danielhanrahan1-0/+4
- added maven stage yaml to release CPS 3.4.8 version Issue-ID: CPS-2205 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ic734ec947de86a30cf7b3f080a82c518403a4f5f
2024-05-01Update OpenAPI docs for 3.4.8danielhanrahan2-18/+42
Issue-ID: CPS-2205 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ib520a55b099b7a89751dab7032cabeba19b6a934
2024-04-30CM Subscription: Link method to publish to DMI in service layeremaclee10-36/+228
- Add call to method to publish to DMI for create request - Refactor calls to all Mappers to a single Mapper class handler Issue-ID: CPS-2196 Change-Id: I6476f47684ea9a0f55a7d207c1931a0fef8aa672 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-04-30Release notes update for 3.4.8danielhanrahan1-0/+2
Add release notes updates for bug fix CPS-2190 (slow module search) Issue-ID: CPS-2190 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: I9c4500727640040593aacbd209c2d5ec6a86a315
2024-04-30Merge "[BUG]Defaults for cps and dmi user and pass"Daniel Hanrahan1-4/+4
2024-04-30Merge "CM Subscription: REfactor classes of producers and consumers"Priyank Maheshwari13-14/+157
2024-04-30[BUG]Defaults for cps and dmi user and passmpriyank1-4/+4
- provided default values for DMI_USERNAME and DMI_PASSWORD which is required to set when DMI_AUTH_ENABLED is true - Also provided defaults for CPS_USERNAME and CPS_PASSWORD as its been used in the logback xml file and if not provided then causes an issue with the logger Issue-ID: CPS-2194 Change-Id: I70fdbeca970556c7bd19f27c1b6b5123fd591283 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-04-30Reverted web client changes as it breaking CPS functionality:sourabh_sourabh23-269/+97
1. Error handling not working 2. Unable to make dmi call as url appens extra "/" Commit 1: Replace RestTemplate with WebClient in synchronous DMI calls - added DmiWebClientConfiguration - use WebClient in DmiRestClient - fixed unit tests - ingnored failing NCMP related integration tests (related to WebClient, will be fixed in a future patch) - encode query params for DMI requests Commit 2: Module sync task failing with WebClient - added configurable buffer size Commit 3: Re-use ncmp.dmi.httpclient.maximumConnectionsTotal parameter (as documented in RTD) to control webclient connection poolsize Issue-ID:CPS-989, CPS-2195, CPS-2198 Change-Id: I6c71ab2801b222708404f51bd60f2b3031ebf32e Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-04-30Add Notification support in cps corerajesh.kumar9-9/+365
Add notification support using cloud events Issue-ID:CPS-2068 Change-Id: I56c34400dc73c71b936a51260efd240223babacd Signed-off-by: rajesh.kumar <rk00747546@techmahindra.com>
2024-04-30Merge "Ajust time limits for performance tests"Daniel Hanrahan8-72/+66
2024-04-29CM Subscription: REfactor classes of producers and consumersemaclee13-14/+157
Issue-ID: CPS-2196 Change-Id: I0b56f37a1b615b9615d6bd7005348733e1739a46 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-04-29Re-use ncmp.dmi.httpclient.maximumConnectionsTotal parametersourabh_sourabh3-19/+25
(as documented in RTD) to control webclient connection poolsize Issue-ID: CPS-2198 Change-Id: Id441208c66433b2539adf83a0881c3c3c79d2b15 Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-04-29Query operation path by alternate idsourabh_sourabh4-14/+117
- New method is written to find longest match for fdn path - Introduced new exception for alternate id Issue-ID: CPS-2142 Change-Id: Ifd048574db88586a2d37a3681034a083eb2b7691 Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-04-29Ajust time limits for performance testshalil.cakal8-72/+66
- new time limits adjusted for dedicated physical cps-performance test-server Issue-ID: CPS-2192 Change-Id: Ia258a8f7460b3fbe68e5b3b61bc918f7d394c1b1 Signed-off-by: halil.cakal <halil.cakal@est.tech>
2024-04-29Merge "Module sync task failing with WebClient"Daniel Hanrahan5-1/+11
2024-04-29Merge "CM SUBSCRIPTION: Remove accepted and rejected cache entries"Priyank Maheshwari5-23/+95
2024-04-26Module sync task failing with WebClientleventecsanyi5-1/+11
- added configurable buffer size Issue-ID: CPS-2195 Change-Id: I7cdb09856fd4d80134a91e54faa7e6600f3721dc Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
2024-04-26CM SUBSCRIPTION: Remove accepted and rejected cache entriesemaclee5-23/+95
Issue-ID: CPS-2185 Change-Id: I01ba8e4f6e0e42a31ad5234ad468ac4625c2b0f3 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-04-26Faster module searches (CPS-2190 #3)danielhanrahan6-21/+18
This greatly improves performance of module searches by eliminating unneeded SQL queries via Hibernate lazy fetching. Issue-ID: CPS-2190 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ie9e65017d0027366456f1741cc37b10679317b25
2024-04-25[BUG] Remove slow validation check during module search (CPS-2190 #2)danielhanrahan4-65/+6
The check for existing modules is extremely slow, being dependent on the file sizes of the Yang modules being queries. The validation check is also completely unnecessary, since it simply ignores any exceptions and returns an empty set, which the main logic does implicitly. (The Dataspace check is also not needed, as the NCMP dataspace is guaranteed to exist.) Issue-ID: CPS-2190 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: I329c1f8aac3f50bda0333e6c9c686f47af2e009f
2024-04-25Perf Test for Module Search is dependent on module size (CPS-2190 #1)danielhanrahan1-0/+113
Add a new test showing that Yang module search is dependent on the file size of the Yang modules being searched for. Issue-ID: CPS-2190 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: I8fd28f4c91be11ef7428251397a887af17649015
2024-04-25Replace RestTemplate with WebClient in synchronous DMI callsleventecsanyi21-99/+257
- added DmiWebClientConfiguration - use WebClient in DmiRestClient - fixed unit tests - ingnored failing NCMP related integration tests (related to WebClient, will be fixed in a future patch) - encode query params for DMI requests Issue-ID:CPS-989 Change-Id: I26434568f68266e95669a51c134978ddd197d802 Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
2024-04-24Merge "Reject create request with duplicated subscriptionId"Priyank Maheshwari7-17/+212
2024-04-24Reject create request with duplicated subscriptionIdemaclee7-17/+212
Issue-ID: CPS-2184 Change-Id: I1bddb02239e6a2d3f47aa731eb24a9f1f64e63b5 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech> Signed-off-by: seanbeirne <sean.beirne@est.tech>
2024-04-24NcmpCloudEventBuilder refactoringmpriyank3-16/+18
- Builder named as NcmpEvent and we have specialised method to build the NcmpEvent as cloud event now - Introduced source as the builder default property Issue-ID: CPS-2191 Change-Id: I51c571313091ce9a4d10f4e1af7423eee999cea8 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-04-23[BUG] Make failed async task report failure on Kafka topicdanielhanrahan11-27/+156
- In event of async task timeout, error code 102 (DMI_SERVICE_NOT_RESPONDING) is sent to client topic. - In event of unexpected error (such as database unavailable), error code 108 (UNKNOWN_ERROR) is sent to client topic. - The default timeouts have been adjusted so that the task timeout (60s) is longer than the HTTP and Database timeouts (30s), so that expected codes are returned. Issue-ID: CPS-2186 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: I84c3447a625e084c445ab2f5c01e2b32a0c971ac
2024-04-19Temporary dsiable Operation Too Large ExceptionToineSiebelink2-3/+4
- settign limit to 50000 to efftively disable it - ericsson is not ready and wil infrom us when it can be enabled again Issue-ID: CPS-2164 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Ib1a8308a20bd77fcba9c4c9fc94739d5dd731b1f
2024-04-18Fix build issue with subscriptionemaclee2-3/+3
Issue-ID: CPS-2164 Change-Id: I49a18df0927b5beff62a12f9bc2a2e8e486537c9 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-04-17Merge "Cm Subscription: Remove subscription method"Daniel Hanrahan4-51/+150
2024-04-17Cm Subscription: Remove subscription methodemaclee4-51/+150
- method to remove a subscription Id from leaflist - condition if subscription list is empty, remove subscription all together Issue-ID: CPS-2164 Change-Id: Id694f441f9675fa9a048e3b824e1f02fae73f87e Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-04-12Merge "Handle Subscription Response from DMI"Priyank Maheshwari4-6/+126
2024-04-12Handle Subscription Response from DMIseanbeirne4-6/+126
Issue-ID: CPS-1972 Change-Id: Idb8ad14f06e365707404d099a4581282d4a0add9 Signed-off-by: seanbeirne <sean.beirne@est.tech>