Skip to content

fix: update safety attribute#2933

Closed
a-zw wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:fix-safety-attribute
Closed

fix: update safety attribute#2933
a-zw wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:fix-safety-attribute

Conversation

@a-zw

@a-zw a-zw commented May 27, 2026

Copy link
Copy Markdown
Contributor

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.

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.
@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

:reqtype: Functional
:security: NO
:safety: QM
:safety: ASIL_B

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
:safety: ASIL_B
:safety: QM

Agree on Alexanders argumentation. We might consider the check when it's not covering this variant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants