Skip to content

Conversation

@aschemmel-tech
Copy link
Contributor

Resolves: #1740

@github-actions
Copy link

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@aschemmel-tech
Copy link
Contributor Author

aschemmel-tech commented Oct 14, 2025

Need to resolve some issues before it's ready to review.

Resolves: #1740

Signed-off-by: aschemmel-tech <aschemmel_job@arcor.de>
@github-actions
Copy link

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

@aschemmel-tech aschemmel-tech marked this pull request as ready for review October 14, 2025 09:56
Assumptions on the OS integration - Community Level
---------------------------------------------------

This is the lowest level of integraton, the higher levels will build on this.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo, integration

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

:id: aou_req__platform__os_integration_assistance
:reqtype: Non-Functional
:security: YES
:safety: ASIL_B
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this requirements be ASIL_B?, If used in combination with the highest level yes, but otherwise I would say no

Copy link
Contributor Author

Choose a reason for hiding this comment

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

change these to QM

:id: aou_req__platform__os_bazel_tooling
:reqtype: Non-Functional
:security: YES
:safety: ASIL_B
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't both community and functional platforms be categorized as "QM" for the safety level?

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd say the same. Everything up to and including the second tier/level cannot be ASIL but must be QM.

Copy link
Contributor

Choose a reason for hiding this comment

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

Or how does this :safety: need to be read? If the platform shall operate as ASIL-B, are all requirements of :safety: QM and the ones of :safety: ASIL_B needed? Or only the ones of :safety: ASIL_B? I'd have expected the "and"-semantics.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, change community/functional platform reqs to QM, will add additional clarifying AoU for ASIL_B.

Copy link
Contributor

@opajonk opajonk left a comment

Choose a reason for hiding this comment

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

First thoughts from my side, and a few typos. Thanks a lot for creating this!


The OS supplier shall perform safety anomaly reporting.

Note: This could be fulfilled by listimg per release version all known and user reported bugs which affect the safe OS functions.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Note: This could be fulfilled by listimg per release version all known and user reported bugs which affect the safe OS functions.
Note: This could be fulfilled by listing per release version all known and user reported bugs which affect the safe OS functions.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

:id: aou_req__platform__os_bazel_tooling
:reqtype: Non-Functional
:security: YES
:safety: ASIL_B
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd say the same. Everything up to and including the second tier/level cannot be ASIL but must be QM.

:id: aou_req__platform__os_bazel_tooling
:reqtype: Non-Functional
:security: YES
:safety: ASIL_B
Copy link
Contributor

Choose a reason for hiding this comment

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

Or how does this :safety: need to be read? If the platform shall operate as ASIL-B, are all requirements of :safety: QM and the ones of :safety: ASIL_B needed? Or only the ones of :safety: ASIL_B? I'd have expected the "and"-semantics.

:safety: ASIL_B
:status: valid

The OS supplier shall provide tools for bazel to be able to build, run and test the S-CORE SW platform on the supplier OS.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The OS supplier shall provide tools for bazel to be able to build, run and test the S-CORE SW platform on the supplier OS.
The OS supplier shall provide tools for Bazel to be able to build, run and test the S-CORE SW platform on the supplier OS.

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with the "build" part, since software builds of S-CORE are done using Bazel - which is fine. I don't think test and run can be prescribed by S-CORE. OS's often use higher-level build/integration systems like BitBake etc., and running things (at least integration tests "upwards") often needs to be orchestrated in special ways. I don't think it is useful or necessary to require Bazel as frontent for these activities. I can be even impossible: if the building of e.g. the OS image requires certain tools to be used (qualification argumentation), this could create a serious conflict.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think this could be a joint effort between the OS provider/supplier and S-CORE, where tests provided by the platform can be integrated into Bazel, if that makes sense.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discussed and agreed a formulation change in the process community meeting Oct-21

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-patch-1 branch 2 times, most recently from f345e37 to 22a8768 Compare October 21, 2025 08:40
Copy link
Contributor

@PandaeDo PandaeDo left a comment

Choose a reason for hiding this comment

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

Fine for me

@aschemmel-tech aschemmel-tech merged commit 668bae4 into main Oct 22, 2025
8 checks passed
@aschemmel-tech aschemmel-tech deleted the aschemmel-tech-patch-1 branch October 22, 2025 06:37
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.

Change Request: Tiered SW Platform (Operating System) Integration Process for Eclipse S-CORE

7 participants