Module: tools
Branch: master
Commit: 428d39c238140d5bca3fa7a9d65c6c09a8c4163c
URL: http://source.winehq.org/git/tools.git/?a=commit;h=428d39c238140d5bca3fa7a9…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Mar 12 14:52:42 2014 +0100
testbot/TestAgent: Track the current RPC and argument name and include them in error messages.
Also keep track of how many bytes were written/read before the error
occurred. This helps diagnose errors. In particular, should a
'network read timeout' occur, it helps knowing where it happened.
---
testbot/lib/WineTestBot/TestAgent.pm | 360 ++++++++++++++++++----------------
1 file changed, 189 insertions(+), 171 deletions(-)
Diff: http://source.winehq.org/git/tools.git/?a=commitdiff;h=428d39c238140d5bca3f…
Module: tools
Branch: master
Commit: a116dac155fd6a6c24684203c20174fd30f9a844
URL: http://source.winehq.org/git/tools.git/?a=commit;h=a116dac155fd6a6c24684203…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Mar 12 14:45:54 2014 +0100
testbot/TestAgent: Better handle connection timeouts.
The timeout can and does sometimes happen when connecting or
authenticating to SSH or starting netcat, not just when creating the
initial socket connection. So apply the timeout and connection
retries to the whole process. This means RevertVM no longer needs its
own attempt loop and the other scripts should no longer fail to
connect where RevertVM succeeded. Also let clients specify how many
attempts to make in case they don't like the default of 3.
---
testbot/bin/RevertVM.pl | 9 +-
testbot/lib/WineTestBot/TestAgent.pm | 239 +++++++++++++++++++---------------
2 files changed, 134 insertions(+), 114 deletions(-)
Diff: http://source.winehq.org/git/tools.git/?a=commitdiff;h=a116dac155fd6a6c2468…
Module: tools
Branch: master
Commit: 8fd1a3be1d792132a397144b03b7b96472afc7de
URL: http://source.winehq.org/git/tools.git/?a=commit;h=8fd1a3be1d792132a397144b…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Mar 12 14:25:58 2014 +0100
testbot/TestAgent: We don't need the server version for SetTime().
Just let the server return an error if it does not know about this RPC.
---
testbot/lib/WineTestBot/TestAgent.pm | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/testbot/lib/WineTestBot/TestAgent.pm b/testbot/lib/WineTestBot/TestAgent.pm
index c138663..f810f2b 100644
--- a/testbot/lib/WineTestBot/TestAgent.pm
+++ b/testbot/lib/WineTestBot/TestAgent.pm
@@ -2,7 +2,7 @@
# to run scripts.
#
# Copyright 2009 Ge van Geldorp
-# Copyright 2012 Francois Gouget
+# Copyright 2012-2014 Francois Gouget
#
# This library is free software; you can redistribute it and/or
# modify it under the terms of the GNU Lesser General Public
@@ -1206,9 +1206,6 @@ sub SetTime($)
my ($self) = @_;
debug("SetTime\n");
- # Make sure we have the server version
- return undef if (!$self->{agentversion} and !$self->_Connect());
-
# Send the command
if (!$self->_StartRPC($RPC_SETTIME) or
!$self->_SendListSize(2) or
Module: tools
Branch: master
Commit: 6157405f4a6dd1e45f3ca6aabc78c8750e6e75d2
URL: http://source.winehq.org/git/tools.git/?a=commit;h=6157405f4a6dd1e45f3ca6aa…
Author: Francois Gouget <fgouget(a)codeweavers.com>
Date: Wed Mar 12 14:25:29 2014 +0100
testbot/TestAgent: _SetError() may be called when connecting, that is outside any RPC.
---
testbot/lib/WineTestBot/TestAgent.pm | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/testbot/lib/WineTestBot/TestAgent.pm b/testbot/lib/WineTestBot/TestAgent.pm
index f313984..c138663 100644
--- a/testbot/lib/WineTestBot/TestAgent.pm
+++ b/testbot/lib/WineTestBot/TestAgent.pm
@@ -212,7 +212,8 @@ sub _SetError($$$)
# We did not even manage to connect but record the error anyway
$self->{err} = $Msg;
}
- debug($RpcNames{$self->{rpcid}} || $self->{rpcid}, ": $self->{err}\n");
+ my $RpcName = defined $self->{rpcid} ? $RpcNames{$self->{rpcid}} || $self->{rpcid} : "Connect";
+ debug("$RpcName: $self->{err}\n");
}
sub GetLastError($)