...
| Source | Class | Status | Comment | Proposed Resolution | JIRA Task | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| n/a | Not a Licence model issue, added comment (03/05) | Introduce in ONAP wiki an update list of approved classes with reference links to detailed description (a page per Release of some part of clean IM is not a long term viable way). Michela Bevilacqua (2020/02/20): why it is required: we need a way to identify classes already approved when an approval request is sent. Some additional comments about current model documentation in Readthedocs to address for improvements:
| No direct impact on License Model. Xu Yang Action to be progressed: This should be addressed to the modeling subcommittee as a whole. Modeling RTD Link: this is the modeling readthedocs for el alto release | Michela Bevilacqua(2020/03/05) : I would suggest to open anyhow a Jira ticket to trace the activity. It can be assigned to Jacqueline that is providing a first attempt for discussion | |||||||||
| all | Agreed | Introduce in any model proposal, the inheritance tree diagram for any new class (e.g. what is the Licensing agreement inheritance tree ?) | Yes. Andy Mayer (2020/02/17) Action to be progressed: Will include inheritance tree diagram. |
| |||||||||
| n/a | Reviewed, added comment (03/05) | #1 Is the current model proposal “as built model” going to cover the licensing model only for VNF considering it refers only to VNFD ? | The scope of the "as built" model is presently VNF and VNFD, but nothing prevents it from being expanded in the future. No further action Michela Bevilacqua (2020/03/05). I would suggest for documentation purpose, to add a label in the diagram (i.e. main diagram as shown in section 2.1.1, License Management, about this decision to explain to any users also in future why we have decided these relationships) | Open JIRA to add note to diagram. | |||||||||
| n/a | Reviewed | #3 License keys, license agreements, and contractual details such as entitlement information, are all sensitive commercial information.
| Presently much of this information is collected in SDC, and then distributed to other appropriate systems. Remember – this is “as built” information model. Security of the systems and related network is the responsibility of the service provider. Also details of the actual licenses and entitlements are held in an asset management system (e.g., license repository) outside the present scope of ONAP. No further action | ||||||||||
EntitlementInstance LicenseKeyInstance | Agreed | #8 Entitlement instance / LicenseKeyInstance classes refer to “As Built in ASDC”.
| LicenseKeyPool and EntitlementPool are generated in SDC. LicenseKeyInstance and EntitlementInstance are not – they are in the operator’s license/asset repository. Reference to this repository is in A&AI. Make the LicenseKeyInstance and EntitlementInstance classes experimental, make the attributes experimental, and remove the reference stereotype. Andy Mayer (2020/02/17) Action to be progressed: AI: Will provide further review for the attributes of LicenseKeyInstance and EntitlementInstance. If further clarification cannot be provided, attributes will be removed. Michela Bevilacqua (2020/02/20): I suggest we clarify in the definition of these classes or in a note associated to them, they are experimental (today) as not used and stored by ONAP platform. Andy Mayer (2020/02/17) Note: AAI represents Unique ID of a license resource for generic-vnf and vce, Just the UUID, relationship to license-group, and version are represented. ACTION: Ensure that the LicenseKeyInstance and EntitlementInstance remain experimental after approval. Add a note to the descriptions of each class stating that these classes are not currently used and stored by the ONAP platform |
| |||||||||
| All | Agreed | #2 Mainly all the class definition in this proposal refer to the concept of software/software product more than network functions but software/sw product is not defined in ONAP model.
| A fair observation. The Vnf is software, and is the product of some vendor that is then being incorporated into an operator’s resource. Definitions should be updated and generalized - this is the original documentation. One option is to generalize when a Pnf or some other non-software resource is added. In addition, we can update “softwareAssetTag” to something like “licenseAssetTag”. Michela Bevilacqua (2020/02/20): the comment is not only about the attribute softwareAssetTag but more about the fact that software is widely used in any class description. There are 38 software instances today in the wiki page. We need to clarify the relation of sotfware vs vnf instance and vnfd. For instance, does software map unique with a vnf instance? Action: update "software" to "VNF Descriptor" throughout License IM |
| |||||||||
| EntitlementPool | Agreed | #13 Entitlement Pool definition is partially duplicated, to be reviewed.
| The duplicate statement was removed. Text was updated to say “Controllers may request entitlements”. Michela Bevilacqua (2020/02/20):
Action: remove the "Controller will..." and "Controller may..." statements from EntitlementPool and LicenseKeyPool classes. |
| |||||||||
| LicenseKeyPool | Agreed | #16 The description of LicenceKeyPool refers to “Asset Inventory”, which is not part of ONAP and not described anywhere in the proposal | The Asset Inventory is presently outside of ONAP just as you say. This is the inventory where actual keys and entitlements would be stored and managed. Michela Bevilacqua (2020/02/20): see point 2 in comment #13 Andy Mayer Check to see of Controller - Asset Inventory interface is described within ONAP. If not, remove these descriptions. Action: Update LicenseKeyPool Description to remove "Asset Inventory" |
| |||||||||
| All | Reviewed, added comment (03/05) | "General" #2 Mainly all class relationships are defined only in the diagram by the relationship name and their cardinality. Today a class definition does not include any relationship definition.
| A fair observation. The Vnf is software, and is the product of some vendor that is then being incorporated into an operator’s resource. Definitions should be updated and generalized - this is the original documentation. One option is to generalize when a Pnf or some other non-software resource is added. In addition, we can update “softwareAssetTag” to something like “licenseAssetTag”. Michela Bevilacqua (2020/02/20): the problem is not specific to software, so I do not understand the proposed resolution/the proposed resolution is not applicable. The problem is more that we need to define and document the relationships between the classes. An arrow with a relationship name in a diagram is not clear enough. We recommend to introduce the relationship definiton in the class definition table. Xu Yang Action to be progressed: This should be addressed to the modeling subcommittee as a whole. | Michela Bevilacqua (2020/03/05) for the comprehension of the model is an important step. I reccomend to open a Jira ticket to be porgressed. Xu Yang to open JIRA. | |||||||||
EntitlementPool LicenseKeyPool | Reviewed, added comment 03/05 | #6 We do not have evidence of the full list of attributes defined in the ONAP as built in model in particular for EntitlementPool and LicenseKeyPool classes. Considering the attributes defined in SDC UI (https://wiki.onap.org/display/DW/Resource+Onboarding, only a subset of attributes should be defined in ONAP model.
| Can provide screenshots of UIs as well as a view of the XML being distributed from SDC. XML from SDC can be found at - https://wiki.onap.org/display/DW/SDC+Deployment+Artifacts. No further action Michela Bevilacqua (2020/03/05): SDC artifacts referenced above and the SDC UI documentation (linked in the comment #6) do not provide evidence of the SP and Vendor Pool Limit in the EntitlementPool and LicensekeyPool but only of Threshold Value and Unit. So I suggest to remove them. There is no evidence of VendorPoolLimit provided by the vendor in ONAP. | Andy Mayer will investigate SPPoolLimit and VendorPoolLimit attributes | |||||||||
| LicenseAgreement | Reviewed | #11 LicenseAgreement class contains free-form text information in the attributes requirementsAndConstraints and statementOfIntent that is not usable in any automation UC and thus should instead be contained in an external document | A fair point, but that is what is built. One could parse that constraint, and if the creator and receiver have agreed on some common semantics, then it could certainly be used via automation. As built representation No further action | ||||||||||
| DesignEntity | Agreed | #12 The “validFor” attribute in EntitlementPool and LicenseKeyPool (inherited from DesignEntity) is mandatory. It’s not obvious whether this validity value then applies to all instances in the pool.
| This is analogous to a Vnfd and and VnfInstance, or a ProductSpec and a Product. One can certainly put in an enddate of something very far in the future, allowing for an unlimited amount of time. We could redefine the property and make it optional as well. Action: change validFor attribute in EntitlementPool and LicenseKeyPool (inherited from DesignEntity) to optional (0..1) |
| |||||||||
| EntitlementPool | Agreed | #14 Entitlement Pool class: entitlementManufactureReferenceNumber has a multiplicity 0..1 and it is defined as “identifier for the entitlement as described…”.
| Not clear why this attribute is not required (updated from 0..1 to 1). Name aligns with the SDC screen, and it does state that it is a Manufacturer’s Reference Number. Updated description, using phrase reference number rather than identifier. ACTION: Entitlement Pool class: entitlementManufactureReferenceNumber change back to optional (0..1) |
| |||||||||
| LicenseKeyInstance | Agreed, added comment (03/05) | #15 LicenseKey attribute (in LicenseKeyInstance) is defined twice, once as a String (with upper-case “k”) and once using the type “File” (with lower-case “k”).
| The attribute with type of “File” was marked as Future to make it clear that this is not “as built”. It is an item presently under development, implementation under way but not yet complete in SDC. Suggestion was to keep this as future, it does highlight present activity. Updated name to licenseKeyFile to distinguish the attributes. Michela Bevilacqua (2020/02/20): The presence of these attributes is subordinated to the action agreed in the comment #8. However, as the purpose of this model is to document the as built SDC solution and File is today not supported, I suggest to remove it to be then added in the future. ACTION: keep stereotype as Experimental after approval. Michela Bevilacqua (2020/03/05): I do not understand main response above . We have seen that EntitlementInstance and LicenseKeyInstance are not managed in ONAP. so why there is a comment about an ongoing development in SDC ? I´m not comfortable to mantain a "future" attribute in the current model. I ask to remove this attribute as it is not part of the as built solution. |
Note: use only Lifecycle Stereotype Proposal | |||||||||
| EntitlementInstance | Agreed, added comment 03/05 | #17 Entitlement instance class, SoftwareAssetTag: swAssetTag not defined in SDC UI. What is a softwareAssetTag ? What is the relation with the ONAP classes ? | Given EntitlementInstances are stored in the Asset Inventory references in question 16, it’s attributes, including swAssetTag would therefore not be found in SDC. Per the definition, this is a operator internally generated value. See response to question 8 Action to be progressed: AI: Will provide further review for the attributes. If further clarification cannot be provided, attributes will be removed. Andy Mayer SoftwareAssetTag could not be found in AAI swagger file. ACTION: Remove attribute SoftwareAssetTag from EntitlementInstance and LicenseKeyInstance classes Michela Bevilacqua 2020/03/05: EntitlementInstance and LicenseKeyInstance classes includes assignmentStatus attribute defined as String but it seems that only specific values are allowed even if not defined. Please define the list of values. In release readthedocs: should we filter or annotate Experiment modeling elements? Xu Yang |
Xu Yang to add JIRA to discuss treatment of Experimental modeling elements. | |||||||||
EntitlementInstance LicenseKeyInstance | Agreed | #18 EntitlementInstance and LicenseKeyInstance classes include the mandatory attribute ssmUserId.
| Still researching. Action to be progressed: AI: Will provide further review for the attributes . If further clarification cannot be provided, attributes will be removed. Andy Mayer SSM User ID not found in AAI swagger file ACTION: Remove attribute ssmUserId from EntitlementInstance and LicenseKeyInstance classes |
| |||||||||
| Vendor | Agreed | #4 “Vendor” class in the top level diagram is not defined in this proposal. It was originally included in the R5 “Party” proposal which was never approved. Current modeling proposal depends on Vendor class due to the introduction of the relation with LicenseAgreement and the definition of an indirect relation through the vendor of the VNFD with the LicensingAgreement.
| At the time the root hierarchy was developed, we agreed to leave the approval of the Party related concepts until it was needed – the license Andy Mayer (2020/02/17): Action to be progressed: Will add description of Vendor Class (and superclasses) to the License model page proposal page. Reference the relevant classes from the Common:Party model. see: Party Michela Bevilacqua (2020/02/20): I assume that the change of scope of the current model review, (including now Party model), will require a new vote for approval. Andy Mayer (2020/02/27) We will import the Vendor class and relevant superclasses from the Common model to the wiki for approval. |
| |||||||||
| SequenceFlows | Agreed | #5 There are two EMPTY classes under SequenceFlows , “License::SequenceFlows::License Setup” and “SimpleOrderFlow class” – these have NO descriptions, NOT included in any diagrams, and are NOT thus possible to understand in the context of the proposal.
| These Papyrus (not information model) classes are the basis of two simple sequence diagram flows. It appears that the diagrams did not get included into the lasted WIKI post (see version 3) . Andy Mayer Action to be progressed:We will incorporate the diagrams on the wiki with the next update. Michela Bevilacqua (2020/02/20): A review the model is required when the two classes will be added. ACTION: Add sequence flow diagrams for review (note these are not Information Classes) |
| |||||||||
EntitlementPool LicenceKeyPool FeatureGroup | Reviewed | #7 Three related inconsistencies:
| 1) Updated to allow zero to many 2) Updated to allow zero to many 3) This is covered by changing the cardinality to 0..* above. Model has been updated No further action | ||||||||||
| LicenseAgreement | Reviewed | #9 VNFD – License Agreement relationship: There is no direct relation between the LicenseAgreement class and the VNFD class but objects are connected through the Vendor class losing the possibility to detect the VNFD/VNF type from a LicensingAgreement.
| Given there is a relationship between vendor and Vnfd as well as FeatureGroup and Vnfd, a direct relationship between LicenseAgreement and Vnfd is not needed. Possible future improvement: Analyze direct relationship from Vnfd to License Agreement No further action | ||||||||||
| VNFinstance | Agreed | #10 A VNFinstance can only have a Entitlement instance and a LKinstance.
| Please clarify what else you see as being needed. Action: Analyze cardinality of relationship between VNFInstance and LKIntance / EntitlementInstance (should a VNFInstance be able to be related to more than one LKInstance) Andy Mayer? Check AAI Andy Mayer 2020/02/27 AAI swagger does not restrict a VNF to a single instance. ACTION: Update cardinality of VNFInstance relationships to allow 0..* EntitlementInstance and LicenseKey Instance |
| |||||||||
LicenseAgreement FeatureGroup | Reviewed | #19 It’s not clear why the LicenseAgreementHasFeatureGroup association has cardinality 1..* on each end.
| Presumably, it is reasonable that a licenseagreement can have some number of feature groups. Similarly, a featuregroup can be related to multiple licenseagreements. We will check and see if SDC will support this. Michela – your perspective on this? Possible Action: include additional description for this relationship Andy Mayer 2020/02/28: We are investigating SDC behavior to verify. | ||||||||||
FeatureGroup EntitlementPool | Reviewed | #20 It’s not clear why the FeatureGroupHasEntitlementPool association has cardinality 1..* on the EntitlementPool end.
| Presently, in the SDC software, the featuregroup MUST have a relationship to an entitlementpool. There is no such requirement for licensekeypool – it is optional. Hence the present multiplicity on the “pool” side of these relationships Possible Action: Describe difference in cardinality in the Class description of FeatureGroup Andy Mayer 2020/02/28: Feature Group must have at least one Entitlement Pool, because the Entitlement Pool represents the actual license assets that are being purchased. No further Action | ||||||||||
Michela Bevilacqua(2020/02/20) | General | Agreed | Substitute the name of ASDC with SDC to refer to the ONAP component. | Agree!!! |
| ||||||||
Michela Bevilacqua (2020/02/20) | Entitlement /License Key instance | Agreed | #8+: Entitlement /License Key instance class definition to be clarified | See resolution to #8 |
| ||||||||
...