summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
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-30Merge "Ajust time limits for performance tests"Daniel Hanrahan8-72/+66
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-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>
2024-04-11Merge "Update depcrecated method in KafaConfig"Toine Siebelink2-7/+11
2024-04-11Update depcrecated method in KafaConfigToineSiebelink2-7/+11
Issue-ID: CPS-2179 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: I882746a68f2d32f07fbcfeeac1f45b9dbbb8c62e
2024-04-10Define a java based datastructure for DataJob and interfacesourabh_sourabh9-4/+340
- Defined java interface to handle datajob - Defined all the modules needed to process data job request - data jon response module id also defined - dummy impl. of java interface Issue-ID: CPS-2155 Change-Id: I20c7b621aa4bd40fe7758cb19b8720fc7cf5164c Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
2024-04-10Introduce and handle Operation Too Large Exception for batch operationsToineSiebelink7-34/+103
Issue-ID: CPS-2164 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Iec05d2013be4f971309f0e75d84dc5d0936eb8ef
2024-04-10Document new NCMP Status Code: 112, message too largeToineSiebelink2-1/+4
- Updated RTD - Added new value to relevant Enum (although not used in our code) Issue-ID: CPS-2178 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: I4ba8c1622921919ae253a292d96307439ef20e2a
2024-04-04Add OpenSFF ScoreCard Action to GitHubToineSiebelink1-0/+72
Issue-ID: CPS-2158 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Idf4f29756317591b616e776b750aca3340e428f1
2024-04-04Merge "Remove old committers from INFO.yaml"Toine Siebelink1-5/+0
2024-04-04Merge "Add integration tests of NCMP REST API"Toine Siebelink1-0/+95
2024-04-04Remove old committers from INFO.yamlToineSiebelink1-5/+0
Issue-ID: CPS-1887 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Ic147dc498307fa24ce5953ba39df0f4e3a2e6a13
2024-04-04Merge "Add moduleSetTag to the request towards dmi plugin"Toine Siebelink5-4/+13
2024-04-04Merge "Modify INFO.yaml for committer rights"Toine Siebelink1-1/+6
2024-04-04Add moduleSetTag to the request towards dmi pluginleventecsanyi5-4/+13
- mapping module set tag to DmiRequest Issue-ID: CPS-2030 Change-Id: I487e48baf98919d3ff5e25d2428a065dd3a30335 Signed-off-by: leventecsanyi <levente.csanyi@est.tech>
2024-04-04Merge "[BUG] Mock DMI will respond to Health Checks during tests"Toine Siebelink4-30/+42
2024-03-29Bump CPS versionmpriyank25-25/+48
- bump cps to 3.4.8-SNAPSHOT - also updated the release notes Issue-ID: CPS-2168 Change-Id: Iee7f1e2febbf28b0d5a2855d3128d9eb8bb3c490 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-03-29Container YAML for CPSmpriyank1-0/+8
- added container yaml for cps 3.4.7 Issue-ID: CPS-2168 Change-Id: Ieb3afc2eb7f62e20bedef28cd5da3a2e7731bfa0 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-03-29Maven stage yaml for 3.4.7mpriyank1-0/+4
- added maven stage yaml to release CPS 3.4.7 version Issue-ID: CPS-2168 Change-Id: Id139fcf1c19acb2ffdd0a9e39066dc961158159b Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-03-29Release notes for 3.4.7 CPSmpriyank1-0/+1
- updated release notes for CPS 3.4.7 version Issue-ID: CPS-2168 Change-Id: I3a6689234e63db301787d49a5c76063c0b6203ae Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-03-29Merge "Uplift Spring boot 3.2.4"Priyank Maheshwari24-133/+87
2024-03-29Uplift Spring boot 3.2.4emaclee24-133/+87
- uplifted spring boot from 3.1.2 to 3.2.4 - removed redundant dependencies - upgraded relevant dependencies - use of @Bean(some_name) is give a specific bean name - using @Bean(some_name) with @Qualifier so on the injection point to specify which bean to actually inject, this is used as we deal with multiple beans of the same type. Issue-ID: CPS-2101 Change-Id: I64d5bf567b9eb03fb39034a46528d0160b332715 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-03-28Add integration tests of NCMP REST APIdanielhanrahan1-0/+95
Add some tests using MockMvc of NCMP REST API: - CM handle registration - CM handle ID searches - CM handle deregistration Issue-ID: CPS-2033 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: I493aca8c3f8c263f5a50f6a2c66e317b9a6e08aa
2024-03-28[BUG] Mock DMI will respond to Health Checks during testsdanielhanrahan4-30/+42
Health Check for DMI is sometimes running during integration tests, causing the mockDmiServer to fail due to unexpected Rest requests. As a fix, the mock DMI will respond to health checks. Issue-ID: CPS-2167 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ie4a0059583e66dfcc00e1aa998671a662ccf053a
2024-03-28Modify INFO.yaml for committer rightsdanielhanrahan1-1/+6
Issue-ID: CPS-1887 Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech> Change-Id: Ibe95eaf58679fc4c2d7486f1402d922d1264b450
2024-03-27Merge "Scheduled task for Subscription Response"Lee Anjella Macabuhay3-18/+182
2024-03-27Fix code smellemaclee1-9/+2
- change string to formatted Issue-ID: CPS-475 Change-Id: I59271c63f244c4d6684b145b1afe2356668e6fd8 Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
2024-03-27Scheduled task for Subscription Responsempriyank3-18/+182
- Need to send the response back to the client in max 30 secs - Also have the capability to send the response right away - Testware added for the same - Also added code to cancel the scheduled task - Added state for cancelling the task Issue-ID: CPS-2140 Change-Id: I3ab321d8221cd8f697c26be46d2e63d89b360923 Signed-off-by: mpriyank <priyank.maheshwari@est.tech>
2024-03-26Merge "Condense Liquibase steps"Toine Siebelink51-8492/+260
2024-03-26Condense Liquibase stepsdavid.mcweeney51-8492/+260
Change-Id: I1493f49bb93c359df3ff7d4f928f9a94fb8b2604 Signed-off-by: david.mcweeney <david.mcweeney@est.tech> Issue-ID: CPS-2061
2024-03-26Merge "Code review cleanup for: Add kafka messaging support to integration ↵Lee Anjella Macabuhay3-22/+25
test module"