Closed mkavulich closed 4 months ago
Note: the new job to auto-push the latest container to Dockerhub makes use of repository secrets: I have provided my dockerhub credentials as DOCKERHUB_USERNAME and DOCKERHUB_TOKEN since I can write to the dtcenter organization on Dockerhub. If in the future I am not around, or if I lose access to that Dockerhub org, those secrets will have to be updated.
@grantfirl Did you want a chance to review this or can I go ahead and merge it?
@mkavulich Can you update to the latest main for good measure before I press the merge button?
@grantfirl Rebased my branch on the latest main
This PR has several updates that improve SCM documentation, overhaul the docker container, updates new and existing tests, and adds modulefiles for building on MacOS and generic Linux.
Documentation
Documentation for this branch is built here: https://ccpp-scm-mkavulich.readthedocs.io/en/feature-macos_and_docker_build/
This update adds some information about testing to the Repository chapter (though more details should probably be included), and completely overhauls the Quick Start chapter.
Among the changes to the Quick Start chapter are:
Docker
main
. This action will build the container and push it to Dockerhub under the tag "dtcenter/ccpp-scm:latest". This will ensure there is always an up-to-date, working container for the top of the main branch.Testing
ci_build_scm_ubuntu_22.04_nvidia.yml
is modified to only run on pull_requests and when manually requestedOther
macos_clang
andlinux_gnu
for use with spack-stack installations.This PR will resolve #409 and partially resolve #455 (will still need a release container when the time comes)