[LU-8248] sanity test_248: fast read was not 4 times faster

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.9.0
• Severity: 3
• Assignee: James Nunez (Inactive)
• Reporter: Maloo

Description

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

This issue relates to the following test suite run: https://testing.hpdd.intel.com/test_sets/28287b5e-2b40-11e6-acf3-5254006e85c2.

The sub-test test_248 failed with the following error:

Test 1: verify that fast read is 4 times faster on cache read
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.33 vs 1.15): 
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.29 vs 1.08):  
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.32 vs 1.08):  
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.26 vs 1.02):  
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.59 vs 0.98):  
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.28 vs 0.89): 
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.37 vs 1.30): 
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.39 vs 0.97): 
 sanity test_248: @@@@@@ IGNORE (fast read was not 4 times faster: 0.41 vs 1.45): 

There are also cases where the fast read was "too fast". There is also an error message that needs to be fixed.

error() without useful message, please fix

Info required for matching: sanity 248

Issue Links

• is related to: LU-4257 parallel dds are slower than serial dds

Comments

[Peter Jones]

Landed for 2.9

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/20698/
Subject: LU-8248 tests: fix sanity test_248 VM checks
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 39582273a7a52fb1118ddcca4d62769f43c6f5cb

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: http://review.whamcloud.com/20698
Subject: LU-8248 tests: fix sanity test_248 VM checks
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: d59093ce90ee5d46514514b9e5254f2d481ebd56

[James Nunez (Inactive)]

Patch is at http://review.whamcloud.com/#/c/20647/

If the patch needs to be refreshed, I will change the LU number in the commit message to reflect this ticket.

[Andreas Dilger]

James, you mentioned that you were going to work on a fix for this failure? It looks like the error_ignore is missing an LU ticket number as the first argument, which may as well be this ticket LU-8248.