Improvement: Safety Management ML 2#437
Conversation
PandaeDo
requested review from
PhilipPartsch,
aschemmel-tech,
masc2023 and
pahmann
as code owners
|
The created documentation from the pull request is available at: docu-html |
masc2023
left a comment
masc2023
left a comment
There was a problem hiding this comment.
Needs discussion, cannot see currently a feature safety plan, only attachment for Platform Plan, called Feature Planning?
| ^^^^^^^^^^^^ | ||
|
|
||
| The SW platform project defines two levels of planning: platform and module. There will be one safety plan on platform level and several safety plans on module level (one for each module). | ||
| The SW platform project defines different levels of planning: platform, feature and module. There will be one safety plan on platform level and |
There was a problem hiding this comment.
Is that a change, currently the feature plan is part of platform plan, no separate template?
E.g. https://eclipse-score.github.io/score/main/features/frameworks/feo/safety_planning/index.html
There was a problem hiding this comment.
Need some time but now I understand it. Changed it back
aschemmel-tech
left a comment
aschemmel-tech
left a comment
There was a problem hiding this comment.
see inline comment
process/process_areas/safety_management/safety_management_concept.rst
Outdated
Show resolved
Hide resolved
|
@attifunel want to put your attention to this. |
| * Create/Maintain Safety Plan | ||
| * Approve Component Classification | ||
| * Approve Safety Package | ||
| * Approve Safety Audit |
There was a problem hiding this comment.
What is the exact meaning here? Audit is assigned to an external entity, why Safety Manager should approve? Perhaps "plan" + "support"?
There was a problem hiding this comment.
I think it makes sense that we will approve the external Audit. With this we can document that we agree on the Audit report. That the Safety Manager and the Process Community will support is defined in the workflow of the Safety Audit. Also relevant is, that we defined that all workflows has an approval. I hope this answered your question. If not I would invite to to "Eclipse S-CORE Safety Topics, Safety Management and Execution (Safety Manager Roles required)" every Tuesday, where we also can discuss your question with the complete community.
There was a problem hiding this comment.
You need to see that in the context of the define workflows,
https://eclipse-score.github.io/process_description/main/process_areas/safety_management/safety_management_workflow.html#wf__p_fs_audit
improvement in concept description could be to include the workflows, where the Safety Manager has stakes in
5 participants
Solve open review findings #398