fix: update safety attribute#2933
Conversation
feat_req__baselibs__core_utilities must be ASIL-B because feat_arc_sta__baselibs__static_view_arch and feat_arc_dyn__baselibs__dynamic_view_arch fulfill it and these are ASIL-B. Same with feat_req__persistency__variant_management and feat_arc_sta__persistency__static.
|
The created documentation from the pull request is available at: docu-html |
| :reqtype: Functional | ||
| :security: NO | ||
| :safety: QM | ||
| :safety: ASIL_B |
There was a problem hiding this comment.
Requirements inherit the ASIL level from their parents and not from the ASIL level of their feature architecture, so the argumentation for changing this to "ASIL B" is not correct. Feature architecture can show components of different ASIL level (if the components provide FFI) and still be classified the higest level of these. But as the Utils Library component requirements are also ASIL B, this change is ok.
| :reqtype: Non-Functional | ||
| :security: NO | ||
| :safety: QM | ||
| :safety: ASIL_B |
There was a problem hiding this comment.
Requirements inherit the ASIL level from their parents and not from the ASIL level of their feature architecture, so the argumentation for changing this to "ASIL B" is not correct. Feature architecture can show components of different ASIL level (if the components provide FFI) and still be classified the higest level of these. Unclear here what is correct ASIL level, because feat_req__persistency__variant_management is currently not fulfilled by a persistency component.
| :reqtype: Non-Functional | ||
| :security: NO | ||
| :safety: QM | ||
| :safety: ASIL_B |
There was a problem hiding this comment.
| :safety: ASIL_B | |
| :safety: QM |
Agree on Alexanders argumentation. We might consider the check when it's not covering this variant.
feat_req__baselibs__core_utilities must be ASIL-B because feat_arc_sta__baselibs__static_view_arch and
feat_arc_dyn__baselibs__dynamic_view_arch fulfill it and these are ASIL-B.
Same with feat_req__persistency__variant_management and feat_arc_sta__persistency__static.