Skip to content

adopt module_load_environment in VEP easyblock#3602

Merged
lexming merged 2 commits intoeasybuilders:5.0.xfrom
boegel:20250216142749_new_pr_vep
Feb 17, 2025
Merged

adopt module_load_environment in VEP easyblock#3602
lexming merged 2 commits intoeasybuilders:5.0.xfrom
boegel:20250216142749_new_pr_vep

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Feb 16, 2025

(created using eb --new-pr)

@boegel boegel force-pushed the 20250216142749_new_pr_vep branch from 3754225 to 616d6b0 Compare February 16, 2025 13:28
@boegel boegel added change EasyBuild-5.0 EasyBuild 5.0 labels Feb 16, 2025
@boegel boegel added this to the 5.0 milestone Feb 16, 2025
@boegel boegel requested a review from lexming February 16, 2025 13:30
@boegel boegel force-pushed the 20250216142749_new_pr_vep branch from 616d6b0 to ccb19f2 Compare February 16, 2025 19:08
@boegel boegel force-pushed the 20250216142749_new_pr_vep branch from ccb19f2 to 6e322ec Compare February 16, 2025 19:11
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Feb 16, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS VEP-103.1-GCC-10.2.0.eb
  • SUCCESS VEP-105-GCC-11.2.0.eb
  • SUCCESS VEP-107-GCC-11.3.0.eb
  • SUCCESS VEP-111-GCC-12.2.0.eb
  • SUCCESS VEP-113.0-GCC-12.3.0.eb
  • SUCCESS VEP-113.3-GCC-13.3.0.eb

Build succeeded for 6 out of 6 (6 easyconfigs in total)
node3596.doduo.os - Linux RHEL 8.8, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegel/0fb256f0dd7caab6c4c322c827aa1c66 for a full test report.

Comment thread easybuild/easyblocks/v/vep.py Outdated
@lexming
Copy link
Copy Markdown
Contributor

lexming commented Feb 17, 2025

Test report by @lexming

Overview of tested easyconfigs (in order)

  • SUCCESS VEP-113.0-GCC-12.3.0.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
login1.cerberus.os - Linux Rocky Linux 8.10, x86_64, Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/lexming/c86a8294be07ff694df374a74b4c331f for a full test report.

Copy link
Copy Markdown
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lexming lexming merged commit 3715abd into easybuilders:5.0.x Feb 17, 2025
@boegel boegel deleted the 20250216142749_new_pr_vep branch February 17, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants