summaryrefslogtreecommitdiffstats
path: root/cps-ri
AgeCommit message (Collapse)AuthorFilesLines
2021-06-01Introducing Antlr4 for cpsPath parsingToineSiebelink6-333/+24
-created new module for cpPathParser -added antlr rule for cpsPathWithSingleLeafCondition -added antlr rule for cpsPathWithDescendant (and with leaf conditions) -added antlr rule for ancestor axis -added unit test (copied from existing CpsPathQuerySpec) -udpated cps-ri to use new cpPathQuery from parser module -'imported' lexer rules from publix xPath grammar -Re-used existing CpsPathException but conversion happens in cps-ri to prevent additional dependency in cps-path-parser module Issue-ID: CPS-376 Change-Id: I2c5df98969402cbf69f6573c52705879450ce606 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
2021-05-28Load only required components in the application contextsRenu Kumari1-1/+2
Issue-ID: CPS-434 Signed-off-by: Renu Kumari <renu.kumari@bell.ca> Change-Id: Iacc12946495b9e6ec42b4b7d4e3b5b010fecceb5
2021-05-24Replace list-node content (part 1): CPS Service and persistence layersRuslan Kashapov2-0/+55
Issue-ID: CPS-362 Change-Id: I669c9fc6ef67c1992fe95e17a765f0c616b00f7e Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-05-24Adding Atomic Grouping to Discard backtracking once matched as part of Sonar ↵DylanB95EST1-2/+2
Security Hotspot Fix Issue-ID: CPS-89 Commit-ID: b23ee774ae2a2305f1feea747931879588f9cada Change-Id: I1f9d54246370b0c0bc8ec3d167eaed27ad58471b Signed-off-by: DylanB95EST <dylan.byrne@est.tech>
2021-05-21Adding Possessive Quantifier to Regex to prevent backtrackingDylanB95EST1-2/+2
Issie-ID: CPS-89 Commit-ID: 22dbd115593cd429b1b75af2000310ce79cfbf10 Signed-off-by: DylanB95EST <dylan.byrne@est.tech> Change-Id: I12f8aec77e9a4faa62234b7108617d47e4ed63fd
2021-05-21Update SonarQube Code SmellJosephKeenan1-1/+1
Issue-ID: CPS-89 Change-Id: I3a3e34b41ed98a0ab2c1accd438e2ec1d4a8d1cf Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
2021-05-11Create list-node elements (part1): CPS service and persistence layersRuslan Kashapov3-18/+75
+ fix integrity violation exception exposed out of persistence layer + refactor CpsDataServiceImplSpec to eliminate repeated code Issue-ID: CPS-360 Change-Id: Id70341fe54bf3c31af661f6aae04a7a80f4a1e9d Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-05-08Merge "Add CpsPathQueries.sql and create data inspired by the bookstore model"Bruno Sakoto5-53/+75
2021-05-07Add CpsPathQueries.sql and create data inspired by the bookstore modelniamhcore5-53/+75
Issue-ID: CPS-363 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I1eaee97d6ced7b0c0676de19d74b4d8262a5e811
2021-05-07Fix cps ran yang data modelBruno Sakoto8-1/+4539
* Add a liquibase change set to update cps ran model data * Rename model "cps-ran-schema-model@2021-01-28.yang" to be compliant with yang specifications related to file layout https://tools.ietf.org/html/rfc7950#section-5.2 * Improve yang validation error handling to provide more information about the validation failure in the request response * Add csit test to verify cps ran model retrieval Issue-ID: CPS-377 Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca> Change-Id: I4f0e337b50d79e81f895eb32f09ae9cb5d9d8a70
2021-05-04Fix ancestor cps path to recognize ancestor list valueniamhcore2-10/+11
Issue-ID: CPS-305 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I28f7f0c120a39190068a3192dccf0a1a6fbfeaf2
2021-04-30Implement cps path query to get ancestor by schema node identifierToineSiebelink7-106/+155
Cleaned up some legcy issues in related testware Issue-ID: CPS-305 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: Ic4b21308478f399e3a454dbcd73943e077b0f3f2 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
2021-04-30Fix Get descendent to support xpaths that end in list valuesputhuparambil.aditya3-17/+36
Issue-ID: CPS-367 Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca> Change-Id: I3234afd8b8b69c5a3c87db1669d9b304f9fcaa49
2021-04-29Investigate and update Spock versionshivasubedi5-30/+1
Issue-ID: CPS-368 Signed-off-by: shivasubedi <shiva.subedi@est.tech> Change-Id: Ie8505cf29068660af6bc6b9887f593f7e3be074f
2021-04-28Fix Code smellsshivasubedi4-42/+39
Issue-ID: CPS-89 Change-Id: Ib1b6fbc6b6d712f169100825683e6db3e7c85cb9 Signed-off-by: shivasubedi <shiva.subedi@est.tech>
2021-04-26Merge "Leaf String value comparison matches mix of single and double quotes"Rishi Chail2-6/+31
2021-04-26Get Data under anchor using single rootRishi.Chail4-13/+41
Issue-ID: CPS-325 Signed-off-by: Rishi.Chail <rishi.chail@est.tech> Change-Id: Id8da3d767199c5767c625b55d175ac6791dcca48
2021-04-26Leaf String value comparison matches mix of single and double quotesshivasubedi2-6/+31
Issue-ID: CPS-345 Signed-off-by: shivasubedi <shiva.subedi@est.tech> Change-Id: Id6db86817878ed5ed8ccaed4a9a71c5a06d6f97c
2021-04-23Upgrade to PostgreSQL 13Rishi.Chail1-1/+1
Upgrade version in docker compose and test containers to match OOM version. Issue-ID: CPS-355 Signed-off-by: Rishi.Chail <rishi.chail@est.tech> Change-Id: I8da816fb932822e50380e7c12487e234d75fa4d7
2021-04-22Implement ends with cps path query to support multiple attributes with 'and' ↵puthuparambil.aditya8-85/+274
condition Issue-ID: CPS-309 Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca> Change-Id: I80bf2650e2cd979b806fc29302fc5cb295f65241 Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>
2021-04-22Document legacy CPS Path functionalityToineSiebelink1-3/+4
Issue-ID: CPS-334 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Ib5266c7b30097661d97f8a499eaddc738dda0539
2021-04-15Fix "ends-with" query syntax to conform with xPath definitionToineSiebelink6-38/+42
"ends-with" is HOW we resolve it in sql query. 'descendant anywhere' is the correct Path name for the '//' operator - Updated method names, variable names, test description to reflect the correct terminolgy - Udpated query to always perfix the target (descendant name) with an '\' so it alwasy only matches whole node names - Updated regex for cpsPath to NOT accept triple /// (as per xPath this is invalid since a ndoeName cannot start with or contain a node separator Issue-ID: CPS-336 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: I9f181d6986d038311b839e3f9c5afc4237c7d347
2021-04-09Fix liquibase changeset so we can upgrade from H to IToineSiebelink3-3/+33
- Undo (accidental) changes in orginal liquiubase cahnge set (03) (see commit : https://gerrit.onap.org/r/c/cps/+/119061) - Add new changeset for required changed related to checksum calcualtion - There is a incorrect field name (checkstyle instead of checksum) in 03-loadData-schema-set.yaml but it seems to be ignored/overruled by the .csv import. If we change it liquibase report an checksum error during upgrade i.e were are not allowed to touch the file! Issue-ID: CPS-310 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: I00c94e3f1ddab7c9c54aac1294019ec7cd51634f
2021-04-07Merge "Fix security hotspots n Regex"Bruno Sakoto1-1/+2
2021-04-07Delete anchor part 1: service and persistence layersRuslan Kashapov3-11/+56
Issue-ID: CPS-312 Change-Id: I10ab5a2d115ffdf8179a99b6ec712f3eccfb5f13 Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-04-06Fix security hotspots n RegexToineSiebelink1-1/+2
Attemting to fix security risks indentified by SQ by reducing unlimited spaces to 0-9 spaces See https://sonarcloud.io/project/security_hotspots?id=onap_cps# I think the proposed regex changes are acceptable and better then adding @SupressWarning annotation (have to test by merging this commit as issues are not spotted when using SonarLinit locally?!) Issue-ID: CPS-89 Change-Id: I00bbacb3145650ab7d21d13152c08d5ccf0c04fa Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
2021-03-31Fix to store data for 2 anchors using same modelBruno Sakoto4-1/+73
This change deletes the database index on fragment table for dataspace and xpath. Issue-ID: CPS-300 Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca> Change-Id: I586cc38f7ed00a16f751ed4dbe3b514226c62657
2021-03-24Internal Server Error when creating the same data node twiceniamhcore6-15/+23
This change adds a generic exception handler class for a already defined object and handles a JsonSyntaxException. Issue-ID: CPS-290 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: Ie645237b5dd5b8e2b1d074c5613e7da560f57484
2021-03-18Fix code smellsRishi.Chail1-2/+1
1. removed typedefs annotation 2. chnaged final string constant name Issue-ID: CPS-89 Signed-off-by: Rishi.Chail <rishi.chail@est.tech> Change-Id: I524aa8c13e8bb4c6a00d810c6e89b30a48f37d10
2021-03-11Fix for security hotspot related to wek cyptographyputhuparambil.aditya5-11/+16
https://sonarcloud.io/project/security_hotspots?id=onap_cps&hotspots=AXfObcurA2pnU4Plp4-j Issue-ID: CPS-286 Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca> Change-Id: I31012f81797396682dbccae0e4992a33bac806c7
2021-03-10Fix SOnarQube bug on RegexToineSiebelink2-14/+20
- Regex included unlimitted repetition is now limited to 99 99 nested yang container should sufice, in my experience in 5G 20-30 levels is the max I have seen we can always record it as a 'known limitation' -tried to improve the redability of the Regex using constant names - Added edge-case senarios testing related to query regex Issue-ID: CPS-89 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Ic3442bc8ca22fe66976948f9913e033245a0e41c
2021-03-09Improve error reporting for invalid cps path queryniamhcore1-4/+4
Issue-ID: CPS-276 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I0e0abbaff32c936c67fc1092dc8385a0bc5ae49e
2021-03-05Update versions for next releaseRishi.Chail1-1/+1
Issue-ID: CPS-256 Signed-off-by: Rishi.Chail <rishi.chail@est.tech> Change-Id: Ifce438495233edd7de9f42129ab60bebc365f0b9
2021-03-04Upgrade version numberBruno Sakoto1-1/+1
Honolulu RC0 will be version 1.0.0 Issue-ID: CPS-260 Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca> Change-Id: Ib4d464100b7b5814070d41dc911f342f58836442
2021-03-04CPS-265 - Update cps path query to support 'ends with'niamhcore6-33/+135
Issue-ID: CPS-265 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I604191feaad820983d86e6fd844f543f51096a4e
2021-03-03CSIT: Integration tests for CPS CoreRuslan Kashapov1-0/+10
+ also fixes issue with fragment_id sequence not reflecting data loaded Issue-ID: CPS-261 Change-Id: If590d6a0859a1acc9005dd2da952b7dd93a23059 Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-03-02CPS-265 - updating cps path to support include-descendants option.niamhcore3-12/+14
Issue-ID: CPS-265 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I9e9b84760dbc8b5eb4b31ab972fdb2d186c6bb48
2021-02-24Merge "Schema creation and Data pre-load to on cps-xnf-proxy start-up using ↵Toine Siebelink15-85/+3090
liquibase"
2021-02-24Schema creation and Data pre-load to on cps-xnf-proxy start-up using liquibaseputhuparambil.aditya15-85/+3090
1. Common change-log master used only to specify the different change sets. 2. Specific change set created for cpsTables creation and for loading data to each cps component like dataspace, schema-set, anchor and dataNode 3. The data load cabability is configurable in the sense, it will be trigered only if the label is specified in the application.yaml Issue-ID: CPS-173, CPS-242 Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca> Change-Id: I44c640ce50b7203f5799d722149397478a98bc05
2021-02-22Upgrade version numberBruno Sakoto1-1/+1
Issue-ID: CPS-175 Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca> Change-Id: I23bdd1b406e2076c612c06c1677af4440c0b883f
2021-02-19Persistence layer - Query Datanodes using cpsPath that contains contains a ↵niamhcore6-5/+194
leaf name and a leaf value Issue-ID: CPS-231 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I9bd483a4b76e233ab6c64b3ef8aacb593e4e9da0
2021-02-16Data fragment update by xpath #2 - persistence layerRuslan Kashapov4-22/+160
Issue-ID: CPS-58 Change-Id: Ifc4580936d06c6907d6b5ab20657063b6707ccbe Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-02-04Fetching data node by xpath - persistence layerRuslan Kashapov4-39/+144
IssueID: CPS-71 Change-Id: I88f76cf36ef8a1e4ccbd4f1eac8867e93ed5be82 Signed-off-by: Ruslan Kashapov <ruslan.kashapov@pantheon.tech>
2021-01-29Introducing Spock/Groovy for Data Integration TestsToineSiebelink8-634/+554
Replaced CpsAdminPersistenceServiceTest with CpsAdminPersistenceServiceSpec Replaced CpsDataPersistenceServiceTest with CpsDataPersistenceServiceSpec Replaced CpsModulePersistenceServiceTest with CpsModulePersistenceServiceSpec Extracted out common integration test base Rationalised test (there was a lot of duplication already!) Issue-ID: CPS-160 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: I3311533fba1398feb00b6adf4209399cea8d3a1b
2021-01-27Draft at proposal where the DataNodeBuilder 'replaces' yang Utils to buidl a ↵ToineSiebelink1-8/+9
DataNode Most complexity is related to immutable collections and the fact taht we are adding data while recursing over the orignal data in an uncontrolled order. I cleaned it up as best I could with no logic in DataNode. Espcially the handling of LitLeaves requires some specialed handling. Thsi is just a draft solution for that I still propose we get back to that in dedicated user stories for handling the various types of Yang elements Hope this helps... Issue-ID: CPS-137 Signed-off-by: ToineSiebelink <toine.siebelink@est.tech> Change-Id: Iab7cfcff67412c01bcdab95e707e1350bf60fab1
2021-01-25Add DataNodeDoesNotExist Exceptionniamhcore2-4/+4
Issue-ID: CPS-182 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: I05fdd0fbc207e4806c187b96d35d7b6c54b795ea
2021-01-21Remove Module TableRishi.Chail1-13/+1
Issue-ID: CPS-185 Signed-off-by: Rishi.Chail <rishi.chail@est.tech> Change-Id: Id93b3ec4dec27184bc46767146d9801063f54916
2021-01-22Add another Child to a Fragment that has already at least one Childniamhcore4-63/+136
Issue-ID: CPS-164 Signed-off-by: niamhcore <niamh.core@est.tech> Change-Id: Ib6a4f8ce4fc9247726e016ecaa0b1e45ff11bdbb
2021-01-22Merge "Introduce caffeine cache"Toine Siebelink1-2/+1
2021-01-21Introduce caffeine cacheClaudio David Gasparini1-2/+1
Issue-ID: CPS-163 Signed-off-by: Claudio David Gasparini <claudio.gasparini@pantheon.tech> Change-Id: Iff9b831c2d895d82aff419f60a8dd86a38b545d0