Skip to content

v5.0.x: help-smsc-knem.txt: fix incorrect messages, remove stale messages #9819

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.

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

Merged

Conversation

jsquyres
Copy link
Member

@jsquyres jsquyres commented Jan 3, 2022

Removed a bunch of stale messages that are not relevant to this
show_help text file any more (they were left over from before the smsc
framework).

Additionally, update the knem show help messages to show the correct
MCA varable name and value. Also fudge the message a bit to no longer
specifically mention the sm BTL.

Thanks to Benjamin Kitor @BKitor for reporting the issue.

Signed-off-by: Jeff Squyres jsquyres@cisco.com
(cherry picked from commit 53f09f7)

Fixes #9802

Removed a bunch of stale messages that are not relevant to this
show_help text file any more (they were left over from before the smsc
framework).

Additionally, update the knem show help messages to show the correct
MCA varable name and value.  Also fudge the message a bit to no longer
specifically mention the sm BTL.

Thanks to Benjamin Kitor @BKitor for reporting the issue.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
(cherry picked from commit 53f09f7)
@jsquyres jsquyres added this to the v5.0.0 milestone Jan 3, 2022
@jsquyres jsquyres requested a review from hjelmn January 3, 2022 16:03
@jsquyres jsquyres linked an issue Jan 3, 2022 that may be closed by this pull request
@awlauria
Copy link
Contributor

awlauria commented Jan 6, 2022

bot:aws:retest

@gpaulsen gpaulsen merged commit 6faeac7 into open-mpi:v5.0.x Jan 6, 2022
@gpaulsen
Copy link
Member

gpaulsen commented Jan 6, 2022

Didn't wait for retest as this is just a .txt file update.

@jsquyres jsquyres deleted the pr/5.0.x/fix-smsc-knem-show-help-messages branch January 6, 2022 21:08
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.

SMSC/knem throws error messages referencing BTL/knem
4 participants