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

          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

            "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

            AT Patch has been deployed

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

            "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 -

            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/ .

            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

            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?

            People

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

              Dates

                Created:
                Updated:
                Resolved: