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

save and restore $DIR1 in sanity-dom.sh

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.11.0
    • Lustre 2.11.0
    • None
    • 3
    • 9223372036854775807

    Description

      In sanity-dom.sh, DIR1 is directly configured as follows at the beginning of the script:

      lfs setstripe -E $DOM_SIZE -L mdt -E EOF $DIR1
      

      While in cfg/local.sh, it's defined as:

      MOUNT=${MOUNT:-/mnt/${FSNAME}}
      MOUNT1=${MOUNT1:-$MOUNT}
      DIR=${DIR:-$MOUNT}
      DIR1=${DIR:-$MOUNT1}
      

      So, $DIR1 is $MOUNT by default, and the DOM configuration change is actually made on $MOUNT directly, which affects all of the tests that run after sanity-dom.sh.

      Attachments

        Issue Links

          Activity

            [LU-10488] save and restore $DIR1 in sanity-dom.sh

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30842/
            Subject: LU-10488 tests: fix sub-test return value issue in sanity-dom.sh
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 6dd8066cdfb9d2f4c33698f76caaeef76081d9e9

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30842/ Subject: LU-10488 tests: fix sub-test return value issue in sanity-dom.sh Project: fs/lustre-release Branch: master Current Patch Set: Commit: 6dd8066cdfb9d2f4c33698f76caaeef76081d9e9

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30836/
            Subject: LU-10488 tests: saved and restore layout for dom test
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 534f15c0fb239cc2c3a9593fec346544d441783b

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30836/ Subject: LU-10488 tests: saved and restore layout for dom test Project: fs/lustre-release Branch: master Current Patch Set: Commit: 534f15c0fb239cc2c3a9593fec346544d441783b

            Jian Yu (jian.yu@intel.com) uploaded a new patch: https://review.whamcloud.com/30842
            Subject: LU-10488 tests: fix sub-test return value issue in sanity-dom.sh
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 34981876c1b76fe847205b60e320a0293795d540

            gerrit Gerrit Updater added a comment - Jian Yu (jian.yu@intel.com) uploaded a new patch: https://review.whamcloud.com/30842 Subject: LU-10488 tests: fix sub-test return value issue in sanity-dom.sh Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 34981876c1b76fe847205b60e320a0293795d540

            Jinshan Xiong (jinshan.xiong@intel.com) uploaded a new patch: https://review.whamcloud.com/30836
            Subject: LU-10488 tests: saved and restore layout for dom test
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 35c0da8ee28fdf7ce0875ac01dfeeee296bf8beb

            gerrit Gerrit Updater added a comment - Jinshan Xiong (jinshan.xiong@intel.com) uploaded a new patch: https://review.whamcloud.com/30836 Subject: LU-10488 tests: saved and restore layout for dom test Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 35c0da8ee28fdf7ce0875ac01dfeeee296bf8beb
            yujian Jian Yu added a comment -

            I found another issue in sanity-dom.sh:
            https://testing.hpdd.intel.com/test_sets/fc89fcf6-f655-11e7-94c7-52540065bddc

            There were two sub-test failures but sanity-dom was marked as PASS.

            yujian Jian Yu added a comment - I found another issue in sanity-dom.sh: https://testing.hpdd.intel.com/test_sets/fc89fcf6-f655-11e7-94c7-52540065bddc There were two sub-test failures but sanity-dom was marked as PASS.
            yujian Jian Yu added a comment -

            If sanity-flr was run after sanity-dom, then sanity-flr test 0d will hit the following failure:

            == sanity-flr test 0d: lfs mirror extend with -N option ============================================== 19:05:51 (1515611151)
            error: lfs mirror extend: doesn't support dom+flr for now
             sanity-flr test_0d: @@@@@@ FAIL: convert and extend /mnt/lustre/d0d.sanity-flr/f0d.sanity-flr failed 
            

            https://testing.hpdd.intel.com/test_sessions/a4607a95-681d-44d5-a566-d73c003ac999

            yujian Jian Yu added a comment - If sanity-flr was run after sanity-dom, then sanity-flr test 0d will hit the following failure: == sanity-flr test 0d: lfs mirror extend with -N option ============================================== 19:05:51 (1515611151) error: lfs mirror extend: doesn't support dom+flr for now sanity-flr test_0d: @@@@@@ FAIL: convert and extend /mnt/lustre/d0d.sanity-flr/f0d.sanity-flr failed https://testing.hpdd.intel.com/test_sessions/a4607a95-681d-44d5-a566-d73c003ac999

            People

              jay Jinshan Xiong (Inactive)
              yujian Jian Yu
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: