Skip to content

Adding missing GNU time dependency to WRF 4.0.2 built with foss/2018b#12179

Merged
branfosj merged 3 commits intoeasybuilders:developfrom
dithwick:20210217093130_new_pr_time19
Feb 26, 2021
Merged

Adding missing GNU time dependency to WRF 4.0.2 built with foss/2018b#12179
branfosj merged 3 commits intoeasybuilders:developfrom
dithwick:20210217093130_new_pr_time19

Conversation

@dithwick
Copy link
Copy Markdown
Contributor

(created using eb --new-pr)

branfosj
branfosj previously approved these changes Feb 17, 2021
Copy link
Copy Markdown
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj
Copy link
Copy Markdown
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

@branfosj: Request for testing this PR well received on generoso

PR test command 'EB_PR=12179 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12179 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 780619161 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

Test report by @boegelbot
FAILED
Build succeeded for 3 out of 4 (2 easyconfigs in total)
generoso-x-3 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/8a7c2f7f0d2fe21b4e8a2bbd3db98a80 for a full test report.

@branfosj branfosj dismissed their stale review February 17, 2021 16:16

dismiss review as there is a build failure to debug

@branfosj
Copy link
Copy Markdown
Member

Failure seems to be (unless there was one higher in the log - before the bit we can see):

fatal error: rpc/types.h: No such file or directory

This suggests needing a dependency on https://github.com/easybuilders/easybuild-easyconfigs/blob/develop/easybuild/easyconfigs/l/libtirpc/libtirpc-1.1.4-GCCcore-7.3.0.eb

I'm currently running a test on CentOS 8 to see if I can confirm.

@dithwick
Copy link
Copy Markdown
Contributor Author

Thanks @branfosj

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 17, 2021

@branfosj See also #11496 (comment)

@dithwick
Copy link
Copy Markdown
Contributor Author

I'll see if the patch with #11496 (comment) works first, if not I'll create one. I'll need someone to test on CentOS 8 though.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 19, 2021

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

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

PR test command 'EB_PR=12179 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12179 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 782222398 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 19, 2021

Test report by @boegel
FAILED
Build succeeded for 4 out of 5 (2 easyconfigs in total)
node3600.doduo.os - Linux RHEL 8.2, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/eb72a0f855e2cbc83c881d6832d964a9 for a full test report.

@boegelbot
Copy link
Copy Markdown
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 1 out of 2 (2 easyconfigs in total)
generoso-c1-s-2 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/70357ea746fd93646cdd7b93b986500b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 19, 2021

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in total)
node2609.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/017448ce0298cee537bb5fcfad8adf26 for a full test report.

@dithwick
Copy link
Copy Markdown
Contributor Author

Hopefully this will work. Also flagging wrf-model/WRF#1072 for reference.

@branfosj
Copy link
Copy Markdown
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Copy Markdown
Collaborator

@branfosj: Request for testing this PR well received on generoso

PR test command 'EB_PR=12179 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_12179 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

Details

- notification for comment with ID 783976156 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

Test report by @boegelbot
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
generoso-c1-s-1 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/680098cec52ebdd5d5f2d7c954c1374d for a full test report.

@branfosj
Copy link
Copy Markdown
Member

Test report by @branfosj
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
bear-pg0211u03a.bear.cluster - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) Gold 6248 CPU @ 2.50GHz (cascadelake), Python 3.6.8
See https://gist.github.com/f6d13ecda2bec5c8b07b6826daadd7db for a full test report.

@boegel boegel changed the title Adding missing GNU time dependency to WRF and moved buildtype/versionsuffix to top. Adding missing GNU time dependency to WRF 4.0.2 built with foss/2018b Feb 26, 2021
Copy link
Copy Markdown
Member

@branfosj branfosj left a comment

Choose a reason for hiding this comment

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

lgtm

@branfosj
Copy link
Copy Markdown
Member

Going in, thanks @dithwick!

@branfosj branfosj merged commit 63bde85 into easybuilders:develop Feb 26, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 26, 2021

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in total)
node2719.swalot.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/9cdfdb865b7dc18b485f6d292da534a9 for a full test report.

@dithwick dithwick deleted the 20210217093130_new_pr_time19 branch March 1, 2021 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants