Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-13086

3c7aca747 LU-12395 breaks compatibility mpi tests with mpich

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.15.0
    • None
    • None
    • 3
    • 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

          Bug - A problem which impairs or prevents the functions of the product. LU-13348 Landing LU-12395 broke an MPICH support

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-12395 Failed dependencies while installing lustre-client-tests and lustre-iokit rpms on RHEL 8.0 client

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Activity

            [LU-13086] 3c7aca747 LU-12395 breaks compatibility mpi tests with mpich
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/38689/
            Subject: LU-13086 tests: restore compatibility with mpich
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: e418f47688facf07f2e9bd6535b71d484af4f8ac

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/38689/ Subject: LU-13086 tests: restore compatibility with mpich Project: fs/lustre-release Branch: master Current Patch Set: Commit: e418f47688facf07f2e9bd6535b71d484af4f8ac
            colmstea Charlie Olmstead added a comment -

            AT Patch has been deployed

            colmstea Charlie Olmstead added a comment - AT Patch has been deployed
            gerrit Gerrit Updater added a comment -

            "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 Gerrit Updater added a comment - "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
            spitzcor Cory Spitz added a comment -

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

            spitzcor Cory Spitz added a comment - Is https://review.whamcloud.com/#/c/41082/ supposed to be a replacement for https://review.whamcloud.com/#/c/38689/ ?
            spitzcor Cory Spitz added a comment -

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

            spitzcor Cory Spitz added a comment - I don't have permissions to view https://review.whamcloud.com/#/c/41082/ .
            gerrit Gerrit Updater added a comment -

            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 Gerrit Updater added a comment - 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
            adilger Andreas Dilger added a comment -

            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.

            adilger Andreas Dilger added a comment - 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.
            spitzcor Cory Spitz added a comment -

            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?

            spitzcor Cory Spitz added a comment - 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?
            spitzcor Cory Spitz added a comment -

            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?

            spitzcor Cory Spitz added a comment - 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?
            spitzcor Cory Spitz added a comment -

            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?

            spitzcor Cory Spitz added a comment - 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 Gerrit Updater added a comment -

            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 Gerrit Updater added a comment - 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

            People

              egryaznova Elena Gryaznova
              egryaznova Elena Gryaznova
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: