[LU-10488] save and restore $DIR1 in sanity-dom.sh Created: 10/Jan/18 Updated: 20/Jan/18 Resolved: 16/Jan/18 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.11.0 |
| Fix Version/s: | Lustre 2.11.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Jian Yu | Assignee: | Jinshan Xiong (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 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. |
| Comments |
| Comment by Jian Yu [ 11/Jan/18 ] |
|
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 |
| Comment by Jian Yu [ 11/Jan/18 ] |
|
I found another issue in sanity-dom.sh: There were two sub-test failures but sanity-dom was marked as PASS. |
| Comment by Gerrit Updater [ 11/Jan/18 ] |
|
Jinshan Xiong (jinshan.xiong@intel.com) uploaded a new patch: https://review.whamcloud.com/30836 |
| Comment by Gerrit Updater [ 11/Jan/18 ] |
|
Jian Yu (jian.yu@intel.com) uploaded a new patch: https://review.whamcloud.com/30842 |
| Comment by Gerrit Updater [ 11/Jan/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30836/ |
| Comment by Gerrit Updater [ 20/Jan/18 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30842/ |