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

Expand templates in checksum keys #22091

Merged
merged 10 commits into from
Dec 21, 2024
Merged

Conversation

Micket
Copy link
Contributor

@Micket Micket commented Dec 21, 2024

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
@boegel
Copy link
Member

boegel commented Dec 21, 2024

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
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
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
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Changed default
Development

Successfully merging this pull request may close these issues.

3 participants