Skip to content

Conversation

@albertogfernan
Copy link
Contributor

(created using eb --new-pr)

@migueldiascosta
Copy link
Member

Test report by @migueldiascosta
FAILED
Build succeeded for 1 out of 2 (2 easyconfigs in this PR)
generoso - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2660 v4 @ 2.00GHz, Python 3.6.8
See https://gist.github.com/8666d0a980581f5cb413003b164c42d5 for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Mar 6, 2020
@migueldiascosta
Copy link
Member

thanks @albertogfernan!

The test failure is due a failing checksum, which is likey due to the use of git_config, see note in https://easybuild.readthedocs.io/en/latest/Writing_easyconfig_files.html#common-easyconfig-param-sources-git-config. I suppose using the release packages is not an option?

Related to easybuilders/easybuild-framework#2727 and easybuilders/easybuild-framework#2727

boegel
boegel previously requested changes Mar 6, 2020
@migueldiascosta migueldiascosta dismissed stale reviews from boegel and themself March 26, 2020 14:27

changes addressed

@migueldiascosta
Copy link
Member

Test report by @migueldiascosta
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
sms - Linux centos linux 7.6.1810, AMD EPYC 7601 32-Core Processor, Python 2.7.5
See https://gist.github.com/1fa38d85e0ff03fce4cfb975d33e4e04 for a full test report.

@migueldiascosta
Copy link
Member

Test report by @migueldiascosta
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
generoso - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2660 v4 @ 2.00GHz, Python 3.6.8
See https://gist.github.com/148de0ff0edc5fc75b2e5657240411b5 for a full test report.

Copy link
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Member

Going in, thanks @albertogfernan!

@migueldiascosta migueldiascosta merged commit efccc5f into easybuilders:develop Mar 27, 2020
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.

3 participants