[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.