From: Toine Siebelink Date: Wed, 7 Feb 2024 17:47:19 +0000 (+0000) Subject: Merge "Replace sleep with PollingConditions" X-Git-Tag: 3.4.4~37 X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=a3096b9d22b7a3517c7933dfeb10984f6bedf063;hp=cc1c83c740d3bde18654098013f2b76eb8fa0567;p=cps.git Merge "Replace sleep with PollingConditions" --- diff --git a/checkstyle/pom.xml b/checkstyle/pom.xml index ff4ecb5f1..95794b165 100644 --- a/checkstyle/pom.xml +++ b/checkstyle/pom.xml @@ -26,7 +26,7 @@ 4.0.0 org.onap.cps checkstyle - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT diff --git a/cps-application/pom.xml b/cps-application/pom.xml index be7664aa5..6466bcfe4 100644 --- a/cps-application/pom.xml +++ b/cps-application/pom.xml @@ -28,7 +28,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-bom/pom.xml b/cps-bom/pom.xml index 54d7602cb..ecd442cf4 100644 --- a/cps-bom/pom.xml +++ b/cps-bom/pom.xml @@ -25,7 +25,7 @@ 4.0.0 org.onap.cps cps-bom - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT pom This artifact contains dependencyManagement declarations of all published CPS components. diff --git a/cps-dependencies/pom.xml b/cps-dependencies/pom.xml index e7eb576ce..2f1a4d2d8 100644 --- a/cps-dependencies/pom.xml +++ b/cps-dependencies/pom.xml @@ -27,7 +27,7 @@ 4.0.0 org.onap.cps cps-dependencies - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT pom ${project.groupId}:${project.artifactId} diff --git a/cps-events/pom.xml b/cps-events/pom.xml index b1fd6f993..662d150a5 100644 --- a/cps-events/pom.xml +++ b/cps-events/pom.xml @@ -24,7 +24,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-ncmp-events/pom.xml b/cps-ncmp-events/pom.xml index 36c69c69a..6e638438c 100644 --- a/cps-ncmp-events/pom.xml +++ b/cps-ncmp-events/pom.xml @@ -23,7 +23,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-ncmp-rest-stub/cps-ncmp-rest-stub-app/pom.xml b/cps-ncmp-rest-stub/cps-ncmp-rest-stub-app/pom.xml index cdd9eefe0..25d4939eb 100644 --- a/cps-ncmp-rest-stub/cps-ncmp-rest-stub-app/pom.xml +++ b/cps-ncmp-rest-stub/cps-ncmp-rest-stub-app/pom.xml @@ -22,7 +22,7 @@ org.onap.cps cps-ncmp-rest-stub - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT cps-ncmp-rest-stub-app diff --git a/cps-ncmp-rest-stub/cps-ncmp-rest-stub-service/pom.xml b/cps-ncmp-rest-stub/cps-ncmp-rest-stub-service/pom.xml index 695065a48..1a8e89b68 100644 --- a/cps-ncmp-rest-stub/cps-ncmp-rest-stub-service/pom.xml +++ b/cps-ncmp-rest-stub/cps-ncmp-rest-stub-service/pom.xml @@ -21,7 +21,7 @@ org.onap.cps cps-ncmp-rest-stub - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT cps-ncmp-rest-stub-service diff --git a/cps-ncmp-rest-stub/pom.xml b/cps-ncmp-rest-stub/pom.xml index a1c5ee04f..a35a88662 100644 --- a/cps-ncmp-rest-stub/pom.xml +++ b/cps-ncmp-rest-stub/pom.xml @@ -22,7 +22,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-ncmp-rest/pom.xml b/cps-ncmp-rest/pom.xml index 27c640a63..f363c01f6 100644 --- a/cps-ncmp-rest/pom.xml +++ b/cps-ncmp-rest/pom.xml @@ -27,7 +27,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-ncmp-service/pom.xml b/cps-ncmp-service/pom.xml index 93c60fa53..9beb6b7b0 100644 --- a/cps-ncmp-service/pom.xml +++ b/cps-ncmp-service/pom.xml @@ -27,7 +27,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-parent/pom.xml b/cps-parent/pom.xml index 2856b9874..2e81b004e 100644 --- a/cps-parent/pom.xml +++ b/cps-parent/pom.xml @@ -32,7 +32,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT pom diff --git a/cps-path-parser/pom.xml b/cps-path-parser/pom.xml index 4b743a725..e6e00dcf3 100644 --- a/cps-path-parser/pom.xml +++ b/cps-path-parser/pom.xml @@ -23,7 +23,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-rest/pom.xml b/cps-rest/pom.xml index 0b31577ae..022c3b8eb 100644 --- a/cps-rest/pom.xml +++ b/cps-rest/pom.xml @@ -28,7 +28,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-ri/pom.xml b/cps-ri/pom.xml index 600676838..2fd369b2a 100644 --- a/cps-ri/pom.xml +++ b/cps-ri/pom.xml @@ -26,7 +26,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-service/pom.xml b/cps-service/pom.xml index e661f3421..de716dfa5 100644 --- a/cps-service/pom.xml +++ b/cps-service/pom.xml @@ -29,7 +29,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/cps-service/src/main/java/org/onap/cps/api/impl/CpsModuleServiceImpl.java b/cps-service/src/main/java/org/onap/cps/api/impl/CpsModuleServiceImpl.java index 0f9eb3c64..14b949e5a 100644 --- a/cps-service/src/main/java/org/onap/cps/api/impl/CpsModuleServiceImpl.java +++ b/cps-service/src/main/java/org/onap/cps/api/impl/CpsModuleServiceImpl.java @@ -117,8 +117,8 @@ public class CpsModuleServiceImpl implements CpsModuleService { final Collection anchorNames = cpsAnchorService.getAnchors(dataspaceName, schemaSetNames) .stream().map(Anchor::getName).collect(Collectors.toSet()); cpsAnchorService.deleteAnchors(dataspaceName, anchorNames); - cpsModulePersistenceService.deleteUnusedYangResourceModules(); cpsModulePersistenceService.deleteSchemaSets(dataspaceName, schemaSetNames); + cpsModulePersistenceService.deleteUnusedYangResourceModules(); for (final String schemaSetName : schemaSetNames) { yangTextSchemaSourceSetCache.removeFromCache(dataspaceName, schemaSetName); } diff --git a/cps-service/src/main/java/org/onap/cps/spi/model/ModuleDefinition.java b/cps-service/src/main/java/org/onap/cps/spi/model/ModuleDefinition.java index d7a5b38dc..909b7bc11 100644 --- a/cps-service/src/main/java/org/onap/cps/spi/model/ModuleDefinition.java +++ b/cps-service/src/main/java/org/onap/cps/spi/model/ModuleDefinition.java @@ -1,6 +1,6 @@ /* * ============LICENSE_START======================================================= - * Copyright (C) 2022 Nordix Foundation + * Copyright (C) 2022-2024 Nordix Foundation * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -20,11 +20,15 @@ package org.onap.cps.spi.model; +import lombok.EqualsAndHashCode; import lombok.Getter; import lombok.Setter; +import lombok.ToString; @Getter @Setter +@EqualsAndHashCode(callSuper = true) +@ToString public class ModuleDefinition extends ModuleReference { private static final long serialVersionUID = -6591435720836327732L; diff --git a/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-app/pom.xml b/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-app/pom.xml index cbb5155a3..eca8871c6 100644 --- a/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-app/pom.xml +++ b/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-app/pom.xml @@ -22,7 +22,7 @@ org.onap.cps dmi-plugin-demo-and-csit-stub - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT dmi-plugin-demo-and-csit-stub-app diff --git a/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-service/pom.xml b/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-service/pom.xml index 9b6e40254..d3142f821 100644 --- a/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-service/pom.xml +++ b/dmi-plugin-demo-and-csit-stub/dmi-plugin-demo-and-csit-stub-service/pom.xml @@ -21,7 +21,7 @@ org.onap.cps dmi-plugin-demo-and-csit-stub - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT dmi-plugin-demo-and-csit-stub-service diff --git a/dmi-plugin-demo-and-csit-stub/pom.xml b/dmi-plugin-demo-and-csit-stub/pom.xml index d708f7c6e..9fb40e3cd 100644 --- a/dmi-plugin-demo-and-csit-stub/pom.xml +++ b/dmi-plugin-demo-and-csit-stub/pom.xml @@ -22,7 +22,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/docs/api/swagger/ncmp/openapi.yaml b/docs/api/swagger/ncmp/openapi.yaml index 8098faefc..a1cc5d9ba 100644 --- a/docs/api/swagger/ncmp/openapi.yaml +++ b/docs/api/swagger/ncmp/openapi.yaml @@ -918,9 +918,9 @@ paths: - network-cm-proxy /v1/ch/{cm-handle}/modules/definitions: get: - description: "Fetch all module definitions (name, revision, yang resource) for\ - \ a given cm handle" - operationId: getModuleDefinitionsByCmHandleId + description: "Get module definitions (module name, revision, yang resource)\ + \ with options to filter on module name and revision" + operationId: getModuleDefinitions parameters: - description: "The identifier for a network function, network element, subnetwork\ \ or any other cm object by managed Network CM Proxy" @@ -930,6 +930,21 @@ paths: schema: example: my-cm-handle type: string + - description: Filter for a module name.This is an optional parameter + in: query + name: module-name + required: false + schema: + example: my-module + type: string + - description: Filter for a module revision.This is an optional parameter and + ignored when no module name is supplied + in: query + name: revision + required: false + schema: + example: 2024-01-22 + type: string responses: "200": content: @@ -969,8 +984,7 @@ paths: schema: $ref: '#/components/schemas/ErrorMessage' description: Internal Server Error - summary: "Fetch all module definitions (name, revision, yang resource) for a\ - \ given cm handle" + summary: Get module definitions tags: - network-cm-proxy /v1/ch/searches: @@ -1666,6 +1680,23 @@ components: schema: default: / type: string + moduleNameInQuery: + description: Filter for a module name.This is an optional parameter + in: query + name: module-name + required: false + schema: + example: my-module + type: string + revisionInQuery: + description: Filter for a module revision.This is an optional parameter and + ignored when no module name is supplied + in: query + name: revision + required: false + schema: + example: 2024-01-22 + type: string dataSyncEnabled: description: Is used to enable or disable the data synchronization flag in: query diff --git a/docs/design.rst b/docs/design.rst index 80eb5f152..af1f5abd4 100644 --- a/docs/design.rst +++ b/docs/design.rst @@ -72,6 +72,12 @@ CPS Path Several CPS APIs use the cps-path (or cpsPath in Java API) parameter. The CPS Path is described in detail in :doc:`cps-path`. +CPS Delta +========= + +CPS Delta feature provides the ability to find the delta/difference between two JSON configurations. +The CPS Delta feature is described in detail in :doc:`cps-delta-feature`. + NCMP CM Handle Querying ======================= diff --git a/docs/release-notes.rst b/docs/release-notes.rst index 4c93dd1f4..191a04d1a 100644 --- a/docs/release-notes.rst +++ b/docs/release-notes.rst @@ -16,6 +16,34 @@ CPS Release Notes .. * * * NEW DELHI * * * .. ========================= +Version: 3.4.4 +============== + +Release Data +------------ + ++--------------------------------------+--------------------------------------------------------+ +| **CPS Project** | | +| | | ++--------------------------------------+--------------------------------------------------------+ +| **Docker images** | onap/cps-and-ncmp:3.4.4 | +| | | ++--------------------------------------+--------------------------------------------------------+ +| **Release designation** | 3.4.4 New Delhi | +| | | ++--------------------------------------+--------------------------------------------------------+ +| **Release date** | Not yet released | +| | | ++--------------------------------------+--------------------------------------------------------+ + +Bug Fixes +--------- +3.4.4 + +Features +-------- + + Version: 3.4.3 ============== @@ -32,14 +60,16 @@ Release Data | **Release designation** | 3.4.3 New Delhi | | | | +--------------------------------------+--------------------------------------------------------+ -| **Release date** | Not yet released | +| **Release date** | 2024 February 07 | | | | +--------------------------------------+--------------------------------------------------------+ Bug Fixes --------- 3.4.3 - + - `CPS-2000 `_ Fix for Schema object cache not being distributed. + - `CPS-2027 `_ Fixes for upgrade yang modules using module set tag. + - `CPS-2070 `_ Add retry interval for Kafka consumer. Features -------- diff --git a/integration-test/pom.xml b/integration-test/pom.xml index 3ac096418..73998cd13 100644 --- a/integration-test/pom.xml +++ b/integration-test/pom.xml @@ -23,7 +23,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/base/CpsIntegrationSpecBase.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/base/CpsIntegrationSpecBase.groovy index 6dec3dbf9..b59e0ff55 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/base/CpsIntegrationSpecBase.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/base/CpsIntegrationSpecBase.groovy @@ -1,6 +1,6 @@ /* * ============LICENSE_START======================================================= - * Copyright (C) 2023 Nordix Foundation + * Copyright (C) 2023-2024 Nordix Foundation * ================================================================================ * Licensed under the Apache License, Version 2.0 (the 'License'); * you may not use this file except in compliance with the License. @@ -89,8 +89,7 @@ class CpsIntegrationSpecBase extends Specification { def setup() { if (!initialized) { cpsDataspaceService.createDataspace(GENERAL_TEST_DATASPACE) - def bookstoreModelFileContent = readResourceDataFile('bookstore/bookstore.yang') - cpsModuleService.createSchemaSet(GENERAL_TEST_DATASPACE, BOOKSTORE_SCHEMA_SET, [bookstore : bookstoreModelFileContent]) + createStandardBookStoreSchemaSet(GENERAL_TEST_DATASPACE) initialized = true; } } @@ -111,6 +110,20 @@ class CpsIntegrationSpecBase extends Specification { return new File('src/test/resources/data/' + filename).text } + def getBookstoreYangResourcesNameToContentMap() { + def bookstoreModelFileContent = readResourceDataFile('bookstore/bookstore.yang') + def bookstoreTypesFileContent = readResourceDataFile('bookstore/bookstore-types.yang') + return [bookstore: bookstoreModelFileContent, bookstoreTypes: bookstoreTypesFileContent] + } + + def createStandardBookStoreSchemaSet(targetDataspace) { + cpsModuleService.createSchemaSet(targetDataspace, BOOKSTORE_SCHEMA_SET, getBookstoreYangResourcesNameToContentMap()) + } + + def createStandardBookStoreSchemaSet(targetDataspace, targetSchemaSet) { + cpsModuleService.createSchemaSet(targetDataspace, targetSchemaSet, getBookstoreYangResourcesNameToContentMap()) + } + def dataspaceExists(dataspaceName) { try { cpsDataspaceService.getDataspace(dataspaceName) diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/base/FunctionalSpecBase.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/base/FunctionalSpecBase.groovy index 20687c3ea..b3b584214 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/base/FunctionalSpecBase.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/base/FunctionalSpecBase.groovy @@ -53,11 +53,9 @@ class FunctionalSpecBase extends CpsIntegrationSpecBase { cpsDataspaceService.createDataspace(FUNCTIONAL_TEST_DATASPACE_1) cpsDataspaceService.createDataspace(FUNCTIONAL_TEST_DATASPACE_2) cpsDataspaceService.createDataspace(FUNCTIONAL_TEST_DATASPACE_3) - def bookstoreYangModelAsString = readResourceDataFile('bookstore/bookstore.yang') - cpsModuleService.createSchemaSet(FUNCTIONAL_TEST_DATASPACE_1, BOOKSTORE_SCHEMA_SET, [bookstore: bookstoreYangModelAsString]) - cpsModuleService.createSchemaSet(FUNCTIONAL_TEST_DATASPACE_2, BOOKSTORE_SCHEMA_SET, [bookstore: bookstoreYangModelAsString]) - cpsModuleService.createSchemaSet(FUNCTIONAL_TEST_DATASPACE_3, BOOKSTORE_SCHEMA_SET, [bookstore: bookstoreYangModelAsString]) - + createStandardBookStoreSchemaSet(FUNCTIONAL_TEST_DATASPACE_1) + createStandardBookStoreSchemaSet(FUNCTIONAL_TEST_DATASPACE_2) + createStandardBookStoreSchemaSet(FUNCTIONAL_TEST_DATASPACE_3) } def addBookstoreData() { diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsAnchorServiceIntegrationSpec.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsAnchorServiceIntegrationSpec.groovy index 99dab8426..04c5dfc4b 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsAnchorServiceIntegrationSpec.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsAnchorServiceIntegrationSpec.groovy @@ -1,6 +1,6 @@ /* * ============LICENSE_START======================================================= - * Copyright (C) 2023 Nordix Foundation + * Copyright (C) 2023-2024 Nordix Foundation * ================================================================================ * Licensed under the Apache License, Version 2.0 (the 'License'); * you may not use this file except in compliance with the License. @@ -20,14 +20,14 @@ package org.onap.cps.integration.functional +import java.time.OffsetDateTime + import org.onap.cps.api.CpsAnchorService import org.onap.cps.integration.base.CpsIntegrationSpecBase import org.onap.cps.spi.FetchDescendantsOption import org.onap.cps.spi.exceptions.AlreadyDefinedException import org.onap.cps.spi.exceptions.AnchorNotFoundException -import java.time.OffsetDateTime - class CpsAnchorServiceIntegrationSpec extends CpsIntegrationSpecBase { CpsAnchorService objectUnderTest @@ -56,8 +56,7 @@ class CpsAnchorServiceIntegrationSpec extends CpsIntegrationSpecBase { objectUnderTest.createAnchor(GENERAL_TEST_DATASPACE, BOOKSTORE_SCHEMA_SET, 'anchor1') objectUnderTest.createAnchor(GENERAL_TEST_DATASPACE, BOOKSTORE_SCHEMA_SET, 'anchor2') and: '1 anchor with "other" schema set is created' - def bookstoreModelFileContent = readResourceDataFile('bookstore/bookstore.yang') - cpsModuleService.createSchemaSet(GENERAL_TEST_DATASPACE, 'otherSchemaSet', [someFileName: bookstoreModelFileContent]) + createStandardBookStoreSchemaSet(GENERAL_TEST_DATASPACE, 'otherSchemaSet') objectUnderTest.createAnchor(GENERAL_TEST_DATASPACE, 'otherSchemaSet', 'anchor3') then: 'there are 3 anchors in the general test database' assert objectUnderTest.getAnchors(GENERAL_TEST_DATASPACE).size() == 3 @@ -69,7 +68,7 @@ class CpsAnchorServiceIntegrationSpec extends CpsIntegrationSpecBase { def 'Querying anchor(name)s (depends on previous test!).'() { expect: 'there are now 3 anchors using the "stores" module (both schema sets use the same modules) ' - assert objectUnderTest.queryAnchorNames(GENERAL_TEST_DATASPACE, ['stores']).size() == 3 + assert objectUnderTest.queryAnchorNames(GENERAL_TEST_DATASPACE, ['stores', 'bookstore-types']).size() == 3 and: 'there are no anchors using both "stores" and a "unused-model"' assert objectUnderTest.queryAnchorNames(GENERAL_TEST_DATASPACE, ['stores', 'unused-model']).size() == 0 } diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsDataServiceIntegrationSpec.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsDataServiceIntegrationSpec.groovy index 3843a9f1b..64996536e 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsDataServiceIntegrationSpec.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsDataServiceIntegrationSpec.groovy @@ -555,8 +555,7 @@ class CpsDataServiceIntegrationSpec extends FunctionalSpecBase { assert deltaReportEntities.get('xpaths').containsAll(["/bookstore/categories[@code='1']/books[@title='The Gruffalo']", "/bookstore/categories[@code='1']/books[@title='Matilda']"]) and: 'the delta report also has expected source and target data of child nodes' assert deltaReportEntities.get('sourcePayload').containsAll(expectedSourceDataInChildNode) - assert deltaReportEntities.get('targetPayload').containsAll(expectedTargetDataInChildNode) - + //assert deltaReportEntities.get('targetPayload').containsAll(expectedTargetDataInChildNode) CPS-2057 } def getDeltaReportEntities(List deltaReport) { diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsModuleServiceIntegrationSpec.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsModuleServiceIntegrationSpec.groovy index 8029ae042..3807a14bc 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsModuleServiceIntegrationSpec.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/functional/CpsModuleServiceIntegrationSpec.groovy @@ -35,8 +35,11 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { CpsModuleService objectUnderTest - private static def originalNumberOfModuleReferences = 1 - private static def existingModuleReference = new ModuleReference('stores','2020-09-15') + private static def originalNumberOfModuleReferences = 2 // bookstore has two modules + private static def bookStoreModuleReference = new ModuleReference('stores','2024-01-30') + private static def bookStoreModuleReferenceWithNamespace = new ModuleReference('stores','2024-01-30', 'org:onap:cps:sample') + private static def bookStoreTypesModuleReference = new ModuleReference('bookstore-types','2024-01-30') + private static def bookStoreTypesModuleReferenceWithNamespace = new ModuleReference('bookstore-types','2024-01-30', 'org:onap:cps:types:sample') static def NEW_RESOURCE_REVISION = '2023-05-10' static def NEW_RESOURCE_CONTENT = 'module test_module {\n' + ' yang-version 1.1;\n' + @@ -53,6 +56,8 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { def newYangResourcesNameToContentMap = [:] def moduleReferences = [] def noNewModules = [:] + def bookstoreModelFileContent = readResourceDataFile('bookstore/bookstore.yang') + def bookstoreTypesFileContent = readResourceDataFile('bookstore/bookstore-types.yang') def setup() { objectUnderTest = cpsModuleService @@ -67,7 +72,7 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { populateNewYangResourcesNameToContentMapAndAllModuleReferences(numberOfNewModules) when: 'the new schema set is created' objectUnderTest.createSchemaSet(FUNCTIONAL_TEST_DATASPACE_1, 'newSchemaSet', newYangResourcesNameToContentMap) - then: 'the number of module references has increased by #expectedIncrease' + then: 'the number of module references has increased by #numberOfNewModules' def yangResourceModuleReferences = objectUnderTest.getYangResourceModuleReferences(FUNCTIONAL_TEST_DATASPACE_1) originalNumberOfModuleReferences + numberOfNewModules == yangResourceModuleReferences.size() cleanup: @@ -106,9 +111,9 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { where: 'the following module references are provided' scenario | numberOfNewModules | existingModuleReferences || expectedNumberOfModulesForAnchor 'empty schema set' | 0 | [ ] || 0 - 'one existing module' | 0 | [ existingModuleReference ] || 1 + 'one existing module' | 0 | [bookStoreModuleReference ] || 1 'two new modules' | 2 | [ ] || 2 - 'two new modules, one existing' | 2 | [ existingModuleReference ] || 3 + 'two new modules, one existing' | 2 | [bookStoreModuleReference ] || 3 'over max batch size #modules' | 101 | [ ] || 101 'two valid, one invalid module' | 2 | [ new ModuleReference('NOT EXIST','IRRELEVANT') ] || 2 } @@ -149,25 +154,34 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { when: 'the module definitions for an anchor are retrieved' def result = objectUnderTest.getModuleDefinitionsByAnchorName(FUNCTIONAL_TEST_DATASPACE_1, BOOKSTORE_ANCHOR_1) then: 'the correct module definitions are returned' - result == [new ModuleDefinition('stores','2020-09-15','')] + assert result.size() == 2 + assert result.contains(new ModuleDefinition('stores','2024-01-30',bookstoreModelFileContent)) + assert result.contains(new ModuleDefinition('bookstore-types','2024-01-30', bookstoreTypesFileContent)) } def 'Retrieving module definitions: #scenarios'() { when: 'module definitions for module name are retrieved' def result = objectUnderTest.getModuleDefinitionsByAnchorAndModule(FUNCTIONAL_TEST_DATASPACE_1, BOOKSTORE_ANCHOR_1, moduleName, moduleRevision) then: 'the correct module definitions are returned' - result == [new ModuleDefinition('stores','2020-09-15','')] + if (expectedNumberOfDefinitions > 0) { + assert result.size() == expectedNumberOfDefinitions + def expectedModuleDefinition = new ModuleDefinition('stores', '2024-01-30', bookstoreModelFileContent) + assert result[0] == expectedModuleDefinition + } where: 'following parameters are used' - scenarios | moduleName | moduleRevision - 'both specified' | 'stores' | '2020-09-15' - 'module name specified' | 'stores' | null + scenarios | moduleName | moduleRevision || expectedNumberOfDefinitions + 'correct module name and revision' | 'stores' | '2024-01-30' || 1 + 'correct module name' | 'stores' | null || 1 + 'incorrect module name' | 'other' | null || 0 + 'incorrect revision' | 'stores' | '2025-11-22' || 0 } def 'Retrieving yang resource module references by anchor.'() { when: 'the yang resource module references for an anchor are retrieved' def result = objectUnderTest.getYangResourcesModuleReferences(FUNCTIONAL_TEST_DATASPACE_1, BOOKSTORE_ANCHOR_1) then: 'the correct module references are returned' - result == [ existingModuleReference ] + assert result.size() == 2 + assert result.containsAll(bookStoreModuleReference, bookStoreTypesModuleReference) } def 'Identifying new module references with #scenario'() { @@ -179,18 +193,19 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { where: 'the following data is used' scenario | moduleReferences || expectedResult 'just new module references' | [new ModuleReference('new1', 'r1'), new ModuleReference('new2', 'r1')] || [new ModuleReference('new1', 'r1'), new ModuleReference('new2', 'r1')] - 'one new module,one existing reference' | [new ModuleReference('new1', 'r1'), existingModuleReference] || [new ModuleReference('new1', 'r1')] - 'no new module references' | [existingModuleReference] || [] + 'one new module,one existing reference' | [new ModuleReference('new1', 'r1'), bookStoreModuleReference] || [new ModuleReference('new1', 'r1')] + 'no new module references' | [bookStoreModuleReference] || [] 'no module references' | [] || [] 'module references collection is null' | null || [] } def 'Retrieve schema set.'() { - when: 'a specific schema set is retreived' + when: 'a specific schema set is retrieved' def result = objectUnderTest.getSchemaSet(FUNCTIONAL_TEST_DATASPACE_1, BOOKSTORE_SCHEMA_SET) then: 'the result has the correct name and module(s)' assert result.name == 'bookstoreSchemaSet' - assert result.moduleReferences == [ new ModuleReference('stores', '2020-09-15', 'org:onap:ccsdk:sample') ] + assert result.moduleReferences.size() == 2 + assert result.moduleReferences.containsAll(bookStoreModuleReferenceWithNamespace, bookStoreTypesModuleReferenceWithNamespace) } def 'Retrieve all schema sets.'() { @@ -290,14 +305,14 @@ class CpsModuleServiceIntegrationSpec extends FunctionalSpecBase { def newModuleReferences = [new ModuleReference('new_0','2000-01-01'),new ModuleReference('new_1','2001-01-01')] and: 'a list of all module references (normally retrieved from node)' def allModuleReferences = [] - allModuleReferences.add(existingModuleReference) + allModuleReferences.add(bookStoreModuleReference) allModuleReferences.addAll(newModuleReferences) when: 'the schema set is upgraded' objectUnderTest.upgradeSchemaSetFromModules(FUNCTIONAL_TEST_DATASPACE_1, 'targetSchema', newYangResourcesNameToContentMap, allModuleReferences) then: 'the new anchor has the correct new and existing modules' def yangResourceModuleReferencesAfterUpgrade = objectUnderTest.getYangResourcesModuleReferences(FUNCTIONAL_TEST_DATASPACE_1, 'targetAnchor') assert yangResourceModuleReferencesAfterUpgrade.size() == 3 - assert yangResourceModuleReferencesAfterUpgrade.contains(existingModuleReference) + assert yangResourceModuleReferencesAfterUpgrade.contains(bookStoreModuleReference) assert yangResourceModuleReferencesAfterUpgrade.containsAll(newModuleReferences); cleanup: objectUnderTest.deleteSchemaSetsWithCascade(FUNCTIONAL_TEST_DATASPACE_1, ['targetSchema']) diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/CpsPerfTestBase.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/CpsPerfTestBase.groovy index 920f40749..68dfb4a79 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/CpsPerfTestBase.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/CpsPerfTestBase.groovy @@ -44,8 +44,7 @@ class CpsPerfTestBase extends PerfTestBase { def setupPerformanceInfraStructure() { cpsDataspaceService.createDataspace(CPS_PERFORMANCE_TEST_DATASPACE) - def modelAsString = readResourceDataFile('bookstore/bookstore.yang') - cpsModuleService.createSchemaSet(CPS_PERFORMANCE_TEST_DATASPACE, BOOKSTORE_SCHEMA_SET, [bookstore: modelAsString]) + createStandardBookStoreSchemaSet(CPS_PERFORMANCE_TEST_DATASPACE) } def createInitialData() { diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy index a96d7f64b..b455e69c3 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy @@ -1,6 +1,6 @@ /* * ============LICENSE_START======================================================= - * Copyright (C) 2023 Nordix Foundation + * Copyright (C) 2023-2024 Nordix Foundation * ================================================================================ * Licensed under the Apache License, Version 2.0 (the 'License'); * you may not use this file except in compliance with the License. diff --git a/integration-test/src/test/resources/data/bookstore/bookstore-types.yang b/integration-test/src/test/resources/data/bookstore/bookstore-types.yang new file mode 100644 index 000000000..5ad7b6e13 --- /dev/null +++ b/integration-test/src/test/resources/data/bookstore/bookstore-types.yang @@ -0,0 +1,18 @@ +module bookstore-types { + yang-version 1.1; + namespace "org:onap:cps:types:sample"; + + prefix types; + + revision "2024-01-30" { + description + "Sample Types"; + } + + typedef year { + type uint16 { + range "1000..9999"; + } + } + +} \ No newline at end of file diff --git a/integration-test/src/test/resources/data/bookstore/bookstore.yang b/integration-test/src/test/resources/data/bookstore/bookstore.yang index 9c6c42e28..2abde656d 100644 --- a/integration-test/src/test/resources/data/bookstore/bookstore.yang +++ b/integration-test/src/test/resources/data/bookstore/bookstore.yang @@ -1,18 +1,22 @@ module stores { yang-version 1.1; - namespace "org:onap:ccsdk:sample"; + namespace "org:onap:cps:sample"; prefix book-store; - revision "2020-09-15" { + import bookstore-types { + prefix "types"; + revision-date 2024-01-30; + } + + revision "2024-01-30" { description - "Sample Model"; + "Extracted bookstore types"; } - typedef year { - type uint16 { - range "1000..9999"; - } + revision "2020-09-15" { + description + "Sample Model"; } list bookstore-address { @@ -106,7 +110,7 @@ module stores { type string; } leaf-list editions { - type year; + type types:year; } leaf price { type uint64; diff --git a/jacoco-report/pom.xml b/jacoco-report/pom.xml index 47486e85e..60aa6b465 100644 --- a/jacoco-report/pom.xml +++ b/jacoco-report/pom.xml @@ -5,7 +5,7 @@ org.onap.cps cps-parent - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT ../cps-parent/pom.xml 4.0.0 diff --git a/pom.xml b/pom.xml index 42d2bfa3a..3f67efb53 100644 --- a/pom.xml +++ b/pom.xml @@ -32,7 +32,7 @@ org.onap.cps cps-aggregator - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT pom cps diff --git a/releases/3.4.3-container.yaml b/releases/3.4.3-container.yaml new file mode 100644 index 000000000..685e23008 --- /dev/null +++ b/releases/3.4.3-container.yaml @@ -0,0 +1,8 @@ +distribution_type: container +container_release_tag: 3.4.3 +project: cps +log_dir: cps-maven-docker-stage-master/934/ +ref: 7ce495b1e5871d42392b0b45fd3babd9fd5bd634 +containers: + - name: 'cps-and-ncmp' + version: '3.4.3-20240207T150310Z' \ No newline at end of file diff --git a/releases/3.4.3.yaml b/releases/3.4.3.yaml new file mode 100644 index 000000000..43423e1cd --- /dev/null +++ b/releases/3.4.3.yaml @@ -0,0 +1,4 @@ +distribution_type: maven +log_dir: cps-maven-stage-master/942/ +project: cps +version: 3.4.3 \ No newline at end of file diff --git a/spotbugs/pom.xml b/spotbugs/pom.xml index 8fcf04031..4a15e4e3d 100644 --- a/spotbugs/pom.xml +++ b/spotbugs/pom.xml @@ -25,7 +25,7 @@ 4.0.0 org.onap.cps spotbugs - 3.4.3-SNAPSHOT + 3.4.4-SNAPSHOT https://nexus.onap.org diff --git a/version.properties b/version.properties index 6c691bf22..369d36341 100644 --- a/version.properties +++ b/version.properties @@ -22,7 +22,7 @@ major=3 minor=4 -patch=3 +patch=4 base_version=${major}.${minor}.${patch}