-
Notifications
You must be signed in to change notification settings - Fork 451
Adding the basic setup of CI jobs running at NERSC nightly. #7799
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
Conversation
I made changes to the CI setup from NERSC to be more in-line with other institutions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple spots where a minor change could be made for consistency
75140b1
to
e918fa9
Compare
Is this file actually used? Will modifications to this file have impact on the actual testing reported to CDash? |
Not yet. A copy of this is currently being used to run the nightly tests. When the PR gets merged, I will mirror Kokkos repo to our internal gitlab and point the CI to run this file. |
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
Signed-off-by: Rahulkumar Gayatri <rgayatri@lbl.gov>
cdbf455
to
4dd370b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add Kokkos_ENABLE_COMPILER_WARNINGS=ON
for all builds.
I added the option to |
This is a draft PR, just to show the NERSC CI setup.
Eventually I would want to move the the type of setup that is similar to other organizations.
The important point of the PR is the configuration options used for the different builds.