Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

allow $UCX_TLS to be overwritten in the easyconfig of impi #2258

Merged
merged 1 commit into from
Nov 28, 2020
Merged

allow $UCX_TLS to be overwritten in the easyconfig of impi #2258

merged 1 commit into from
Nov 28, 2020

Conversation

lexming
Copy link
Contributor

@lexming lexming commented Nov 28, 2020 •
edited
Loading

Improvement of #2253 to allow user to define UCX_TLS in the easyconfig of impi and added a reference to the original PR where there is a detailed explanation of the issue.

@lexming lexming added this to the 4.3.2 (next release) milestone Nov 28, 2020
@boegel boegel changed the title allow UCX_TLS to be overwritten in the easyconfig of impi allow $UCX_TLS to be overwritten in the easyconfig of impi Nov 28, 2020
@boegel
Copy link
Member

boegel commented Nov 28, 2020

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS impi-2019.7.217-iccifort-2020.1.217.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node2643.swalot.os - Linux centos linux 7.8.2003, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/988fa1cd34bc9aacaba29f47e7c5c883 for a full test report.

boegel
boegel approved these changes Nov 28, 2020
View reviewed changes
Copy link
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

@boegel boegel merged commit da4ea87 into easybuilders:develop Nov 28, 2020
@lexming
Copy link
Contributor Author

lexming commented Nov 28, 2020

Test report by @lexming

Overview of tested easyconfigs (in order)

  • SUCCESS impi-2019.9.304-iccifort-2020.4.304.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
login2.cerberus.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz, Python 2.7.5
See https://gist.github.com/9608ddab70df71953d85a8837fa7ab32 for a full test report.

@lexming
Copy link
Contributor Author

lexming commented Nov 28, 2020

Test report by @lexming

Overview of tested easyconfigs (in order)

  • SUCCESS impi-2019.7.217-iccifort-2020.1.217.eb

Build succeeded for 1 out of 1 (1 easyconfigs in total)
node226.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2680 v4 @ 2.40GHz, Python 2.7.5
See https://gist.github.com/7d9b0d736b7da74cd767eba685149c04 for a full test report.

@lexming lexming deleted the impi_ucxtls branch November 28, 2020 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@boegel boegel boegel approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
4.3.2
Development

Successfully merging this pull request may close these issues.
2 participants
@lexming @boegel