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

ci,ascent: enable ascent builds #3610

Merged
merged 2 commits into from
May 12, 2023

Conversation

vicentebolea
Copy link
Collaborator

@vicentebolea vicentebolea commented May 10, 2023

fixes: #3601
fixes: #3598
fixes: #3551
fixes: #3599
fixes: #3546
fixes: #3523
fixes: #3603
related: #3602

This PR adds the following power9 builds:

  • Ascent IBM XL (SST evpath)
  • Ascent NVHPC (SST evpath)
  • Ascent CUDA/GCC (SST libfabric)
  • Ascent Kokkos_CUDA/GCC (SST libfabric)

The builds are accessible from the Github PR page and it is shown as one more check.

Screenshot from 2023-05-11 13-29-14

Builds are here: https://code.ornl.gov/ecpcitest/adios2/-/pipelines

@vicentebolea vicentebolea marked this pull request as ready for review May 10, 2023 22:13
@vicentebolea vicentebolea force-pushed the enable-olcf-build branch 9 times, most recently from f8f954d to cad3bc6 Compare May 11, 2023 18:02
@vicentebolea vicentebolea self-assigned this May 11, 2023
Copy link
Collaborator

@scottwittenburg scottwittenburg left a comment

Choose a reason for hiding this comment

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

I'm just reviewing the changes related to CI here, and leaving the changes in source/ to someone else. I had one question about printing out the all environment variables, but otherwise I didn't notice any gotchas. It's great you got this working @vicentebolea!

scripts/ci/gitlab-ci/run.sh Outdated Show resolved Hide resolved
@vicentebolea
Copy link
Collaborator Author

@eisenhauer can you review the source code changes?

Copy link
Member

@eisenhauer eisenhauer left a comment

Choose a reason for hiding this comment

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

I think the C++ changes are OK...

@vicentebolea vicentebolea merged commit c4af0cf into ornladios:master May 12, 2023
@vicentebolea vicentebolea deleted the enable-olcf-build branch May 12, 2023 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants