[LU-3279] Interop 2.3.0 <-> 2.4 failure on test suite lustre-rsync-test test_7: Failure in replication; differences found

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.4.0, Lustre 2.1.6
• Affects Version/s: Lustre 2.4.0
• Labels: MB
• Environment: client: 2.3.0; server: http://review.whamcloud.com/#change,6252 patch set 5; client uses the t-f.sh from server
• Severity: 3
• Assignee: Mikhail Pershin
• Reporter: Maloo

Description

This issue was created by maloo for sarah <[e-mail]>

This issue relates to the following test suite run: http://maloo.whamcloud.com/test_sets/5584e96e-b5de-11e2-9d08-52540035b04c.

The sub-test test_7 failed with the following error:

Failure in replication; differences found.

Info required for matching: lustre-rsync-test 7

Issue Links

• is related to: LU-3190 Interop 2.3.0<->2.4 Failed on lustre-rsync-test test 3b: ASSERTION( lio->lis_lsm != ((void *)0) ) failed

Comments

[Peter Jones]

Andreas

All the present patches in flight have landed. Is any further work required or can this ticket be marked as resolved?

Peter

[Andreas Dilger]

I've pushed http://review.whamcloud.com/6338 as a follow-on patch for master. This changes the comment for CL_LAYOUT, and also changes the output string for this record type to match the 5-character convention used by other names.

I pushed http://review.whamcloud.com/6335 as a combined patch for b2_1.

[Andreas Dilger]

We have concerns about the comment that has been added to lustre/lustre_user.h:650 though. As Jinshan Xiong noted, stating that CL_LAYOUT doesn't reflect any actual data change is misleading.

My thought is that the content of the file is not changing, either in the case of file release and file migrate. In the case of file release, the content is still in the archive. Even if it is not in Lustre, any application accessing that file would get the same data back after it is restored from the archive.

There is of course some concern that a poorly-written layout swap could change the content of the file (e.g. source and target file do not contain the same data), but I can't see any reason to do that. Only the file owner could do this migrate, and they could just as easily write some other data directly into the file if they want.

In any case, I'll remove the comment from the CL_LAYOUT description.

[Sarah Liu]

interop between 2.1.5 client and 2.4 server also hit this issue in tag-2.3.65 testing:

https://maloo.whamcloud.com/test_sets/3e989b54-b9a5-11e2-875f-52540035b04c

[Henri Doreau (Inactive)]

Thanks for the heads up. The change doesn't affect our tools.

We have concerns about the comment that has been added to lustre/lustre_user.h:650 though. As Jinshan Xiong noted, stating that CL_LAYOUT doesn't reflect any actual data change is misleading.

[Prakash Surya (Inactive)]

I appreciate the heads up! . AFAIK, we do not use any tools external to the Lustre tree which make use of changelogs except robinhood. Can one of the robinhood developers comment on this? I'm not familiar with its internals to know if it would affect it or not..

[Oleg Drokin]

Just a heads up to CEA and LLNL:

Current patch in review changes LAYOUT changelog record type to an incompatible value. Do you guys currently monitor it from your tools, would you be affected by the change?