[LU-10344] sanity-hsm test_24a: atime is not less than mtime

• Status: Resolved
• Project: Lustre
• Fix Version/s: Lustre 2.11.0
• Type: Bug
• Priority: Critical
• Reporter: Mikhail Pershin
• Assignee: John Hammond
• Resolution: Fixed
• Severity: 3

Description

this test is failing time to time in Maloo:

sanity-hsm test_24a: @@@@@@ FAIL: atime 1512175765 is not less than mtime 1512175765 

https://testing.hpdd.intel.com/sub_tests/71f642c0-d7a6-11e7-8027-52540065bddc
https://testing.hpdd.intel.com/sub_tests/99484bf4-d7e0-11e7-9840-52540065bddc
https://testing.hpdd.intel.com/sub_tests/cfab05ee-d5f4-11e7-9840-52540065bddc

Info required for matching: sanity-hsm 24a

Issue Links

• is related to: LU-8950 tests: sanity-hsm: use make_custom_fi...

Comments

[John Hammond]

First seen on 2017-11-29 14:53:15 when testing in a full session on
v2_10_55_0-70-g35531ce. This is almost certainly due to patch https://review.whamcloud.com/24704 "LU-8950 tests: standardize file creation in sanity-hsm". Files in the /mnt/lustre are now being created on the CT node (via do_facet $SINGLEAGT dd ...) whereas before they were created on the first client node (via dd ...). This breaks the timing logic in sanity-hsm test_24a() and possibly others.

[Gerrit Updater]

John L. Hammond ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30433
Subject: LU-10344 test: create files on local node in sanity-hsm
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 5aaad227a1030228827d97795f19a60654a7c8c5

[Quentin Bouget]

What if there is no client locally availalbe?
create_file() was implemented so that it would be easy to create files on remote clients (not just the local one), if the interest of not having any local client is arguable, being able to easily create files on remote client is definitely valuable.

I believe the issue lies in the test itself. One shouldn't use:

sleep 1
echo >> file

to change the atime/ctime/mtime of a file to begin with. touch provides options just to do that, without the unreliable delay.

[John Hammond]

> What if there is no client locally availalbe?

Every sanity* test script assumes that it is running on a Lustre client.

[Bob Glossman (Inactive)]

another on master:
https://testing.hpdd.intel.com/test_sets/fc1f2ab2-df07-11e7-9c63-52540065bddc

[Gerrit Updater]

Quentin Bouget ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30496
Subject: LU-10344 tests: fix atime in sanity-hsm/test_24a
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: b8a884c642802b0f15788defb5b230957809d6c2

[Jinshan Xiong (Inactive)]

another one: https://testing.hpdd.intel.com/test_sets/7f7214fe-df4b-11e7-9c63-52540065bddc

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30433/
Subject: LU-10344 test: create files on local node in sanity-hsm
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 11aae875be5fb30a88281ab47e491b75447b299b

[Gerrit Updater]

Minh Diep ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31421
Subject: LU-10344 test: create files on local node in sanity-hsm
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: 77cd481a16f266e0b49793315c20b12d07711a3a

[Minh Diep]

reopen for LTS port

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34365
Subject: LU-10344 test: re-enable sanity test 103a for Ubuntu
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4e7396e327ceea2b7cdd43fcf1ec353b7d785e9b

[James A Simmons]

Typo with last patch. Sorry.

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: https://review.whamcloud.com/34819
Subject: LU-10344 test: re-enable sanity test 103a for Ubuntu
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: bf2f093f3a4892467781da0494d4d69256cdc33f