[zorg] ClangTestCommand lets pass Timeout failure as succeeded

[llvmbot]

Bugzilla Link	11317
Version	unspecified
OS	All
Reporter	LLVM Bugzilla Contributor

Extended Description

Example:
http://lab.llvm.org:8011/builders/clang-x86_64-ubuntu/builds/166/steps/test-clang
http://bb.pgr.jp/builders/cmake-clang-x86_64-linux/builds/5482/steps/test_clang

[llvmbot]

I apologize my little right finger to press ENTER instead of ']'.

As you can see in the logs above, zorg.buildbot.commands.ClangTestCommand() overrides failure status as buildbot.status.builder.SUCCESS.

--
(snip)
PASS: Clang :: ASTMerge/typedef.c (46 of 3995)

command timed out: 1200 seconds without output, attempting to kill
process killed by signal 9
program finished with exit code -1
elapsedTime=1205.342159

[llvmbot]

LitTestCommand, instead of ClangTestCommand, works as my expected.

Daniel, would you like to mark this as WONTFIX and deprecate ClangTestCommand?

FYI, AFAIK, no one uses LitTestCommand but most of builders use ClangTestCommand.

[llvmbot]

Yes, everything should move to use LitTestCommand which is better.

[arsenm]

Comment says mark as WONTFIX

[llvmbot]

@llvm/issue-subscribers-infrastructure