Merge "Merge 2 'query' end points in NCMP"

Merge 2 'query' end points in NCMP

- merge two endpoint for a same backend
- use xPath query instead of sql query
- modify searches endpoint to return a cmHandle object with all public
properties
- handle old (deprecated) queries
- handle public property queries
- create useful examples
- use more verbose error messages
- simplify openapi yamls
- create new query service
- change second endpoint name to a better matched name
- modify legacy tests with new requirements
- create new tests for the new scenarios

Issue-ID: CPS-1016
Change-Id: I7476e9dbd510ec93b5b48ce85d477ecb2dadffff
Signed-off-by: kissand <andras.zoltan.kiss@est.tech>

Merge "CPS-1020 : DuplicatedYangResourceException error at parallel cmHandle registration"

CPS-1020 : DuplicatedYangResourceException error at parallel cmHandle registration

- @EnableRetry is added into application.
- Added retry count into log to add more information about retry.

Issue-ID: CPS-1020
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I00c58ef17a8c066f98711c1cf8e5e5cb5f22974b

Merge "Add module name to cps core output"

Merge "NcmpEvent creation for ModuleSync"

Add module name to cps core output

- add withModuleName method
- add getNodeIdentifierWithPrefix method
- add unit test in DataMapUtilsSpec
- add unit test in DataNodeBuilderSpec
- fix all existing unit tests

Issue-ID: CPS-870
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: I51b70fa2dd3381eef9500b4339d4922c017e3000

NcmpEvent creation for ModuleSync

- NcmpEvent schema creation which will be published when module sync is
  completed
- Added cmHandleId in the event payload and made eventCorrelationId as
  optional
- Upcoming : NcmpEvent publisher code and mapping of the events

Issue-ID: CPS-1036
Change-Id: I7731e6e6a4a8f718967ce68cc6ae0e411eb8c12b
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Module Sync Lock State implementation"

Module Sync Lock State implementation

Implementation of Lock state for module sync watchdog
Cm Handle state is locked if any exception is found
during sync process
Make changes around READY state method in line with the
new schema set
Add last updated time to composite state
Remove running datastore references as this is being done
at a later time

Issue-ID: CPS-875
Change-Id: I6bd159faefef2fa84dbf536c292ff0a132793381
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "DMI-NCMP Asynchronously Publish Response Event to Client Topic"

Fixing CompositeState mapping

- Mapper null check
- NcmpServiceCmHandle mapping for the new composite state
- Added global parameter for jackson to include only NON_NULL attributes
  in the response

Issue-ID: CPS-1066
Change-Id: I1924fef45345c77873f9c788baa701311e72bea7
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

DMI-NCMP Asynchronously Publish Response Event to Client Topic

- [NCMP] Added Unit Tests for NcmpAsyncRequestResponseEventMapper.java

Issue-ID: CPS-830
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I64dad7f0bb9b64e1864f19db5a5d74922dc6a83b

Merge "Handle RestTemplate Error handling so NCMP cna report correct server status"

Handle RestTemplate Error handling so NCMP cna report correct server status

- Moved resposne handling to DmRestClient
- Clean up incorrect/redundant tests in CmProxyDataServiceImplSpec

Issue-ID: CPS-1056
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I78ba87d6af7255d85758893cfef237d75227875a

Merge "CompositeStateBuilder added for building the compositeState"

CompositeStateBuilder added for building the compositeState

Added composite state to YangModelCmHandleRetriever

Issue-ID: CPS-878
Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>
Change-Id: I8bdea55c0a8e27a906e24fc367dedf81c9b3501c
Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>

Async request response NCMP -> Client

-Added consumer for DMI events and producer for forwarding to client
-Added schemas for events
-Updated tests
-Added new module for ncmp events
-Used mapstruct for event mapping

Issue-ID: CPS-830
Change-Id: I096d08af9d69092cf8651e11eaa00ce441fc3605
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>

Enhanced response with Complex State in API

- Introduced RestOutputCmHandleState in API specs of
  retrieveCmHandleDetailsById
- Mapper to map CompositeState to RestOutputCmHandleState
- Enhanced existing test cases and introduced new one to test the
  mapping result

Issue-ID: CPS-1047
Change-Id: I34fa198287e5d920bc0cea312ee4e368f3be2b90
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Merge "Get cm-handle public properties endpoint"

Get cm-handle public properties endpoint

Added RestOuputCmHandlePublicProperties OpenApi Object
Added Get cm-handle public properties endpoint
Added rest and service layer functionality for endpoint with tests
Fixed Copyright Checker violations

Issue-ID: CPS-1018
Signed-off-by: lukegleeson <luke.gleeson@est.tech>
Change-Id: Ifc13cde350a49f6ba705a09e31853dc9c73be168

Merge "URI is not absolute when service name is not set"

Merge "Composite State to handle dmi-reg YANG updates"

Composite State to handle dmi-reg YANG updates

- Introduce CompositeState object which handles change in updated YANG for dmi-registry
- Used Builder pattern as some of the fields are optional
- Removed the abstract ready method from CmHandleState which was used as
  state machine
- Fixed few test cases

Issue-ID: CPS-1042
Change-Id: I8aaf6f819c66b3a9d30c5e8f0a0007f9528b247f
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

URI is not absolute when service name is not set

BUG - URI is not absolute exception when dmi-service-name
is not set in the database, due to null String
Has now been set to null value

Issue-ID: CPS-1043
Change-Id: If48c34d8caee302a1849555755960dccfd26ab7a
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Remove NCMP tag from properties

We have decided to remove the NCMP tag from properties
and will add it later as part of our release process for
KOHN.

Issue-ID: CPS-875
Change-Id: I7282c3c11879f2252c8a1b193df17f3f4bb1cf14
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Watchdog-process that syncs 'ADVISED' CM Handles"

Watchdog-process that syncs 'ADVISED' CM Handles

- Sync and Create Schema Set during
module sync watchdog process.

- Add a cm handle state transition machine to handle
state changes for a cm handle during watchdog
which syncs module service.

Issue-ID: CPS-875
Change-Id: I3b178f5693cb7ac30577dd81cdc82b462555389a
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Add graceful shutdown for Session Manager"

Add graceful shutdown for Session Manager

Introduce singleton session manager
Add unit test for session manager

Issue-Id: CPS-898
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: Iaf91f1aa6c1ebfe0ab907e7f7d80a01e940a0fdd

Merge "Update DMI Registry yang to support DataStoreSyncState"

Update DMI Registry yang to support DataStoreSyncState

- Added dmi-registry@2022-05-10 yang to support DataStoreSyncState
- Added changelog 14 to add the dmi-registry

Issue-ID: CPS-1014
Change-Id: I910d2496a359fdf2441e172ca270036272d31dc6
Signed-off-by: Lathish <lathishbabu.ganesan@est.tech>

Watchdog Process-Make Configurable

Make time configurable for the watchdog process.
Add documentation outlining this.

Issue-ID: CPS-875
Change-Id: I1307c82d02789097efda1ba981eb1fd36c9efc38
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Add ncmp tag to application yaml"

Merge "Improve Local Running of CSIT Tests"

Improve Local Running of CSIT Tests

Issue-ID: CPS-734
Signed-off-by: lukegleeson <luke.gleeson@est.tech>
Change-Id: Iff92f43dc69cdaf3e1e78551f68ecfb08bde4b93

Add ncmp tag to application yaml

Add an NCMP tag to Application.yml to store
NCMP related propeties

Issue-ID: CPS-875
Change-Id: I2bce3d765ae69cc2a4bb52a472eea7cae745fd2c
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Security Logging Fields

- Made logging.format value case insensitive for consistency as we did same change for cps-temporal, ncmp-dmi-plugin and oom.

Issue-ID: CPS-986
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I085a3777e8f4cf9e6b62e40d05672ce455916a85
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>

Security Logging Fields

- logback core and encoder dependencies are added.
- New appender is added to log JSON format content on console.

Issue-ID: CPS-986
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: Ia72b27f306019fa259b3d6de4c5399f0ae734076

Merge "Fix sonar code smells"

Merge "Watchdog-process that changes CM Handles state"

Fix sonar code smells

* Increase code coverage in DataNodeBuilder

Issue-ID: CPS-475
Change-Id: I00a3b1c81911c1ccd3db810eec2f2989946809c5
Signed-off-by: kissand <andras.zoltan.kiss@est.tech>

Merge "Updated the documentation with limitation of leaf condition"

Merge "[STRIMZI] Add strimzi kafka notes"

Watchdog-process that changes CM Handles state

Add a fixed delay scheduler to switch cm-handles found in an ADVISED
state to a READY state
Scheduler currently runs every 30 seconds
Will only update a single cm-handle at a time
Queries CM-Handle with Advised States Only using CPS Path.
Will choose cm handle at random

Issue-ID: CPS-875
Change-Id: Ie1b49c89a0350d20e14748a65f9c1d260d8502d2
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Fix sonar code smells"

Fix sonar code smells

* Increase code coverage
  https://sonarcloud.io/component_measures?metric=uncovered_conditions&id=onap_cps

Issue-ID: CPS-475
Change-Id: I124bad6c01acbcb9658f52eb68079628e074168e
Signed-off-by: kissand <andras.zoltan.kiss@est.tech>

Updated the documentation with limitation of leaf condition

Issue-ID: CPS-988
Change-Id: I6d02b35b949c62f956b291a86d4abe6e1a70b3ec
Signed-off-by: Lathish <lathishbabu.ganesan@est.tech>

Add methods to Lock Anchor entity

Added separate integration and mock-based unit tests

Issue-ID: CPS-898
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: I14d1d1c41759ce028e2417fdd2df001280e19ab2

Merge "Fix Absolute Path to list with Integer/String key"

Fix Absolute Path to list with Integer/String key

 - Introduced normalizedXpath to normalize the xpath and cpspath
 - Introduced normalizedAncestorpath to normalize the ancestor path in xpath and cpspath
 - Added new condition in the ANTLR Grammar to capture the invalid path in the xpath
 - Introduced PathParsingException to replace the IllegalStateException

Issue-ID: CPS-961
Change-Id: Ie10f5c6cfc466387e79eec184b933297d1d79587
Signed-off-by: Lathish <lathishbabu.ganesan@est.tech>

Review spring framework dependency

Use default spring framework version from spring boot dependencies.
Then, cps application is now currently using spring-web 5.3.16 (from
spring boot 2.6.4) instead of previously specified 5.3.13.

Issue-ID: CPS-998
Change-Id: If16c5f4c7176a34b5a595a1411d3caa763d3c2f0
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>

Document ncmp-inventory error code in openapi

Issue-ID: CPS-896
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: Ic39da86c18ac921a6a3d213064f0b7c77d08e06f

Refactor existing model sync code into separate package

- Created a new class to seperate sync related operations of modules.
- Kept newly created class into .../api/inventory/sync package.
- Created a separate groovy test.
- Fixed existing groovy test.

Issue-ID: CPS-950
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: Ie7cc36ea75daf43f1383757bcf2057c5cb4894b1

Fix ANTLR version issue

 - Updated ANTLR compiler version to 4.9.2 to be compatible with runtime version

Issue-ID: CPS-961
Change-Id: I78a3fe541c9edcfc891fb045a60ed117d3aa5b08
Signed-off-by: Lathish <lathishbabu.ganesan@est.tech>

Add bounds to sphinx requirement

As ONAP was created before Oct 20 2020 in RTD, we
must set bounds to enforce a compatible Sphinx version
(see manual).

Issue-ID: DOC-782
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Change-Id: I5ed268d592986764bf040690a47c5024692768b7

Fix Sonar Qube Violations

Immediately return expression

Issue-ID: CPS-475
Change-Id: I2c33c94bef1ff097786299d3a037feea2b4ebbff
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Fix Sonar Qube Violations - Recurring task

Fix sonar issues in CPS.

- Remove Unnecessary cast to List
- Immediately return expession

Issue-ID: CPS-475
Change-Id: Ic3fa2f3f92784682045a780d87c6737d04f37659
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Refactoring/ Adding Tests for Validation"

Refactoring/ Adding Tests for Validation

Refactored classes affected by validation
Have added tests for anywhere where validation is used
Have refactored the parse and sync modules validation
to be validated at the public api method

Issue-ID: CPS-322
Change-Id: I4989cfd03300fbdca41571d0aa2d0b96978858ba
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Fix to stop build failure on CopyrightCheck Error"

Merge "Update info.yml for new Committer"

Update info.yml for new Committer

-Updating INFO.yaml to include new committer for CPS (JosephKeenan)

Issue-ID: CPS-985
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>
Change-Id: I08ca90749e13a977f83d173f2cd72146753c2370

Fix to stop build failure on CopyrightCheck Error

- This will make the maven build see an error thrown by the python script as a success and won't fail the build

Issue-ID: CPS-911
Signed-off-by: lukegleeson <luke.gleeson@est.tech>
Change-Id: Ibc5311de8a2f173a92e44c20c1c244f016a99588

Merge "dmi plugin version updated to 1.2.0-SNAPSHOT-latest"

Query based on Public CM Properties

-Updated OpenAPI for new Endpoint
-Will replace SQL with CPSPathQuery once investigation is complete
-Functionality in place to determine if public properties match -
-Added Unit and CSIT tests - small modifications may need to be made
-CSIT tests enhanced to add additional nodes and tests

Issue-ID: CPS-731
Change-Id: I403e603ce79c4a4a6994d51b459b5703510d5a83
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>

dmi plugin version updated to 1.2.0-SNAPSHOT-latest

CSIT updated to test the changes on Accept header

Issue-ID: CPS-957
Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>
Change-Id: I2fb4abd5dff5ab7b02345ad6b4899987c2553130

Async: NCMP Rest impl. including Request ID generation

- Restructured code and moved some of them at controller and service
  layer.
- Unit is fixed and organized to it's belonging classes.

Issue-ID: CPS-828
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I0919218e35b1d11cb579d707f376b76de80409da

[cps] Fix getResourceDataForPassthroughOperational endpoint

Get endpoint only accepts application/json.

Issue-ID: CPS-957
Signed-off-by: puthuparambil.aditya <aditya.puthuparambil@bell.ca>
Change-Id: Ifc7426fbcd6073a39554014dc18b2107b6343b16

Merge "Structured Exception details for DMI"

Structured Exception details for DMI

- Introduced DmiErrorMessage in API docs with 502 Bad Gateway
- HttpClientRequestException will be thrown which will be exposed as 502 BAD Gateway for the client from NCMP

Issue-ID: CPS-917
Change-Id: Iba8f159e8216bc1f63a9ab86208e5c802437e2e8
Signed-off-by: mpriyank <priyank.maheshwari@est.tech>

Update CM-Handle registration response

- update openapi.yml with new response structure
- send only details of failed cm-handle operations
- updated csit to validate 200 status

Issue-ID: CPS-896
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: I3b868bcc5b8ff488c31faef51edc82c771452234

Fix issue around duplicate build tag

Fixes failing build after merge of CPS-322.
Duplicate build tag found in checkstyle POM

Issue-ID: CPS-322
Change-Id: I099e4242e419d7b393e7787d3f75f9909dce52ac
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>

Merge "Additional validation for names/identifiers"

Merge "Copyright Check Script"

Additional validation for names/identifiers

Implementing additional regex validation for names/identifiers
in CPS

- Introduces a CpsValidator
- Applies to relevent java public API's
- Tests have been updated where necessary

Issue-ID: CPS-322
Change-Id: I29ab8820bb1fe0eee247e425d6bb018bcd38f28e
Signed-off-by: DylanB95EST <dylan.byrne@est.tech>

Merge "Updated the Documentation with sample yang and json"

Registration Response for Create cmhandles operations

- Changed implementation to register each cm-handle at a time
  instead of registering all at once
- Removed YangCMHandleList class and using String format to
  generated expected JSON
- Response of all three operation types is returned in the
  updateDmiRegistrationAndSyncModule method
- Changed parseAndRemoveCmHandlesInDmiRegistration to protected
  to keep in sync with create flow
- Refactored existing create CMHandle test cases

Issue-ID: CPS-896
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: I5b0c01b3b8e31ca7c257b1e04069e35268be1132

Merge "Add Start and Stop sessions on JAVA API"

Copyright Check Script

- Prints warnings to build log if copyright issues detected
- File ignore included for checkstyle folder and common extensions which don't have copyrights
- Included Tests

Issue-ID: CPS-911
Signed-off-by: lukegleeson <luke.gleeson@est.tech>
Change-Id: Idbdd050af964335cc32218e3c11c77d4101f9ecd

Registration Response for Update and Delete cmhandles operations

- Process registration in the delete, create and update order
- updateRegistration returns DMIRegistartionRespons that contains
  response for all operations
- Added logic to not delete cmhandle if schemaset deletion fails

Issue-ID: CPS-896
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: I907ff678d478894e7d4d3cc24b23a979c02ed97a

Add Start and Stop sessions on JAVA API

Issue-ID: CPS-899
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: Idbeb922790824b1ca601d6d4798df45efa57d685

Merge "Update maven build deploy version"

Updated the Documentation with sample yang and json

Issue-ID: CPS-940
Change-Id: I66b2f0cc1007a23498803cca695aa3d19c0eb433
Signed-off-by: Lathish <lathishbabu.ganesan@est.tech>

Update SpringBoot when approved version has Log4J 2.17.1 or higher

-Updated cps-dependencies/pom.xml SpringBoot to v2.6.4
-Updated cps-parent/pom.xml springboot maven to v2.6.4
-Updated spring cloud dependencies to 2021.0.1

Issue-ID: CPS-958
Change-Id: I37aca89edf6da83501414b4f202d3bb88464c801
Signed-off-by: JosephKeenan <joseph.keenan@est.tech>

Update maven build deploy version

Issue-ID: CPS-953
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: Icc79e3836a186a2a9ce79c0e97965bc7e1fbb347

Merge "Update log4j dependency version from 2.17.1 to 2.17.2"

Merge "Update docs to add info on providing additional config"

Merge "Return Registration response for updating cmhandles"

Update docs to add info on providing additional config

- Added information on overriding values of spring-supported properties

Issue-ID: CPS-918
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: I660cd9ea0c4c73d972f9c1645f5ed28308282fdc

Update log4j dependency version from 2.17.1 to 2.17.2

Issue-ID: CPS-953
Signed-off-by: emaclee <lee.anjella.macabuhay@est.tech>
Change-Id: I1f97b1864128c9e1105ecef2bfa5e598eefcd5d5

Update versions for next release

Update Minor version number (on master) for next release as part of creating
Jakarta release branch

Issue-ID: CPS-952

Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: Ie96ae7ce8997774d3ea27275c73e7d0703257e5e

Return Registration response for updating cmhandles

- Added CMHandleRegistrationResponse class for registration response
- Send response in the case of success and failure
- Each update operation in the request is independent on previous one

Issue-ID: CPS-896
Signed-off-by: Renu Kumari <renu.kumari@bell.ca>
Change-Id: Ic6cc69da48bd2078029784cabede4f5965054c03

Upgrade to next snapshot version

Issue-ID: CPS-933
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>
Change-Id: I5ba87c850aa932737cc62e312a825c69a3db5a02

Release container artifact for version 3.0.0

Issue-ID: CPS-933
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>
Change-Id: I3131b12f025248250755360fd1e3221c8fc89278

Release maven artifacts for version 3.0.0

Issue-ID: CPS-933
Change-Id: I8bede2dfd5f230e42096d54c5f6644a588c15110
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>

Create primary documentation for Jakarta release

* Update release notes
* Update openapi documentation
* Fix section title hierarchy
* Fix admin container directive not found

Issue-ID: CPS-935
Signed-off-by: Bruno Sakoto <bruno.sakoto@bell.ca>
Change-Id: Iabdc357aa0168c1becafd6de1ac7703ec3bd1afa

[STRIMZI] Add strimzi kafka notes

Signed-off-by: efiacor <fiachra.corcoran@est.tech>
Change-Id: Id85d2708bdb2eb1ddfaa72740a5a9df5599f298b
Issue-ID: DMAAP-1681

Async: NCMP Rest impl. including Request ID generation

- In case of invalid topic return http status 400 with error message
- Unit test is modified to handle/validate InvalidTopicException

Issue-ID: CPS-828
Signed-off-by: sourabh_sourabh <sourabh.sourabh@est.tech>
Change-Id: I05645c92ccebb8aa422a47f6edcde7b64088a118

Merge "Replacing ModelMapper with MapStruct"