Skip to content

PGI: add OpenMPI and pompi toolchain easyconfigs.#2899

Merged
boegel merged 5 commits intoeasybuilders:developfrom
bartoldeman:pompi
May 11, 2016
Merged

PGI: add OpenMPI and pompi toolchain easyconfigs.#2899
boegel merged 5 commits intoeasybuilders:developfrom
bartoldeman:pompi

Conversation

@bartoldeman
Copy link
Copy Markdown
Contributor

I hope to be able to uncomment the sanity_check_commands. For some strange reason, when they are run, mpicc does not find pgcc in the PATH; however after loading the module the mpicc command works just fine.

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@bartoldeman
Copy link
Copy Markdown
Contributor Author

bartoldeman commented Apr 15, 2016

(depends on easybuilders/easybuild-framework#1342 and #2879).

@boegel
Copy link
Copy Markdown
Member

boegel commented Apr 15, 2016

Jenkins: ok to test

@boegel boegel added this to the v2.8.0 milestone Apr 15, 2016
@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7569/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 2, 2016

@bartoldeman This PR is missing an hwloc easyconfig, fixed in bartoldeman#2

bump hwloc to 1.11.3, add missing hwloc easyconfig file
# ('mpicc --version | grep pgcc', ''),
# ('mpicxx --version | grep pgc++', ''),
# ('mpifort --version | grep pgfortran', ''),
#]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@bartoldeman we'll need to sort this out too...

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7950/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

@bartoldeman I traced these OpenMPI sanity check commands failing for you but not for me down to a bug in the EasyBuild framework, see easybuilders/easybuild-framework#1754

The problem was that the module load being done at the start of the sanity check was not being done with the correct (short) module name, which may causes problems when using a hierarchical module naming scheme. The problem remained hidden up until now because it's only a real issue if: the toolchain being used is a non-bundle but an actual software package (like PGI is) and if there are sanity check commands listed that can fail if the module for the toolchain itself is not loaded (as is the case here).

Can you verify whether the fix in easybuilders/easybuild-framework#1754 fixes the issue, i.e. whether you can install OpenMPI using the PGI toolchain with having these sanity check commands active using your hierarchical module naming scheme ToolchainMNS?

@bartoldeman
Copy link
Copy Markdown
Contributor Author

It works with PR easybuilders/easybuild-framework#1754 so I uncommented those lines now.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7963/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 10, 2016

@bartoldeman great, thanks for the confirmation!

@boegel
Copy link
Copy Markdown
Member

boegel commented May 10, 2016

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in this PR)
node2159.delcatty.os - Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/7b08be48f1ae9244b241feac5639f270 for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 10, 2016

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in this PR)
node2565.golett.os - Linux centos linux 7.2.1511, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/a024f9289ae5b56f397c4e9cd2696d0b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 10, 2016

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (3 easyconfigs in this PR)
node2159.delcatty.os - Linux SL 6.7, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/7b7065bfcafbbe7f29b0cdb6bd6b0836 for a full test report.

edit: using HierarchicalMNS, as confirmation of bugfix easybuilders/easybuild-framework#1754

@boegel
Copy link
Copy Markdown
Member

boegel commented May 10, 2016

Jenkins: test this please

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7968/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 11, 2016

Good to go now that easybuilders/easybuild-framework#1342 is merged, thanks @bartoldeman!

@boegel boegel merged commit 74c649f into easybuilders:develop May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants