-
Notifications
You must be signed in to change notification settings - Fork 62
Orchestration architecture update #1914
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
base: main
Are you sure you want to change the base?
Orchestration architecture update #1914
Conversation
Signed-off-by: Pawel Rutka <pawel.rutka.ext@qorix.ai>
Updated files to fix review comments, CI failure and appearance.
|
The created documentation from the pull request is available at: docu-html |
:id: doc__orchestration | ||
:status: valid | ||
:safety: ASIL_B | ||
:realizes: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add :security: YES
# SPDX-License-Identifier: Apache-2.0 | ||
# ******************************************************************************* | ||
Executor Module |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might move the complete folder from docs/features/orchestration/modules to docs/modules/orchestration
# SPDX-License-Identifier: Apache-2.0 | ||
# ******************************************************************************* | ||
Orchestration Component Architecture |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's not shown in html. Isn't it?
Orchestration Component Architecture | ||
************************************ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please always use the templates https://eclipse-score.github.io/process_description/main/folder_templates/modules/module_name/component_name/docs/architecture/index.html. If it's not used it's also not listed in the documentation plan because it's not a document. Please also check it for all other documents.
@@ -0,0 +1,13 @@ | |||
.. _comp_doc_orchestrator: | |||
|
|||
orchestrator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
orchestrator | |
Orchestrator |
.. document:: Orchestration Requirements | ||
:id: doc__orch_requirements | ||
:status: valid | ||
:safety: ASIL_B |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:safety: ASIL_B | |
:safety: ASIL_B | |
:security: YES |
:status: valid | ||
:safety: ASIL_B | ||
:realizes: PROCESS_wp__requirements_feat | ||
:tags: feature_request |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:tags: feature_request | |
:tags: feature_request; orchestration |
@@ -0,0 +1,13 @@ | |||
.. _comp_doc_executor: | |||
|
|||
executor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
executor | |
Executor |
# | ||
# SPDX-License-Identifier: Apache-2.0 | ||
# ******************************************************************************* | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we shall add a disclosure that we are using AI https://eclipse-score.github.io/process_description/main/workproducts/index.html in all document headers
Updated files to fix review comments, CI failure and appearance.