Skip to content

Commit a81ed7e

Browse files
RolandJentschETASRolandJentschETAS
committed
add stakeholder requirements template
1 parent 77db209 commit a81ed7e

File tree

3 files changed

+180
-1
lines changed

3 files changed

+180
-1
lines changed

‎process/folder_templates/features/feature_name/requirements/chklst_req_inspection.rst‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ Requirement Inspection Checklist
124124
-
125125
-
126126
* - REQ_07_02
127-
- Is the attribute *security* set correctly?
127+
- Is the *security* attribute set correctly?
128128
- For feature requirements this checklist item is supported by automated check: "Every requirement which satisfies a stakeholder requirement with security attribute set to YES inherits this". But the feature requirements/architecture may additionally also be subject to a :ref:`Software Security Analysis <security_analysis>`.
129129
-
130130
-

‎process/folder_templates/platform/index.rst‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,4 @@ Platform
2121
:hidden:
2222

2323
safety_analysis/platform_dfa.rst
24+
requirements/stakeholder/chklst_req_inspection.rst

‎process/folder_templates/platform/requirements/stakeholder/chklst_req_inspection.rst‎

Lines changed: 178 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,178 @@
1+
..
2+
# *******************************************************************************
3+
# Copyright (c) 2025 Contributors to the Eclipse Foundation
4+
#
5+
# See the NOTICE file(s) distributed with this work for additional
6+
# information regarding copyright ownership.
7+
#
8+
# This program and the accompanying materials are made available under the
9+
# terms of the Apache License Version 2.0 which is available at
10+
# https://www.apache.org/licenses/LICENSE-2.0
11+
#
12+
# SPDX-License-Identifier: Apache-2.0
13+
# *******************************************************************************
14+
15+
16+
.. document:: [Your Stakeholder Name] Requirements Inspection Checklist
17+
:id: doc__stakeholder_name_req_inspection
18+
:status: draft
19+
:safety: ASIL_B
20+
:security: YES
21+
:realizes: wp__requirements_inspect
22+
:tags: template
23+
24+
.. attention::
25+
The above directive must be updated according to your Stakeholder.
26+
27+
- Modify ``Your Stakeholder Name`` to be your Stakeholder Name
28+
- Modify ``id`` to be your Stakeholder Name in lower snake case preceded by ``doc__`` and followed by ``_req_inspection``
29+
- Adjust ``status`` to be ``valid``
30+
- Adjust ``safety``, ``security`` and ``tags`` according to your needs
31+
32+
Stakeholder Requirement Inspection Checklist
33+
============================================
34+
35+
**Purpose**
36+
37+
The purpose of this requirement inspection checklist is to collect the topics to be checked during requirements inspection.
38+
39+
**Conduct**
40+
41+
As described in the concept :need:`doc_concept__wp_inspections` the following "inspection roles" are expected to be filled:
42+
43+
- author: these are the persons who did the last commits on the requirements in scope (can be derived from version mgt tool)
44+
- reviewer: these are all persons committing into this inspection document or giving a pull request verdict on it (can be derived from version mgt tool)
45+
- moderator: only needed for conflict resolution between author and reviewers, is the safety manager, security manager or quality manager called in as a reviewer (can be derived from version mgt tool)
46+
- test expert: <one of the reviewers explicitly named here, to cover REQ_08_01 as described>
47+
48+
**Checklist**
49+
50+
.. list-table:: Stakeholder Requirement Inspection Checklist
51+
:header-rows: 1
52+
:widths: 10,30,50,6,6,8
53+
54+
* - Review ID
55+
- Acceptance Criteria
56+
- Guidance
57+
- Passed
58+
- Remarks
59+
- Issue link
60+
* - REQ_01_01
61+
- Is the requirement formulation template used?
62+
- see :need:`gd_temp__req_formulation`, this includes the use of "shall".
63+
-
64+
-
65+
-
66+
* - REQ_02_01
67+
- Is the requirement description *comprehensible* ?
68+
- If you think the requirement is hard to understand, comment here.
69+
-
70+
-
71+
-
72+
* - REQ_02_02
73+
- Is the requirement description *unambiguous* ?
74+
- Especially search for "weak words" like "about", "etc.", "relevant" and others (see the internet documentation on this). This check shall be supported by tooling.
75+
-
76+
-
77+
-
78+
* - REQ_02_03
79+
- Is the requirement description *atomic* ?
80+
- A good way to think about this is to consider if the requirement may be tested by one (positive) test case or needs more of these. The requirement formulation template should also avoid being non-atomic already. Note that there are cases where also non-atomic requirements are the better ones, for example if those are better understandable.
81+
-
82+
-
83+
-
84+
* - REQ_02_04
85+
- Is the requirement description *feasible* ?
86+
- If at the time of the inspection the requirement has already some implementation, the answer is yes. This can be checked via traces, but also :need:`gd_req__req_attr_impl` shows this. In case the requirement has no implementation at the time of inspection (i.e. not implemented at least as "proof-of-concept"), a development expert should be invited to the Pull-Request review to explicitly check this item.
87+
-
88+
-
89+
-
90+
* - REQ_02_05
91+
- Is the requirement description *independent from implementation* ?
92+
- This checkpoint should improve requirements definition in the sense that the "what" is described and not the "how" - the latter should be described in architecture/design derived from the requirement. But there can also be a good reason for this, for example we would require using a file format like JSON and even specify the formatting standard already on stakeholder requirement level because we want to be compatible. A finding in this checkpoint does not mean there is a safety problem in the requirement.
93+
-
94+
-
95+
-
96+
* - REQ_03_01
97+
- Is the *rationale* correct?
98+
- Rationales explain why the top level requirements were created. Do those cover the requirement?
99+
-
100+
-
101+
-
102+
* - REQ_03_02
103+
- Is the *linkage to the parent requirement* correct?
104+
- Linkage to correct levels and ASIL attributes is checked automatically, but it needs checking if the child requirement implements (at least) a part of the parent requirement.
105+
-
106+
-
107+
-
108+
* - REQ_04_01
109+
- Is the requirement *internally and externally consistent*?
110+
- Does the requirement contradict other requirements within the same or higher levels?
111+
-
112+
-
113+
-
114+
* - REQ_05_01
115+
- Do the software requirements consider *timing constraints*?
116+
- This checkpoint encourages to think about timing constraints even if those are not explicitly mentioned in the parent requirement. If the reviewer of a requirement already knows or suspects that the code execution will be consuming a lot of time, one should think of the expectation of a "user".
117+
-
118+
-
119+
-
120+
* - REQ_06_01
121+
- Does the requirement consider *external interfaces*?
122+
- The SW platform's external interfaces (to the user and external systems) are defined, so the Feature and Component Requirements should determine the input data use and setting of output data for these interfaces. Are all output values defined?
123+
-
124+
-
125+
-
126+
* - REQ_07_01
127+
- Is the *safety* attribute set correctly?
128+
- For the top level requirements (and also all AoU) this needs to be checked manually for correctness.
129+
-
130+
-
131+
-
132+
* - REQ_07_02
133+
- Is the *security* attribute set correctly?
134+
- For the top level requirements (and also all AoU) this needs to be checked manually for correctness.
135+
-
136+
-
137+
-
138+
* - REQ_08_01
139+
- Is the requirement *verifiable*?
140+
- If at the time of the inspection already tests are created for the requirement, the answer is yes. This can be checked via traces, but also :need:`gd_req__req_attr_test_covered` shows this. In case the requirement is not sufficiently traced to test cases already, a test expert is invited to the inspection to give his opinion whether the requirement is formulated in a way that supports test development and the available test infrastructure is sufficient to perform the test.
141+
-
142+
-
143+
-
144+
* - REQ_09_01
145+
- Do those requirements cover assumed safety mechanisms needed by the hardware and system?
146+
- Note that stakeholder requirements covering safety mechanisms come from rationales.
147+
-
148+
-
149+
-
150+
151+
Note: If a Review ID is not applicable for your requirement, then state ""n/a" in status and comment accordingly in remarks. For example "no stakeholder requirement (no rationale needed)"
152+
153+
The following requirements in "valid" state and with "inspected" tag set are in the scope of this inspection:
154+
155+
.. needtable::
156+
:filter: "stakeholder_name" in docname and "requirements" in docname and docname is not None and status == "valid"
157+
:style: table
158+
:types: stkh_req
159+
:tags: stakeholder_name
160+
:columns: id;status;tags
161+
:colwidths: 25,25,25
162+
:sort: title
163+
164+
And also the following AoUs in "valid" state and with "inspected" tag set (for these please answer the questions above as if the AoUs are requirements, except questions REQ_03_01 and REQ_03_02):
165+
166+
.. needtable::
167+
:filter: "stakeholder_name" in docname and "requirements" in docname and docname is not None and status == "valid"
168+
:style: table
169+
:types: aou_req
170+
:tags: stakeholder_name
171+
:columns: id;status;tags
172+
:colwidths: 25,25,25
173+
:sort: title
174+
175+
.. attention::
176+
The above tables filtering must be updated according to your Stakeholder.
177+
178+
- Modify ``stakeholder_name`` to be your Stakeholder Name in lower snake case

0 commit comments

Comments
 (0)