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

Optimize Databricks Jenkins scripts [skip ci] #11817

Merged
merged 3 commits into from
Dec 9, 2024

Conversation

NvTimLiu
Copy link
Collaborator

@NvTimLiu NvTimLiu commented Dec 4, 2024

Remove duplicate try/catch/container script blocks

Move default Databricks parameters into the common Groovy library

Remove duplicate try/catch/container script blocks

Move default Databricks parameters into the common Groovy library

Signed-off-by: timl <[email protected]>
@NvTimLiu NvTimLiu added the build Related to CI / CD or cleanly building label Dec 4, 2024
@NvTimLiu NvTimLiu self-assigned this Dec 4, 2024
@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Dec 4, 2024

build

pxLi
pxLi previously approved these changes Dec 4, 2024
Copy link
Collaborator

@pxLi pxLi left a comment

Choose a reason for hiding this comment

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

LGTM if CI pass, thanks

@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Dec 5, 2024

build

@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Dec 6, 2024

build

pxLi
pxLi previously approved these changes Dec 9, 2024
@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Dec 9, 2024

There's no actual change from commit2 to commit3 for the pre-merge CI,
As we just PASS commit2's pre-merge CI, let's skip CI for the commit3 change.

@NvTimLiu NvTimLiu changed the title Optimize Databricks Jenkins scripts [databricks] Optimize Databricks Jenkins scripts [skip ci] Dec 9, 2024
@NvTimLiu
Copy link
Collaborator Author

NvTimLiu commented Dec 9, 2024

build

@pxLi pxLi merged commit d1466b7 into NVIDIA:branch-25.02 Dec 9, 2024
50 checks passed
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.

2 participants