[LU-12639] sanity test 317 uses uninitialized variable $facet Created: 07/Aug/19 Updated: 21/Nov/19 Resolved: 15/Aug/19 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.13.0, Lustre 2.12.3 |
| Fix Version/s: | Lustre 2.13.0, Lustre 2.12.4 |
| Type: | Bug | Priority: | Minor |
| Reporter: | James Nunez (Inactive) | Assignee: | James Nunez (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | tests | ||
| Severity: | 3 |
| Rank (Obsolete): | 9223372036854775807 |
| Description |
|
sanity test 317 checks the file system type of $facet to determine if the test should be skipped. Unfortunately, facet is not initialized/defined in the test.
19369 test_317() {
19370 [ $MDS1_VERSION -lt $(version_code 2.11.53) ] &&
19371 skip "Need MDS version at least 2.11.53"
19372 local trunc_sz
19373 local grant_blk_size
19374
19375 if [ "$(facet_fstype $facet)" == "zfs" ]; then
19376 skip "LU-10370: no implementation for ZFS" && return
19377 fi
...
We need to replace $facet with the correct server. Test 317 was added to sanity with patch https://review.whamcloud.com/31073 . |
| Comments |
| Comment by Gerrit Updater [ 07/Aug/19 ] |
|
James Nunez (jnunez@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/35716 |
| Comment by Gerrit Updater [ 15/Aug/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/35716/ |
| Comment by Peter Jones [ 15/Aug/19 ] |
|
Landed for 2.13 |
| Comment by Gerrit Updater [ 03/Oct/19 ] |
|
James Nunez (jnunez@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36366 |
| Comment by Gerrit Updater [ 21/Nov/19 ] |
|
Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36366/ |