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.

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

Update UCC test_template container #37

Merged
merged 1 commit into from
Jun 6, 2024
Merged

Update UCC test_template container #37

merged 1 commit into from
Jun 6, 2024

Conversation

lappazos
Copy link
Contributor

@lappazos lappazos commented May 28, 2024 •
edited
Loading

Summary

Update UCC test_template container

Test Plan

Passed full installation + test on real system.

Commands

python ./cloudaix.py --mode install --system_config_path conf/v0.6/general/system/<>.toml
python ./cloudaix.py --mode run --system_config_path conf/v0.6/general/system/<>.toml--test_scenario_path conf/v0.6/general/test_scenario/ucc_test_shortend.toml

@lappazos lappazos marked this pull request as draft May 28, 2024 09:56
@TaekyungHeo TaekyungHeo added the enhancement New feature or request label Jun 5, 2024
@lappazos lappazos force-pushed the update_ucc_container branch from 814b7ba to 5665b4c Compare June 5, 2024 23:33
@lappazos lappazos marked this pull request as ready for review June 6, 2024 00:14
@lappazos
Copy link
Contributor Author

lappazos commented Jun 6, 2024

@srinivas212 @TaekyungHeo please review

TaekyungHeo
TaekyungHeo approved these changes Jun 6, 2024
View reviewed changes
Copy link
Member

@TaekyungHeo TaekyungHeo left a comment

Choose a reason for hiding this comment

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

LGTM. Let @srinivas212 merge.

@srinivas212 srinivas212 merged commit 9720f96 into NVIDIA:main Jun 6, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TaekyungHeo TaekyungHeo TaekyungHeo approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lappazos @TaekyungHeo @srinivas212