Skip to content

Expand templates in checksum keys#22091

Merged
boegel merged 10 commits intoeasybuilders:5.0.xfrom
Micket:checksum_templates
Dec 21, 2024
Merged

Expand templates in checksum keys#22091
boegel merged 10 commits intoeasybuilders:5.0.xfrom
Micket:checksum_templates

Conversation

@Micket
Copy link
Copy Markdown
Contributor

@Micket Micket commented Dec 21, 2024 •
edited by boegel
Loading

There are more cases for things that do nested dicts with arch specific binary source tarballs. I started changing those as well, but there were a lot and it seems to work anyway, so i'm not sure if we should "fix" those as well.
They are all cases where a normal --inject-checksums wouldn't work (due to multiple source variants).

edit: fixes #22090

@Micket Micket added the EasyBuild-5.0 EasyBuild 5.0 label Dec 21, 2024
@Micket Micket added this to the 5.0 milestone Dec 21, 2024
@Micket Micket force-pushed the checksum_templates branch from deb2b89 to fcf6f70 Compare December 21, 2024 14:31
lexming
lexming reviewed Dec 21, 2024
View reviewed changes
Comment thread easybuild/easyconfigs/c/CellBender/CellBender-0.2.1-foss-2021a-CUDA-11.3.1.eb Outdated
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 21, 2024 •
edited
Loading

Test report by @boegel
SUCCESS
Build succeeded for 27 out of 27 (27 easyconfigs in total)
node3509.doduo.os - Linux RHEL 8.8, x86_64, AMD EPYC 7552 48-Core Processor, Python 3.6.8
See https://gist.github.com/boegel/bb13c02dcf4915cfce1955fd3a1d1aec for a full test report.

edit: this was using only --fetch, but that includes verifying of checksums

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 21, 2024

Test report by @boegel
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
node3509.doduo.os - Linux RHEL 8.8, x86_64, AMD EPYC 7552 48-Core Processor, Python 3.6.8
See https://gist.github.com/boegel/f49355993ea069c0af260d37c9f69ada for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 21, 2024

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3509.doduo.os - Linux RHEL 8.8, x86_64, AMD EPYC 7552 48-Core Processor, Python 3.6.8
See https://gist.github.com/boegel/b40d719cde503292ba71b5184ff27fb5 for a full test report.

@boegel boegel added bug fix and removed change labels Dec 21, 2024
@boegel boegel merged commit 9487eb3 into easybuilders:5.0.x Dec 21, 2024
@boegel boegel mentioned this pull request Dec 21, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lexming lexming lexming left review comments

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix EasyBuild-5.0 EasyBuild 5.0

Projects

None yet

Milestone

5.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@Micket @boegel @lexming