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
            pjones Peter Jones added a comment -

            Landed for 2.15

            pjones Peter Jones added a comment - Landed for 2.15
            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.

            People

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

              Dates

                Created:
                Updated:
                Resolved: