[LU-16813] sanity-flr/42 fails due to LU-16518 Created: 10/May/23  Updated: 31/May/23  Resolved: 31/May/23

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.16.0

Type: Bug Priority: Minor
Reporter: Alex Zhuravlev Assignee: Tim Day
Resolution: Fixed Votes: 0
Labels: None

Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

LU-16518 introduced this change:

@@ -2942,7 +2942,7 @@ int llapi_mirror_resync_many(int fd, struct llapi_layout *layout,
                return -rc;
 
        while (pos < end) {
-               uint64_t mirror_end;
+               uint64_t mirror_end = LUSTRE_EOF;

and this breaks sanity-flr/42:

== sanity-flr test 42: lfs mirror verify ================= 07:18:32 (1683703112)
..
lfs mirror verify: chunk [0x400000, 0x600000) has different checksum value on mirror 2 and mirror 4.
CRC-32 checksum value for chunk [0x600000, 0xa00000):
Mirror 2:	0x1f8ad0d8
Mirror 4:	0x18975bf9

lfs mirror verify: chunk [0x600000, 0x800000) has different checksum value on mirror 2 and mirror 4.
CRC-32 checksum value for chunk [0x800000, 0xc00000):
Mirror 2:	0x4649ddde
Mirror 4:	0x6d648e1d

lfs mirror verify: chunk [0x800000, 0xa00000) has different checksum value on mirror 2 and mirror 4.
CRC-32 checksum value for chunk [0xa00000, 0xe00000):
Mirror 2:	0xf6c7f2c4
Mirror 4:	0xddeaa107

lfs mirror verify: chunk [0xa00000, 0x1000000) has different checksum value on mirror 2 and mirror 4.
lfs mirror verify: '/mnt/lustre/d42.sanity-flr/f42.sanity-flr-1' chunk [0x1000000, 0xffffffffffffffff) exceeds file size 0xa00002: skipped
lfs mirror verify: chunk [0x400000, 0x600000) has different checksum value on mirror 1 and mirror 2.
 sanity-flr test_42: @@@@@@ FAIL: verify /mnt/lustre/d42.sanity-flr/f42.sanity-flr-1 with mirror 1 and 2 should succeed 
  Trace dump:
  = ./../tests/test-framework.sh:6597:error()
  = sanity-flr.sh:2023:test_42()
  = ./../tests/test-framework.sh:6937:run_one()
  = ./../tests/test-framework.sh:6986:run_one_logged()
  = ./../tests/test-framework.sh:6808:run_test()
  = sanity-flr.sh:2042:main()
Dumping lctl log to /tmp/ltest-logs/sanity-flr.test_42.*.1683703116.log
Dumping logs only on local client.
FAIL 42 (4s)


 Comments   
Comment by Gerrit Updater [ 10/May/23 ]

"Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50906
Subject: LU-16813 utils: initialize mirror_end var
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0f898f3ae31e5f37aa7ed93ba5657502f5b71dfb

Comment by Gerrit Updater [ 10/May/23 ]

"Timothy Day <timday@amazon.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/50919
Subject: LU-16813 utils: move mirror_end initialization
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 731d596f95a967835b11ece5829002c14e8bb451

Comment by Gerrit Updater [ 31/May/23 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50919/
Subject: LU-16813 utils: move mirror_end initialization
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4cd4bfba473fb370767e1f2014d9fe1531889f82

Comment by Peter Jones [ 31/May/23 ]

Landed for 2.16

Generated at Sat Feb 10 03:30:13 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.