-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add weekly build/test scripts #1213
base: develop
Are you sure you want to change the base?
Add weekly build/test scripts #1213
Conversation
closing temporarily so I don't spam the CI with jobs... |
Cron entry for Acorn: |
TO BE DISCUSSED AT THE MEETING - Feature/weekly build nrl
Within a Re-running everything after the recent merge, now I'm seeing a dreaded
Ironically, the build of Not clear on what I'm doing wrong / how to correct. Will try taking this out of |
@rickgrubin-tomorrow The error message is right there:
You need to use a shorter path. |
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.
I am fine with merging this into develop in its current stage, it works perfectly for my github action runs on nautilus. Maybe we should all just get to the step where the environments build and then populate the cache. We can make adjustments and add more capabilities in subsequent PRs, for example for downstream application testing and package unit testing.
If I can ever manage to log onto Acorn again I want to do another end-to-end test on there... But after that yes that's fine with me. |
I saw the message, and it's meaning was / is clear. Curious as to why I didn't see this prior, and the only thing that changed was the merge. That said, I imagine that's not the culprit. Thanks. I'll have to get the pipleline to shorten:
|
I re-tested on Acorn after the last batch of updates and it was smooth sailing, so @climbfuji if you're still down to merge this sooner and build it out further over time, I'm fine with that. |
Let's merge it after the spack dev update if that's ok |
@climbfuji can you check that the contents of .github/workflows/ look correct? Was deleting the ubuntu containers CI one of your modifications? |
Oh no - that must have slipped through. Can you restore the file |
Co-authored-by: Dom Heinzeller <[email protected]>
…ack into weekly_build
I just noticed that on Acorn, it was creating a directory called "file:" in $SPACK_ENV because of the way it handles the mirror path. I'm tweaking it to drop the "file://" part when it looks up the directory; @rickgrubin-tomorrow @climbfuji please re-test with this change and make sure I didn't break anything... |
Yes, that's fine. I never used |
Summary
This PR will add scripting for weekly unit testing and build caching of the unified env through Jenkins, cron jobs, etc.
Also a few updates for Acorn, including a change to
spack stack create env --compiler
that allows compiler version to be specified (spack stack create env --compiler intel
->envs/myenv-intel
;spack stack create env --compiler intel@2022
->envs/myenv-intel-2022
), which is needed to differentiate onplatformsplatform where more than one version of a compiler is installed.Testing/implementation by platform
On each system:
Platforms (check when all of the above items are complete):
Applications affected
all
Systems affected
all (eventually)
Dependencies
none
Issue(s) addressed
#1184, #365
Checklist