Skip to content

process: document safety analysis #902

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

PandaeDo
Copy link
Contributor

No description provided.

Copy link

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ✅ Passed

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
INFO: Invocation ID: 5089b760-a49c-4727-856d-cc8392839909
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (91 packages loaded, 10 targets configured)

Analyzing: target //:license-check (124 packages loaded, 277 targets configured)

Analyzing: target //:license-check (131 packages loaded, 823 targets configured)

Analyzing: target //:license-check (135 packages loaded, 1637 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2465 targets configured)

Analyzing: target //:license-check (145 packages loaded, 2465 targets configured)

Analyzing: target //:license-check (149 packages loaded, 4611 targets configured)

INFO: Analyzed target //:license-check (150 packages loaded, 4737 targets configured).
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 14.714s, Critical Path: 0.33s
INFO: 13 processes: 4 disk cache hit, 9 internal.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt -review -project automotive.score -repo https://github.com/eclipse-score/score -token otyhZ4eaRYK1tKLNNF-Y
[main] INFO Querying Eclipse Foundation for license data for 76 items.
[main] INFO Found 52 items.
[main] INFO Querying ClearlyDefined for license data for 24 items.
[main] INFO Found 24 items.
[main] INFO Vetted license information was found for all content. No further investigation is required.

Copy link

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

Copy link
Contributor

@masc2023 masc2023 left a comment

Choose a reason for hiding this comment

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

Please check also required updates of Software Development Plan

:complies: std_wp__iso26262__analysis_851, std_wp__iso26262__software_752

Bottom-Up Safety Analysis with e.g. FMEA method, verifies the feature architecture (as part of SW Safety Concept)
- Detection and prevention mitigations linked to Software Feature Requirements or Assumptions of Use
Copy link
Contributor

Choose a reason for hiding this comment

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

May link here to the correct WPs per need, AoU are here concrete Feature Assumption of Use

:status: valid
:complies: std_wp__iso26262__software_754

Depends on architecture, FMEA and DFA tooling.
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be updated to make it concrete and as said, update or link to SW Development Plan accordingly


todo: need to add guidance and standard links


.. workflow:: Analyse Feature Architecture
:id: wf__analyse_featarch
:status: draft
Copy link
Contributor

Choose a reason for hiding this comment

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

We defined to have a new role for that as Resp., Safety Engineer (t.b.d. including Role description), rl_contributor, rl_committer shall support, would remove technical lead as supporter, comment is valid for all workflows, where is the Architecture Verification done?


Contributing Roles:

* :need:`Contributor <rl__contributor>`
Copy link
Contributor

Choose a reason for hiding this comment

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

Add it to workflows, otherwise not part here, would remove leads, and add missing Safety Engineer Role with complete description

:maxdepth: 1

guidance/index
safety_analysis_concept
Copy link
Contributor

Choose a reason for hiding this comment

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

may change order, to start with getting started, concept, guidance, etc.

The analysis is finished, if for each identified faults a mechanism/measure exists.
Unless the attribute sufficient is yes, measure and argument attribute can be still empty.

**Examples:**
Copy link
Contributor

Choose a reason for hiding this comment

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

may link here toe the introduction, building model, where this example is available as image


Safety Analysis shall be linked to its adjacent level via the attribute mitigates.

* Feature Safety Analysis <-> feature architecture
Copy link
Contributor

Choose a reason for hiding this comment

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

check traceability model, there are more links

* 0 to 100 percent

.. gd_req:: Safety Analysis attribute: link to Aou
:id: gd_req__saf__attr_aou
Copy link
Contributor

Choose a reason for hiding this comment

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

see above, also requirements


It shall be checked that Safety Analysis (Safety != QM) can only be linked against elements with the same ASIL.

.. needextend:: "process_areas/requirements_engineering" in docname
Copy link
Contributor

Choose a reason for hiding this comment

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

you need to change that to safety_analysis

It shall be checked if all mandatory attributes for each Safety Analysis is provided by the user. For all Safety Analysis following attributes shall be mandatory:

.. needtable:: Overview mandatory Safety Analysis attributes
:filter: "mandatory" in tags and "attribute" in tags and "safety analysis" in tags and type == "gd_req"
Copy link
Contributor

Choose a reason for hiding this comment

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

does not work, other attributes are shown, compare sphinx-docu, see comment for needextend

@masc2023
Copy link
Contributor

Architecture Verification is part of Architecture Design, can be removed from Safety Analysis, see https://github.com/eclipse-score/score/pull/907/files#diff-c89206dc1e9d878523aa319232e3ee40c242ee096987e2cdcee145c61cee71ab

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants