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

interop: 2.1(server)<->2.8(client): conf-sanity test_82: mkfs.lustre FATAL: Bad argument: /dev/vdb

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.10.0
    • Lustre 2.1.6
    • Interop: 2.1.6 <-> 2.8.0
      server- 2.1.6 and older.
      client- after 2.7.
    • 3
    • 9223372036854775807

    Description

      In conf-sanity_test_83 two appearance of "$dev" var will give reported error when server is 2.1 or older.

              add ost1 $(mkfs_opts ost1 $dev) $FSTYPE_OPT \
                      --reformat $dev $dev > /dev/null ||
                      error "format ost1 error"
      

      Although it is clear that check is added to to skip the test if the ost version is older than 2.6.91

      [[ $(lustre_version_code ost1) -ge $(version_code 2.6.91) ]] ||
                      { skip "Need OST version at least 2.6.91" && return 0; }
      

      still as a good coding practice that parameter is erroneous hence should be removed.

      Attachments

        Activity

          [LU-8730] interop: 2.1(server)<->2.8(client): conf-sanity test_82: mkfs.lustre FATAL: Bad argument: /dev/vdb
          pjones Peter Jones added a comment -

          Landed for 2.10

          pjones Peter Jones added a comment - Landed for 2.10

          Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/23249/
          Subject: LU-8730 test: Remove duplicate $dev from conf-sanity test_83
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 185251325971a8111f3e821cde4f18f9533f8746

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/23249/ Subject: LU-8730 test: Remove duplicate $dev from conf-sanity test_83 Project: fs/lustre-release Branch: master Current Patch Set: Commit: 185251325971a8111f3e821cde4f18f9533f8746

          ashish.maurya (ashish.maurya@seagate.com) uploaded a new patch: http://review.whamcloud.com/23249
          Subject: LU-8730 test: Remove duplicate $dev from conf-sanity test_83
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 583da287f3ea23c197b357513c70a12cf2724570

          gerrit Gerrit Updater added a comment - ashish.maurya (ashish.maurya@seagate.com) uploaded a new patch: http://review.whamcloud.com/23249 Subject: LU-8730 test: Remove duplicate $dev from conf-sanity test_83 Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 583da287f3ea23c197b357513c70a12cf2724570

          People

            wc-triage WC Triage
            ashish.maurya Ashish Maurya (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: