Skip to content

don't fix shebang line in mympirun script, to make sure it uses system Python#22916

Merged
laraPPr merged 1 commit intoeasybuilders:developfrom
boegel:20250520121717_new_pr_vsc-mympirun530
May 20, 2025
Merged

don't fix shebang line in mympirun script, to make sure it uses system Python#22916
laraPPr merged 1 commit intoeasybuilders:developfrom
boegel:20250520121717_new_pr_vsc-mympirun530

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 20, 2025 •
edited
Loading

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 20, 2025

@boegelbot please test @ jsc-zen3

@boegel boegel added bug fix and removed change labels May 20, 2025
@boegel boegel added this to the next release (5.0.1 or 5.1.0) milestone May 20, 2025
@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented May 20, 2025

@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de

PR test command 'if [[ develop != 'develop' ]]; then EB_BRANCH=develop ./easybuild_develop.sh 2> /dev/null 1>&2; EB_PREFIX=/home/boegelbot/easybuild/develop source init_env_easybuild_develop.sh; fi; EB_PR=22916 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs EB_BRANCH=develop /opt/software/slurm/bin/sbatch --job-name test_PR_22916 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen3.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 6397

Test results coming soon (I hope)...

Details

- notification for comment with ID 2893815078 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented May 20, 2025

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
jsczen3c1.int.jsc-zen3.fz-juelich.de - Linux Rocky Linux 9.5, x86_64, AMD EPYC-Milan Processor (zen3), Python 3.9.21
See https://gist.github.com/boegelbot/83044c98a16e429fbdfec429de1fa3ec for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 20, 2025

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
node3625.doduo.os - Linux RHEL 9.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.9.18
See https://gist.github.com/boegel/d209af1604b4bc2a887eccd801d52d2d for a full test report.

@laraPPr
Copy link
Copy Markdown
Contributor

laraPPr commented May 20, 2025

Going in, thanks @boegel!

@laraPPr laraPPr merged commit 16d62c8 into easybuilders:develop May 20, 2025
8 checks passed
@boegel boegel deleted the 20250520121717_new_pr_vsc-mympirun530 branch May 20, 2025 14:16
@boegel boegel changed the title don't fix shebang line in mympirun script, to make sure it used system Python don't fix shebang line in mympirun script, to make sure it uses system Python May 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hajgato hajgato hajgato approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

5.1.0

Development

Successfully merging this pull request may close these issues.

4 participants

@boegel @boegelbot @laraPPr @hajgato