[LU-10217] Sanity 248 fails with newer dd output

• Status: Resolved
• Project: Lustre
• Affects Version/s: Upstream, Lustre 2.10.1, Lustre 2.11.0
• Fix Version/s: Lustre 2.11.0, Lustre 2.10.3
• Type: Bug
• Priority: Minor
• Reporter: Nikolay Borisov
• Assignee: Bob Glossman (Inactive)
• Resolution: Fixed

Environment

SlE12SP2/SP3

• Severity: 3

Description

Running the sanity test suite on SLES12SP2/SP3 fails with the following error:

Test 1: verify that fast read is 4 times faster on cache read
(standard_in) 1: illegal character: M
(standard_in) 1: syntax error
(standard_in) 1: illegal character: M
./sanity.sh: line 14321: [: -eq: unary operator expected
 sanity test_248: @@@@@@ FAIL: fast read was not 4 times faster:  MiB) vs MiB) 
  Trace dump:
  = /usr/lib64/lustre/tests/test-framework.sh:5293:error()
  = /usr/lib64/lustre/tests/test-framework.sh:5326:error_not_in_vm()
  = ./sanity.sh:14322:test_248()
  = /usr/lib64/lustre/tests/test-framework.sh:5569:run_one()
  = /usr/lib64/lustre/tests/test-framework.sh:5608:run_one_logged()
  = /usr/lib64/lustre/tests/test-framework.sh:5407:run_test()
  = ./sanity.sh:14345:main()

This is due to changed output dd has in recent versions:

32768+0 records in
32768+0 records out
134217728 bytes (134 MB, 128 MiB) copied, 3,85718 s, 34,8 MB/s

Essentially there are more fields being added with the inclusion of base 2 value. Further testing showed that sometimes dd can print the quantity of elapsed time in scientific notation, which is not understood by bc:

4096 bytes (4,1 kB, 4,0 KiB) copied, 9,1229e-05 s, 44,9 MB/s 

Comments

[Gerrit Updater]

Nikolay Borisov ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30005
Subject: LU-10217 test: Make sanity 248 work with newer dds
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: da683713aad84d6ca5c5d2be3cad3b7ea90731c3

[Gerrit Updater]

Nikolay Borisov ([e-mail]) uploaded a new patch: https://review.whamcloud.com/30144
Subject: LU-10217 test: Make sanity 248 work with newer dds
Project: fs/lustre-release
Branch: b2_10
Current Patch Set: 1
Commit: dca54901f3b799c04a9a225213c3ac1f324a436f

[Nikolay Borisov]

The patch has +1 from jenkins/maloo and has been code reviewed is there anything else that's impeding it's merge?

[Peter Jones]

It needs a second review - Bob, can you do the honours?

[Bob Glossman (Inactive)]

The error report is specific to SLES12sp2/3, but I see only test results on RHEL.
Would really prefer to see some +test on SLES before giving a +review.

[Peter Jones]

Bob

Good idea. I daresay that Nikolay is unfamiliar with test parameters. Can you refresh the patch to add them?

Thanks

Peter

[Bob Glossman (Inactive)]

Peter,
Yes, I can refresh the patch. It's just a commit header edit.

Bob

[Bob Glossman (Inactive)]

changes passed test on SLES12SP2 and SLES12SP3.
gave it +review now.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/30005/
Subject: LU-10217 test: Make sanity 248 work with newer dds
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 22de28dc7c1247f2e7c557efa1ef8e6263dd7221

[Peter Jones]

Landed for 2.11

[Gerrit Updater]

John L. Hammond ([e-mail]) merged in patch https://review.whamcloud.com/30144/
Subject: LU-10217 test: Make sanity 248 work with newer dds
Project: fs/lustre-release
Branch: b2_10
Current Patch Set:
Commit: 0e8d207cfebc14d7263a8e13dca27b8c69ad2bb5