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

#128: set up GPU CI pipelines #129

Merged
merged 79 commits into from
Sep 6, 2023
Merged

#128: set up GPU CI pipelines #129

merged 79 commits into from
Sep 6, 2023

Conversation

cwschilly
Copy link
Collaborator

@cwschilly cwschilly commented Aug 21, 2023 •
edited by tlamonthezie
Loading

Fixes #128

GPU builds will be triggered only for PR's that have either the EpetraMpi T2 or EpetraMpi T3 labels

@cwschilly cwschilly linked an issue Aug 21, 2023 that may be closed by this pull request
Framework: Set up GPU build for CI #128
Closed
@cwschilly cwschilly mentioned this pull request Aug 16, 2023
Open
@JacobDomagala JacobDomagala force-pushed the 128-framework-set-up-gpu-build-for-ci branch from 3bbdc6d to e79183e Compare August 29, 2023 17:48
cwschilly and others added 20 commits September 6, 2023 12:38
@cwschilly
#128: set up GPU CI pipelines
5a5d575
@cwschilly
#128: specify cuda dir
f881b55
@cwschilly
#128: temporarily run GPU on every push; stop MPI builds
255daad
@cwschilly
#128: specify correct dockerfile
85060fc
@cwschilly
#128: provide correct build/source dirs
2a3eea2
@cwschilly
#128: rework MPI in build script
aef2599
@cwschilly
#128: rework MPI in other gpu build script
1703f80
@tlamonthezie @cwschilly
#128: CI build script try another configuration and fix invalid path
2d688b7
@tlamonthezie @cwschilly
#128: fix missing letter in path for a gpu build
4627fe7
@cwschilly
#128: add newlines to end of files
38ecc84
@cwschilly
#128: add spack find -p to find cuda root
8272576
@cwschilly
#128: only run one pipeline; add cuda paths
9c43e86
@cwschilly
#128: add kokkos variables
dcecd73
@cwschilly
#128: add Tpetra_INST_SERIAL:BOOL=ON
7f3b058
@cwschilly
#128: add CUDA root flag
03b5026
@cwschilly
#128: use correct kokkos architecture
f14bc21
@cwschilly
#128: enable cusolver and cusparse
61a75d0
@cwschilly
#128: emulate local build
4d9dff7
@cwschilly
#128: try with different docker image
c9dc527
@cwschilly
#128: update cuda path
9e4a37d
tlamonthezie and others added 21 commits September 6, 2023 12:39
@tlamonthezie @cwschilly
#128: try again changing path dynamically
692ec17
@tlamonthezie @cwschilly
#128: fix another path
42379f6
@tlamonthezie @cwschilly
#128: fix blas path
3e5067a
@tlamonthezie @cwschilly
#128: apply working conffiguration to other build scripts
d132a62
@tlamonthezie @cwschilly
#128: restore triggering workflows on PR
baab02e
@tlamonthezie @cwschilly
#128: disable GPU build job for PR having EpetraMPI T1 label
49b8d4e
@tlamonthezie @cwschilly
#128: enable GPU build only with EpetraMPI T2 and EpetraMPI T3 labels
33b1209
@tlamonthezie @cwschilly
#128: upload test log
9416d09
@tlamonthezie @cwschilly
#128: fix typo
0e87ff8
@tlamonthezie @cwschilly
#128: fix artifacts
4213943
@tlamonthezie @cwschilly
#128: add junit report for tests
135e81e
@tlamonthezie @cwschilly
#128: add junit reporting in CI and set
6a4faf3
@tlamonthezie @cwschilly
#128: fix artifact name
d9e5dd6
@tlamonthezie @cwschilly
#128: fix artifacts missing
6ce83a6
@tlamonthezie @cwschilly
#128 fix extra slach char in path
8111ae4
@tlamonthezie @cwschilly
#128: fix artifacts path
94e68a6
@tlamonthezie @cwschilly
#128: fix path in gitbub action
e60b7e2
@tlamonthezie @cwschilly
#128: try mounting artifacts folder into the host runner
ca9ec66
@tlamonthezie @cwschilly
#128: use same logic for gpu or non-gpu pipelines
0364649
@cwschilly
128: Finalize pipelines (GPU on push, MPI cancellations)
5b44baa
@cwschilly
128: remove label requirements
7ed1947
@cwschilly cwschilly force-pushed the 128-framework-set-up-gpu-build-for-ci branch from 867823e to 7ed1947 Compare September 6, 2023 16:39
@cwschilly cwschilly force-pushed the 128-framework-set-up-gpu-build-for-ci branch from 2cf0b61 to 9a00d32 Compare September 6, 2023 20:12
stmcgovern
stmcgovern approved these changes Sep 6, 2023
View reviewed changes
Copy link
Collaborator

@stmcgovern stmcgovern left a comment

Choose a reason for hiding this comment

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

There will be some failing tests in the GPU pipeline as is.

@cwschilly cwschilly merged commit bc06de5 into NGA-FY23-develop Sep 6, 2023
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stmcgovern stmcgovern stmcgovern approved these changes

Assignees

@tlamonthezie tlamonthezie

@cwschilly cwschilly

Labels
EpetraMPI T1 EpetraMPI T2 EpetraMPI T3 High Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Framework: Set up GPU build for CI
4 participants
@cwschilly @stmcgovern @tlamonthezie @JacobDomagala