8000 process: add isosae21434 by masc2023 · Pull Request #576 · eclipse-score/score · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

process: add isosae21434 #576

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 19, 2025
Merged

process: add isosae21434 #576

merged 1 commit into from
Mar 19, 2025

Conversation

masc2023
Copy link
Contributor
@masc2023 masc2023 commented Mar 6, 2025

Add isosae21434 mapping list

@github-actions GitHub Actions
Copy link
github-actions bot commented Mar 6, 2025 •
edited
Loading

License Check Results

🚀 The license check preparation job ran successfully.

Status: ⚠️ Needs Review

Click to expand output 
2025/03/19 08:50:22 Downloading https://releases.bazel.build/7.4.0/release/bazel-7.4.0-linux-x86_64...
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: docs
Loading: 0 packages loaded
    currently loading: docs
Loading: 0 packages loaded
    currently loading: docs
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)
Analyzing: target //docs:license.check.python (1 packages loaded, 0 targets configured)

Analyzing: target //docs:license.check.python (90 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (92 packages loaded, 10 targets configured)

Analyzing: target //docs:license.check.python (106 packages loaded, 189 targets configured)

Analyzing: target //docs:license.check.python (129 packages loaded, 824 targets configured)

Analyzing: target //docs:license.check.python (133 packages loaded, 1601 targets configured)

Analyzing: target //docs:license.check.python (136 packages loaded, 1624 targets configured)

Analyzing: target //docs:license.check.python (143 packages loaded, 2560 targets configured)

Analyzing: target //docs:license.check.python (144 packages loaded, 4561 targets configured)

Analyzing: target //docs:license.check.python (148 packages loaded, 4611 targets configured)

INFO: Analyzed target //docs:license.check.python (149 packages loaded, 4736 targets configured).
[9 / 13] JavaToolchainCompileClasses external/rules_java~/toolchains/platformclasspath_classes; 0s processwrapper-sandbox ... (2 actions running)
[11 / 13] JavaToolchainCompileBootClasspath external/rules_java~/toolchains/platformclasspath.jar; 0s processwrapper-sandbox
[12 / 13] Building docs/license.check.python.jar (); 0s multiplex-worker
INFO: Found 1 target...
Target //docs:license.check.python up-to-date:
  bazel-bin/docs/license.check.python
  bazel-bin/docs/license.check.python.jar
INFO: Elapsed time: 22.858s, Critical Path: 2.34s
INFO: 13 processes: 9 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 13 total actions
INFO: Running command line: bazel-bin/docs/license.check.python docs/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 69 items.
[main] INFO Found 45 items.
[main] INFO Querying ClearlyDefined for license data for 28 items.
[main] INFO Found 28 items.
[main] INFO License information could not be automatically verified for the following content:
[main] INFO 
[main] INFO pypi/pypi/-/debugpy/1.8.12
[main] INFO pypi/pypi/-/docutils/0.21.2
[main] INFO pypi/pypi/-/matplotlib/3.10.0
[main] INFO pypi/pypi/-/pillow/11.1.0
[main] INFO 
[main] INFO This content is either not correctly mapped by the system, or requires review.
[main] INFO A review is required for pypi/pypi/-/docutils/0.21.2.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19880 .
[main] INFO A review is required for pypi/pypi/-/pillow/11.1.0.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19890 .
[main] INFO A review is required for pypi/pypi/-/debugpy/1.8.12.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19903 .
[main] INFO A review is required for pypi/pypi/-/matplotlib/3.10.0.
[main] INFO A review request already exists https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/19892 .

@github-actions GitHub Actions
Copy link
github-actions bot commented Mar 6, 2025

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

@masc2023 masc2023 self-assigned this Mar 12, 2025
@masc2023 masc2023 moved this to In Progress in Process Community Mar 12, 2025
aschemmel-tech
aschemmel-tech reviewed Mar 17, 2025
View reviewed changes
Copy link
Contributor
@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

see inline comments

docs/process/standards/isosae21434/isosae21434.rst
"Clause 14: End of cybersecurity support and decommissioning" - as this is in responsibility of system integrator

"Annex A-H:" - as it contains no requirements and work products

Copy link
Contributor

Choose a reason for hiding this comment

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

where will the tailoring within the relevant clauses be documented? If not done in this PR please create respective follow-up ticket

Copy link
Contributor Author

Choose a reason for hiding this comment

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

see #696

docs/process/standards/isosae21434/isosae21434.rst
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

* Requirements
.. std_req:: org_management_1
Copy link
Contributor

Choose a reason for hiding this comment

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

Recent communication from VDA suggests that use of clause/sub-clauses numbers and identification of work products in Open Source is possible without asking for approval.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

see #696

@masc2023 masc2023 force-pushed the masc2023_add_isosae21434 branch from 65aed2f to 254c34d Compare March 17, 2025 08:05
@masc2023 masc2023 requested a review from aschemmel-tech March 17, 2025 08:06
pahmann
pahmann reviewed Mar 17, 2025
View reviewed changes
docs/process/standards/isosae21434/isosae21434.rst

"Clause 7: Distributed cybersecurity activities" - as the project is organized as open source project

"Clause 9: Concept" - as this is in responsibility of system integrator
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is responsibility of the system integrator, would it make sense to put this somewhere as a requirement for the system integrator.
We should consider to place our "expectations" to external aspects (Operating System, Integrator, applications, hardware abstraction) somewhere.

Copy link
Contributor Author
@masc2023 masc2023 Mar 17, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Compare ISO 26262 blueprint and link to PR for the security plan, see #696

pahmann
pahmann reviewed Mar 17, 2025
View reviewed changes
docs/process/standards/isosae21434/isosae21434.rst Outdated


.. note::
Titles of the ISO/SAE 21434 standard clauses are from official `ISO website <https://www.iso.org/search.html>`_ (search for "21434").
Copy link
Contributor
@pahmann pahmann Mar 17, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Do we have to state that this list is based on version :2021 (or 1st edition) to make sure where it is created against?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Followed the style of ISO 26262, also not mentioned 2018 there yet

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added version now

@masc2023 masc2023 force-pushed the masc2023_add_isosae21434 branch 3 times, most recently from 47b4440 to bf0515a Compare March 19, 2025 08:44
@masc2023
process: add isosae21434
77c3ca5 
Add isosae21434 mapping list
@masc2023 masc2023 force-pushed the masc2023_add_isosae21434 branch from bf0515a to 77c3ca5 Compare March 19, 2025 08:50
aschemmel-tech
aschemmel-tech approved these changes Mar 19, 2025
View reviewed changes
Copy link
Contributor
@aschemmel-tech aschemmel-tech left a comment

Choose a reason for hiding this comment

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

Ok for now

@masc2023 masc2023 merged commit 474bb0d into main Mar 19, 2025
10 checks passed
@masc2023 masc2023 deleted the masc2023_add_isosae21434 branch March 19, 2025 09:19
@github-project-automation github-project-automation bot moved this from In Progress to Done in Process Community Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pahmann pahmann pahmann left review comments

@aschemmel-tech aschemmel-tech aschemmel-tech approved these changes

@hoe-jo hoe-jo Awaiting requested review from hoe-jo

Assignees

@masc2023 masc2023

Labels
community:process
Projects
Process Community
Status: Done
Milestone
2025-05-31 Process & Tools Enable Dev...
Development

Successfully merging this pull request may close these issues.
3 participants
@masc2023 @pahmann @aschemmel-tech
0