CPS-872 CM Handle State: define and agree new dmi-registry yang model supporting States
References
CPS-872: CM Handle State: define and agree new dmi-registry yang model supporting StatesClosed
CPS-799 Spike: Define states and state handling for CM handle
Ā
Tasks
NCMP should be able to handle a large (10,000s!) batch of registrations at once. To make this possible we need toĀ
Task | |
---|---|
1 | Create updated DMI Registry Yang Schema (using @yyyy-mm-dd) inĀ changelog/db/changes/data/yang-models/dmi-registry @ 2021-12-13.yangĀ to store Handle State |
2 | AddĀ State,Ā LockReasonĀ andĀ LockReasonDetailsĀ as (Yang)Strings to the schema. Any validation or enum-limitations can be handled in the Java code. |
3 | Consider timestamp last-update-time for retry and timeout related scenarios as part of same schema update to reduce overhead of Liquibase changesetsĀ |
4 | Test/demo using CPS-CoreĀ
|
Ā
Yang Revision Proposal
Reordered the revision tags based on reverse chronical order convention
Added state leaf to hold the synchronization state information
Added lock-reason leaf to hold a lock reason enum
Added lock-reason-details leaf non enum description of lock reason. More descriptive and scenario based.
Added model-sync-start-time to hold the start time of when the most recent sync attempt on this cmHandle was started.Ā
Ā
dmi-registry@2022-02-10.yang
module dmi-registry {
yang-version 1.1;
namespace "org:onap:cps:ncmp";
prefix dmi-reg;
contact "dylan.byrne@est.tech";
revision "2022-02-10" {
description
"Added State, LockReason, LockReasonDetails to aid with cmHandle sync and timestamp to aid with retry/timeout scenarios";
}
revision "2021-12-13" {
description
"Added new list of public additional properties for a Cm-Handle which are exposed to clients of the NCMP interface";
}
revision "2021-10-20" {
description
"Added dmi-data-service-name & dmi-model-service-name to allow separate DMI instances for each responsibility";
}
revision "2021-05-20" {
description
"Initial Version";
}
container dmi-registry {
list cm-handles {
key "id";
leaf id {
type string;
}
leaf dmi-service-name {
type string;
}
leaf dmi-data-service-name {
type string;
}
leaf dmi-model-service-name {
type string;
}
list additional-properties {
key "name";
leaf name {
type string;
}
leaf value {
type string;
}
}
list public-properties {
key "name";
leaf name {
type string;
}
leaf value {
type string;
}
}
leaf state {
type string;
}
leaf lock-reason {
type string;
}
leaf lock-reason-details {
type string;
}
leaf last-update-time {
type string;
}
}
}
}
Ā
Timestamp
last-update-time will be recorded once model/data sync attempt is started and will be removed once sync is completed. CmHandle state should be changed from advised to locked during sync. Extending the query from ncmp to look for cmHandle state and modelSyncStartTime would remove need for watchdog.
Ā
Prevention of multiple NCMP attempting sync
It is possible for multiple NCMP instances to attempt to sync the same cmHandle from the singular cps instance. In order to prevent this we can implement an exclusive lock using the PESSIMISTIC_WRITE JPA locking mechanism. This will prevent more than one NCMP instance from reading, updating or deleting data related to that cmHandle.Ā
Ā