Skip to content

set $TEMPDIRPATH for testsuite in the BerkeleyGW easyblock, to avoid polluting /tmp#2836

Merged
boegel merged 1 commit intoeasybuilders:developfrom
migueldiascosta:20221129080718_new_pr_berkeleygw
Dec 27, 2022
Merged

set $TEMPDIRPATH for testsuite in the BerkeleyGW easyblock, to avoid polluting /tmp#2836
boegel merged 1 commit intoeasybuilders:developfrom
migueldiascosta:20221129080718_new_pr_berkeleygw

Conversation

@migueldiascosta
Copy link
Copy Markdown
Member

(created using eb --new-pr)

@migueldiascosta migueldiascosta added this to the next release (4.7.0) milestone Nov 29, 2022
@boegel boegel changed the title set TEMPDIRPATH for testsuite in the BerkeleyGW easyblock, to avoid polluting /tmp set $TEMPDIRPATH for testsuite in the BerkeleyGW easyblock, to avoid polluting /tmp Dec 6, 2022
@migueldiascosta
Copy link
Copy Markdown
Member Author

this shouldn't be a blocker for 4.7.0, but it's also trivial, do we need to test all the existing easyconfigs with it?

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 27, 2022

this shouldn't be a blocker for 4.7.0, but it's also trivial, do we need to test all the existing easyconfigs with it?

Not a blocker indeed, but some test report here would be nice (I just started the process for submitting one)

@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 27, 2022

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS BerkeleyGW-2.1.0-foss-2019b-Python-3.7.4.eb
  • SUCCESS BerkeleyGW-2.1.0-intel-2019b-Python-3.7.4.eb
  • SUCCESS BerkeleyGW-3.0.1-foss-2020b.eb
  • SUCCESS BerkeleyGW-3.0.1-foss-2021a.eb

Build succeeded for 4 out of 4 (4 easyconfigs in total)
node3101.skitty.os - Linux RHEL 8.6, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/ef3e3d1dc3e00a4e6dd8187d9f296ba9 for a full test report.

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

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.

2 participants