[LU-6206] conf-sanity test_53a: Insane OST thread counts

• Type: Bug
• Resolution: Incomplete
• Priority: Minor
• Severity: 3
• Assignee: Andreas Dilger
• Reporter: Maloo

Description

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

This issue relates to the following test suite run: https://testing.hpdd.intel.com/test_sets/29448bbe-ac56-11e4-b832-5254006e85c2.

The sub-test test_53a failed with the following error:

$'Assertion 27 failed: (($tmax2 < $tmin)) (expanded: ((4 < 3)))
Insane OST thread counts'

Suspect this may be due to the very recent landing of LU-1214 ptlrpc: start minimum service threads.
It made changes in the close neighborhood of the failed tests.

Info required for matching: conf-sanity 53a

Issue Links

• is related to: LU-6563 conf-sanity test_53b: FAIL: Assertion 25 failed: (($tstarted >= $tmin2)) (expanded: ((7 >= 8)))
• is related to: LU-1214 PTLRPC related modules cleanup
• is related to: LU-1214 PTLRPC related modules cleanup

Comments

[Andreas Dilger]

This test is no longer failing. Will leave cleanups to some future date.

[Gerrit Updater]

Andreas Dilger ([e-mail]) merged in patch http://review.whamcloud.com/14682/
Subject: Revert "LU-6206 ptlrpc: start minimum service threads"
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e15535ff63bf90b93fa33393ed5d7f7f85813895

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: http://review.whamcloud.com/14682
Subject: Revert "LU-6206 ptlrpc: start minimum service threads"
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 241fd8ca40271665c399d79ded71d0a8d28a247a

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/13823/
Subject: LU-6206 ptlrpc: start minimum service threads
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: cafa669a43062c5097d40803b9ba14e66edbae25

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: http://review.whamcloud.com/13823
Subject: LU-6206 ptlrpc: start minimum service threads
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 34ec2c93de940ecca8dcf2aa6c760cf8c1b133cc

[Andreas Dilger]

PS: we are also investigating how/why Maloo marked this test failure with Verified +1 when it clearly failed the review-dne-part-1 test.

We know that review-zfs is currently optional so the presence of a failure in that test wouldn't itself cause Maloo to mark the overall result -1, but the other failure should have. It should be noted that (excluding the hit from this issue) we have resolved a major blocker for ZFS testing (LU-5242) and are expecting to change review-zfs test results to be enforced in the near future now that we expect it to pass regularly.

[Andreas Dilger]

Thanks James.

[James A Simmons]

Yes Bob it was that one line in the test script that was breaking everything. I will push a new version of LU-1214 with that fix. The lesson to learn here is never trust Maloo claiming to pass but to always look at the logs.

[Andreas Dilger]

Patches that are currently based on a tree with that patch in it should rebase to after the reverted patch. We are also looking to disable these tests temporarily so that any patches that haven't hit this failure yet can pass.

[Andreas Dilger]

We've reverted the original patch from master.