Add spack-based workflow to add all supported Geant4 versions to CI #1149
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sethrj
added
documentation
sethrj
force-pushed
the
spack-workflow-actions
branch
2 times, most recently
from
e68a0af to
d03cd0c
Compare
sethrj
force-pushed
the
spack-workflow-actions
branch
from
089a864 to
fe5fbe6
Compare
sethrj
requested review from
amandalund,
drbenmorgan and
pcanal
and removed request for
amandalund
sethrj
force-pushed
the
spack-workflow-actions
branch
from
fe5fbe6 to
84b93f2
Compare
sethrj
force-pushed
the
spack-workflow-actions
branch
from
f273cfe to
0a0d134
Compare
|
@drbenmorgan This is finally working 😅 and by working I'm including "demonstrating the existing failures of everything but Geant4@11.0". Immediately after this is merged I'll update #1152 to show that the builds are fixed. |
drbenmorgan
approved these changes
Mar 18, 2024
|
@drbenmorgan Correct, it shouldn't. I'm explicitly using the hash of the spack version and the installer fixed, and I'm even caching the concretization, so nothing should be rebuilt. The caching job is actually getting too big for the free github runners (see https://github.com/celeritas-project/spack-gha-buildcache/actions/runs/8304439200 ) so I may soon have to build the cache on a non-GHA system using docker. |
sethrj
added a commit
that referenced
this pull request
Mar 18, 2024
…1149) * Add note about pull requests * Show ccache stats even on failure * Add spack-based CI build * Add spack build to push/pr * Mark failing builds as experimental * Fix missing compiler/version from matrix * Use patched geant4 in upstream spack * Use only celer buildcache * Shorten padded length * Don't push the buildcache... * Load version information to fix examples
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds a new github action for testing arbitrary combinations of dependencies, starting with all Geant4 versions. It uses a spack buildcache hosted at https://github.com/celeritas-project/spack-gha-buildcache to speed up builds. Because of build/configure/example errors, basically everything but 11.0 is failing at the moment, so I am labeling the jobs as "experimental" (allowed to fail) and will fix the relevant code in a follow-on PR (#1152).