diff mbox

[OSSTEST] ts-xtf-run: Understand ./xtf-runner returning CRASH

Message ID 1488900412-13034-1-git-send-email-andrew.cooper3@citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andrew Cooper March 7, 2017, 3:26 p.m. UTC
./xtf-runner wants to distinguish between a clean and unclean exits of the
test.  OSSTest doesn't care, so map unclean exit to failure.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Ian Jackson <Ian.Jackson@eu.citrix.com>
CC: Wei Liu <wei.liu2@citrix.com>
---
 ts-xtf-run | 1 +
 1 file changed, 1 insertion(+)

Comments

Ian Jackson March 7, 2017, 3:38 p.m. UTC | #1
Andrew Cooper writes ("[OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH"):
> ./xtf-runner wants to distinguish between a clean and unclean exits of the
> test.  OSSTest doesn't care, so map unclean exit to failure.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Wei Liu March 13, 2017, 9:23 a.m. UTC | #2
On Tue, Mar 07, 2017 at 03:26:52PM +0000, Andrew Cooper wrote:
> ./xtf-runner wants to distinguish between a clean and unclean exits of the
> test.  OSSTest doesn't care, so map unclean exit to failure.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>

Reviewed-by: Wei Liu <wei.liu2@citrix.com>
Andrew Cooper April 6, 2017, 6:20 p.m. UTC | #3
On 07/03/17 15:38, Ian Jackson wrote:
> Andrew Cooper writes ("[OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH"):
>> ./xtf-runner wants to distinguish between a clean and unclean exits of the
>> test.  OSSTest doesn't care, so map unclean exit to failure.
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Ping on this getting accepted?  I have some XTF improvements queued
behind not breaking OSSTest.
Ian Jackson April 7, 2017, 9:40 a.m. UTC | #4
Andrew Cooper writes ("Re: [OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH"):
> Ping on this getting accepted?  I have some XTF improvements queued
> behind not breaking OSSTest.

Sorry for dropping this.  I have queued it up and will push it with
the TLS MITM series.

Ian.
diff mbox

Patch

diff --git a/ts-xtf-run b/ts-xtf-run
index d405bfb..c95ec5f 100755
--- a/ts-xtf-run
+++ b/ts-xtf-run
@@ -39,6 +39,7 @@  sub xtf_result_to_osstest_result ($) {
     return "skip" if $xret == 3; # XTF SKIP
     return "fail" if $xret == 4; # XTF ERROR
     return "fail" if $xret == 5; # XTF FAILURE
+    return "fail" if $xret == 6; # XTF CRASH
     die "xtf runner gave unexpected result $xret";
 }