Skip to content

Set additional environment variables needed by MDSplus#3027

Merged
boegel merged 1 commit intoeasybuilders:developfrom
SimonPinches:3026_MDSplus-environment-variables
May 9, 2016
Merged

Set additional environment variables needed by MDSplus#3027
boegel merged 1 commit intoeasybuilders:developfrom
SimonPinches:3026_MDSplus-environment-variables

Conversation

@SimonPinches
Copy link
Copy Markdown
Contributor

Add missing environment variables to MDSplus modules.

@hpcugentbot
Copy link
Copy Markdown

Automatic reply from Jenkins: Can I test this?

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

Jenkins: ok to test

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

@SimonPinches ignored the failed test reports, too strict ulimit settings causing trouble with Java...

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2417.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/79a374b99f7157911a7a97db46a007bb for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 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/45e24e6b9b014da0b34ae75df640ea97 for a full test report.

@hpcugentbot
Copy link
Copy Markdown

Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7944/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 9, 2016

lgtm

@boegel
Copy link
Copy Markdown
Member

boegel commented May 9, 2016

Going in, thanks @SimonPinches!

@boegel boegel merged commit 18b128b into easybuilders:develop May 9, 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