Skip to content

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

Merged
boegel merged 1 commit intoeasybuilders:developfrom
lexming:impi_ucxtls
Nov 28, 2020
Merged

allow $UCX_TLS to be overwritten in the easyconfig of impi#2258
boegel merged 1 commit intoeasybuilders:developfrom
lexming:impi_ucxtls

Conversation

@lexming
Copy link
Copy Markdown
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
Copy Markdown
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
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

@boegel boegel merged commit da4ea87 into easybuilders:develop Nov 28, 2020
@lexming
Copy link
Copy Markdown
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
Copy Markdown
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