-
Notifications
You must be signed in to change notification settings - Fork 65
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
Orchestration architecture update #1914
Conversation
|
|
|
The created documentation from the pull request is available at: docu-html |
...features/orchestration/modules/orchestration/orchestrator/docs/architecture/orchestrator.rst
Outdated
Show resolved
Hide resolved
docs/modules/orchestrator/orchestrator/docs/architecture/orchestrator.rst
Show resolved
Hide resolved
docs/features/orchestration/modules/orchestration/orchestrator/docs/index.rst
Outdated
Show resolved
Hide resolved
docs/features/orchestration/modules/executor/executor/docs/index.rst
Outdated
Show resolved
Hide resolved
21ddb8e to
04bbe00
Compare
docs/modules/orchestrator/orchestrator/docs/architecture/orchestrator.rst
Show resolved
Hide resolved
04bbe00 to
28977b9
Compare
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.
Fine for the moment
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 change the names as described in the names conventions so that they are listed in e.g. the documentation plan
28977b9 to
1194854
Compare
b78f9c0 to
b7d91f7
Compare
| :included_by: logic_arc_int__orchestration__design | ||
|
|
||
|
|
||
| .. logic_arc_int_op:: bind_events_as_global |
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 align API with current state for deployment design and OrchProgManager
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.
done
| # SPDX-License-Identifier: Apache-2.0 | ||
| # ******************************************************************************* | ||
|
|
||
| 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.
we are missing dependecy to mw_com or ?
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.
There is no ID defined with mw_com and related text in communication feature. So, it cannot be added.
|
@PandaeDo @vinodreddy-g all req should be tagged asilb or ? |
Yes for all the functional req for orchestration. |
Signed-off-by: Pawel Rutka <[email protected]>
Updated files to fix review comments, CI failure and appearance.
Updated files to fix review comments, CI failure and appearance.
b7d91f7 to
2bdcabb
Compare
|
Rebased and fixed review comments. |
add depndency on mw::com frontend
|
The safety levels and meanings will be adjusted in follow up PRs |
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.
LGTM as of now
fix modules to components in orchestrator .Two components executor and orchestrator.
Updated files to fix review comments, CI failure and appearance.