add component to requirement for future fix of meta model #524
Conversation
RolandJentschETAS
requested review from
PandaeDo,
aschemmel-tech,
masc2023 and
pahmann
as code owners
|
The created documentation from the pull request is available at: docu-html |
| :safety: ASIL_B | ||
| :satisfies: feat_req__feature_name__some_title | ||
| :status: invalid | ||
| :belongs_to: comp__component_name_template |
There was a problem hiding this comment.
Seems related to here, #515, first time introduce belogns_to, now in templates, and will be there anothr PR to add it to the process requirements, If would expect, that such a major chang is introduced in ONE PR, otherwise it is really difficult to follow that up
There was a problem hiding this comment.
No, that's comes with the update of the meta model in december. The update of the picture is only for introduction of the logic interfaces in the picture which are there since a long time. See https://github.com/eclipse-score/docs-as-code/pull/331/files for introduction of the belongs_to.
There was a problem hiding this comment.
Do not see any process requirements for that?
There was a problem hiding this comment.
It is gd_req__arch_linkage_requirement. For that check this is necessary. Also the building block view requests that by "has".
process/folder_templates/modules/module_name/component_name/docs/architecture/index.rst
Show resolved
Hide resolved
process/folder_templates/modules/module_name/component_name/docs/requirements/index.rst
Show resolved
Hide resolved
process/folder_templates/modules/module_name/component_name/docs/requirements/index.rst
Show resolved
Hide resolved
process/process_areas/requirements_engineering/guidance/requirements_templates.rst
Show resolved
Hide resolved
process/folder_templates/modules/module_name/component_name/docs/architecture/index.rst
Show resolved
Hide resolved
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
Several findings to the metamodel picture (also ones not introduced by this change). Please correct nevertheless.
process/general_concepts/_assets/score_building_blocks_meta_model.drawio.svg
Show resolved
Hide resolved
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
Checked adaptions/explanations ok
There was a problem hiding this comment.
Software Module becomes Dependable Element
SEooC -> "Delivery container", can contain 1,* dependable_elements
May we add a new image as future wish, merge it and then make the rest consistent
@RolandJentschETAS let discuss how to manage, separate PR, etc.
Compare Image here:
https://github.com/orgs/eclipse-score/discussions/407#discussioncomment-15605348
3 participants
Add component id to requirement for future fix of meta model. Otherwise meta model update will fail in future.
Because of the cyclic dependency of between process model and docs as code repo (docs as code repo uses sphinx needs elements from process repo for integration tests, while process repo uses docs-as-code-meta model for internal examples and documentation) the changes in the meta model was introduced as as optional parameters and links. Therefore it is difficult to use the right elements. Then change will correct that, but the test is complicated, because every time the meta modell is changed, a complete rebuild (delete build folder and bazel clean is necessary). Additionally the build breaks with the first change. So the changes in the meta model must be done step by step and by revert the previous change to see the additional errors. Therefore the changes was done as different PRs in the beginning, to test any change individually. Now it is merged to one change.