[LU-13086] 3c7aca747 LU-12395 breaks compatibility mpi tests with mpich - Whamcloud Community JIRA

Details

Type: Bug
Resolution: Fixed
Priority: Minor
Fix Version/s: Lustre 2.15.0
Affects Version/s: None
Labels:
None

Severity:
3
Rank (Obsolete):
9223372036854775807

Description

commit 3c7aca747 ~~LU-12395~~ adds "--oversubscribe" MPI run option which causes the MPI tests built with mpich fail:

[mpiexec@fre1307] match_arg (./utils/args/args.c:160): unrecognized argument oversubscribe
[mpiexec@fre1307] HYDU_parse_array (./utils/args/args.c:175): argument matching returned error

Attachments

Issue Links

is duplicated by

LU-13348 Landing LU-12395 broke an MPICH support

Resolved

is related to

LU-12395 Failed dependencies while installing lustre-client-tests and lustre-iokit rpms on RHEL 8.0 client

Resolved

Activity

[LU-13086] 3c7aca747 LU-12395 breaks compatibility mpi tests with mpich

Gerrit Updater added a comment - 02/Sep/21 6:44 PM

"Charlie Olmstead <charlie@whamcloud.com>" merged in patch https://review.whamcloud.com/41082/
Subject: ~~LU-13086~~ config: add --oversubscribe to MPIRUN_OPTIONS
Project: private/autotest
Branch: master
Current Patch Set:
Commit: c01e4db25d8a47a9c27cada04d8e3b4ea83292d4

Gerrit Updater added a comment - 02/Sep/21 6:44 PM "Charlie Olmstead <charlie@whamcloud.com>" merged in patch https://review.whamcloud.com/41082/ Subject: LU-13086 config: add --oversubscribe to MPIRUN_OPTIONS Project: private/autotest Branch: master Current Patch Set: Commit: c01e4db25d8a47a9c27cada04d8e3b4ea83292d4

Cory Spitz added a comment - 22/Feb/21 3:33 AM

Is https://review.whamcloud.com/#/c/41082/ supposed to be a replacement for https://review.whamcloud.com/#/c/38689/ ?

Cory Spitz added a comment - 22/Feb/21 3:33 AM Is https://review.whamcloud.com/#/c/41082/ supposed to be a replacement for https://review.whamcloud.com/#/c/38689/ ?

Cory Spitz added a comment - 04/Feb/21 5:47 PM

I don't have permissions to view https://review.whamcloud.com/#/c/41082/.

Cory Spitz added a comment - 04/Feb/21 5:47 PM I don't have permissions to view https://review.whamcloud.com/#/c/41082/ .

Gerrit Updater added a comment - 23/Dec/20 10:15 AM

Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/41082
Subject: ~~LU-13086~~ config: add --oversubscribe to MPIRUN_OPTIONS
Project: private/autotest
Branch: master
Current Patch Set: 1
Commit: cca5447c7ee27ec2c7bd916f6a634ef8371b5cbd

Gerrit Updater added a comment - 23/Dec/20 10:15 AM Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/41082 Subject: LU-13086 config: add --oversubscribe to MPIRUN_OPTIONS Project: private/autotest Branch: master Current Patch Set: 1 Commit: cca5447c7ee27ec2c7bd916f6a634ef8371b5cbd

Andreas Dilger added a comment - 22/Jul/20 12:22 AM

spitzcor, I've abandoned 38130 and updated 38689 to address the minor defect therein. It needs a second review and testing to finish before it can land. It looks like a reasonable compromise to include --oversubscribe so that the testing works out-of-the-box for RHEL (which is far-and-away the most common distro used with Lustre), but still allow the external config file to specify different options based on the MPI version.

Andreas Dilger added a comment - 22/Jul/20 12:22 AM spitzcor , I've abandoned 38130 and updated 38689 to address the minor defect therein. It needs a second review and testing to finish before it can land. It looks like a reasonable compromise to include --oversubscribe so that the testing works out-of-the-box for RHEL (which is far-and-away the most common distro used with Lustre), but still allow the external config file to specify different options based on the MPI version.

Cory Spitz added a comment - 21/Jul/20 10:27 PM

mdiep, colmstea, and jamesanunez, so https://review.whamcloud.com/#/c/38130/ should be abandoned? And in https://review.whamcloud.com/#/c/38689/ Elena has proposed to "totally get rid of --oversubscribe". Can you agree?

Cory Spitz added a comment - 21/Jul/20 10:27 PM mdiep , colmstea , and jamesanunez , so https://review.whamcloud.com/#/c/38130/ should be abandoned? And in https://review.whamcloud.com/#/c/38689/ Elena has proposed to "totally get rid of --oversubscribe". Can you agree?

Cory Spitz added a comment - 09/Jul/20 4:35 PM

mdiep, colmstea, and jamesanunez, with the activity at https://review.whamcloud.com/#/c/38689/ are we to assume that it is the preferred direction? And should https://review.whamcloud.com/#/c/38130/ be abandoned?

Cory Spitz added a comment - 09/Jul/20 4:35 PM mdiep , colmstea , and jamesanunez , with the activity at https://review.whamcloud.com/#/c/38689/ are we to assume that it is the preferred direction? And should https://review.whamcloud.com/#/c/38130/ be abandoned?

Cory Spitz added a comment - 22/Jun/20 5:26 PM

mdiep and jamesanunez, now we have options between https://review.whamcloud.com/#/c/38130/ and https://review.whamcloud.com/#/c/38689/, but reviews have gone stale. Can you please express what approach you want to proceed with and why?

Cory Spitz added a comment - 22/Jun/20 5:26 PM mdiep and jamesanunez , now we have options between https://review.whamcloud.com/#/c/38130/ and https://review.whamcloud.com/#/c/38689/ , but reviews have gone stale. Can you please express what approach you want to proceed with and why?

Gerrit Updater added a comment - 21/May/20 10:15 AM

Elena Gryaznova (c17455@cray.com) uploaded a new patch: https://review.whamcloud.com/38689
Subject: ~~LU-13086~~ tests: restore compatibility with mpich
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0ad3e4795d6490a630432950184b42f8a83811e3

Gerrit Updater added a comment - 21/May/20 10:15 AM Elena Gryaznova (c17455@cray.com) uploaded a new patch: https://review.whamcloud.com/38689 Subject: LU-13086 tests: restore compatibility with mpich Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0ad3e4795d6490a630432950184b42f8a83811e3

Cory Spitz added a comment - 20/May/20 4:57 PM - edited

mdiep and jamesanunez can you get together and reconcile your opinions about --oversubscribe and where you set it? James, it sounds like you can't get --oversubscribe in your env with the patch as-is. Is that right? If so, will you and Minh both be happy if it is moved to cfg/local.sh? Please update and clarify your comments in the Gerrit review. Thanks!

Cory Spitz added a comment - 20/May/20 4:57 PM - edited mdiep and jamesanunez can you get together and reconcile your opinions about --oversubscribe and where you set it? James, it sounds like you can't get --oversubscribe in your env with the patch as-is. Is that right? If so, will you and Minh both be happy if it is moved to cfg/local.sh? Please update and clarify your comments in the Gerrit review. Thanks!

Cory Spitz added a comment - 13/May/20 2:57 PM

mdiep, we could use your assistance with some questions in the review of https://review.whamcloud.com/#/c/38130/. Thanks!

Cory Spitz added a comment - 13/May/20 2:57 PM mdiep , we could use your assistance with some questions in the review of https://review.whamcloud.com/#/c/38130/ . Thanks!

People

Assignee:: Elena Gryaznova

Reporter:: Elena Gryaznova

Votes:: 0 Vote for this issue

Watchers:: 8 Start watching this issue

Dates

Created:: 18/Dec/19 2:15 PM

Updated:: 17/Sep/21 2:40 PM

Resolved:: 17/Sep/21 2:40 PM