Skip to content

update TALYS sources to (hopefully) ensure constant checksums#14797

Merged
boegel merged 1 commit intoeasybuilders:developfrom
jfgrimm:20220120115645_new_pr_TALYS195
Feb 2, 2022
Merged

update TALYS sources to (hopefully) ensure constant checksums#14797
boegel merged 1 commit intoeasybuilders:developfrom
jfgrimm:20220120115645_new_pr_TALYS195

Conversation

@jfgrimm
Copy link
Copy Markdown
Member

@jfgrimm jfgrimm commented Jan 20, 2022

(created using eb --new-pr)

@jfgrimm jfgrimm changed the title Update sources to (hopefully) ensure constant checksums Update TALYS sources to (hopefully) ensure constant checksums Jan 20, 2022
@boegel boegel added the bug fix label Feb 2, 2022
@boegel boegel added this to the next release (4.5.3?) milestone Feb 2, 2022
@boegel boegel changed the title Update TALYS sources to (hopefully) ensure constant checksums update TALYS sources to (hopefully) ensure constant checksums Feb 2, 2022
boegel
boegel approved these changes Feb 2, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2022

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 2, 2022

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=14797 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_14797 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1027687249 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).

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2022

@boegelbot please test @ jsc-zen2

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 2, 2022

@boegel: Request for testing this PR well received on jsfl1.int.jusuf.sebastian.cluster

PR test command 'EB_PR=14797 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_14797 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 1027691144 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 Feb 2, 2022

Test report by @boegelbot
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/37cfb6f4ef954b43498d3b7239ffc2da for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Feb 2, 2022

Test report by @boegelbot
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
jsfc01.int.jusuf.sebastian.cluster - Linux Rocky Linux 8.4, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/92f9125a9410cfa2b9cf791bf9b6aad4 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2022

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3112.skitty.os - Linux UNKNOWN UNKNOWN, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/c8b65e6b951c909969acaade547e0341 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 2, 2022

Going in, thanks @jfgrimm!

@boegel boegel merged commit 2432168 into easybuilders:develop Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.3

Development

Successfully merging this pull request may close these issues.

3 participants

@jfgrimm @boegel @boegelbot