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

Put shared part together for Databricks test scripts [databricks] #8828

Merged
merged 2 commits into from
Aug 2, 2023

Conversation

NvTimLiu
Copy link
Collaborator

To fix issue: #8779

Put shared together for Databricks test scripts test.sh and run_it.sh

to make update of the test scripts easy, need not to modify both

test.sh and run_it.sh if we change the shared script.

@NvTimLiu NvTimLiu changed the title Put shared part together for Databricks test scripts Put shared part together for Databricks test scripts [databricks] Jul 27, 2023
@NvTimLiu
Copy link
Collaborator Author

build

@NvTimLiu NvTimLiu marked this pull request as ready for review July 27, 2023 21:07
@NvTimLiu NvTimLiu requested a review from pxLi July 28, 2023 00:50
zhanga5
zhanga5 previously approved these changes Jul 28, 2023
To fix issue: NVIDIA#8779

Put shared part together for Databricks scripts test.sh and run_it.sh

to make update of the scripts easy, need not to modify both

test.sh and run_it.sh if we change the shared part.

Signed-off-by: Tim Liu <[email protected]>
@NvTimLiu
Copy link
Collaborator Author

build

@pxLi
Copy link
Collaborator

pxLi commented Jul 31, 2023

Please help confirm if csputil require any update about the new script, thanks!

@pxLi pxLi merged commit 36b63c5 into NVIDIA:branch-23.08 Aug 2, 2023
@sameerz sameerz added the build Related to CI / CD or cleanly building label Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants