Skip to content
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

Add artifact name of a node in runtime system model [Merge 2nd] #48

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ipa-rwu
Copy link
Collaborator

@ipa-rwu ipa-rwu commented Feb 15, 2024

For runtime system , it also has artifact and pkg name via the same way as for component

@ipa-rwu ipa-rwu requested a review from ipa-nhg February 15, 2024 07:16
@ipa-rwu ipa-rwu force-pushed the rwu/fix/runtime_system_artifact_name branch 3 times, most recently from d92d6e5 to 569a99a Compare February 27, 2024 19:37
@ipa-rwu ipa-rwu changed the base branch from main to rwu/feature/write_param_value_in_system February 27, 2024 19:39
@ipa-rwu ipa-rwu force-pushed the rwu/fix/runtime_system_artifact_name branch from 569a99a to 4b0df96 Compare February 27, 2024 19:40
@ipa-rwu ipa-rwu changed the title Add artifact name of a node in runtime system model Add artifact name of a node in runtime system model [Merg 2nd] Feb 27, 2024
@ipa-rwu ipa-rwu changed the title Add artifact name of a node in runtime system model [Merg 2nd] Add artifact name of a node in runtime system model [Merge 2nd] Feb 27, 2024
@ipa-rwu ipa-rwu force-pushed the rwu/fix/runtime_system_artifact_name branch from 4b0df96 to 8ecb21d Compare February 28, 2024 08:10
@ipa-rwu ipa-rwu changed the base branch from rwu/feature/write_param_value_in_system to main February 28, 2024 08:11
@ipa-rwu ipa-rwu force-pushed the rwu/fix/runtime_system_artifact_name branch from 8ecb21d to b55ba4a Compare February 28, 2024 08:14
Copy link
Member

@ipa-nhg ipa-nhg left a comment

Choose a reason for hiding this comment

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

Changes tested locally for a system with 2 nodes, it works! :)

@ipa-rwu Thanks!

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.

2 participants