[PATCH 1/4] testbot/WineSendLog: Use LogUtils to identify (new) errors.
Francois Gouget
fgouget at codeweavers.com
Sun Aug 26 03:17:16 CDT 2018
Added LogUtils::GetNewLogErrors() to identify new errors in test
reports and their associated extra errors files. Note that unlike
WineSendLog.pl's old CompareLogs() function GetNewLogErrors() can deal
with reports containing the errors for multiple modules, which is a
requirement for running the tests on Wine.
WineSendLog.pl now uses LogUtils instead of duplicating the code.
Besides reaping the benefits from the improved GetNewLogErrors() this
also improves error detection and ensures it is consistent with
JobDetails.pl.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
testbot/bin/WineSendLog.pl | 380 ++++++----------------------
testbot/lib/WineTestBot/LogUtils.pm | 106 +++++++-
2 files changed, 183 insertions(+), 303 deletions(-)
diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index 2127eadb7..f3b56d8c9 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -78,129 +78,47 @@ sub Error(@)
#
-# Log analysis
+# Reporting
#
-sub IsBotFailure($)
+sub GetTitle($$)
{
- my ($ErrLine) = @_;
+ my ($StepTask, $LogName) = @_;
- return ($ErrLine =~ m/Can't set VM status to running/ ||
- $ErrLine =~ m/Can't copy exe to VM/ ||
- $ErrLine =~ m/Can't copy log from VM/ ||
- $ErrLine =~ m/Can't copy generated executable from VM/);
-}
-
-sub CheckErrLog($)
-{
- my ($ErrLogFileName) = @_;
-
- my $BotFailure = !1;
- my $Messages = "";
- if (open ERRFILE, "$ErrLogFileName")
- {
- my $Line;
- while (defined($Line = <ERRFILE>))
- {
- if (IsBotFailure($Line))
- {
- if (! $Messages)
- {
- $BotFailure = 1;
- }
- }
- else
- {
- $Messages .= $Line;
- }
- }
- close ERRFILE;
- }
-
- return ($BotFailure, $Messages);
-}
-
-sub ReadLog($$$)
-{
- my ($LogName, $BaseName, $TestSet) = @_;
-
- my @Messages;
- if (open LOG, "<$LogName")
- {
- my $Line;
- my $Found = !1;
- while (! $Found && defined($Line = <LOG>))
- {
- $Found = ($Line =~ m/${BaseName}:${TestSet} start/);
- }
- if ($Found)
- {
- $Found = !1;
- while (! $Found && defined($Line = <LOG>))
- {
- $Line =~ s/[\r\n]*$//;
- if ($Line =~ m/${BaseName}:${TestSet}(?::[0-9a-f]+)? done/)
- {
- if ($Line =~ m/${BaseName}:${TestSet}(?::[0-9a-f]+)? done \(258\)/)
- {
- push @Messages, "The test timed out";
- }
- $Found = 1;
- }
- else
- {
- push @Messages, $Line;
- }
- }
- }
-
- close LOG;
- }
- else
+ my $Label = GetLogLabel($LogName);
+ if ($LogName !~ /\.report$/ and
+ ($StepTask->Type eq "build" or $StepTask->VM->Type eq "wine"))
{
- Error "Unable to open '$LogName' for reading: $!\n";
+ $Label = "build log";
}
- return \@Messages;
+ return $StepTask->VM->Name . " ($Label)";
}
-sub GetLineKey($)
+sub DumpLogAndErr($$)
{
- my ($Line) = @_;
-
- $Line =~ s/^([\w_.]+:)\d+(:.*)$/$1$2/;
-
- return $Line;
-}
-
-sub CompareLogs($$$$)
-{
- my ($SuiteLog, $TaskLog, $BaseName, $TestSet) = @_;
-
- my $Messages = "";
+ my ($File, $Path) = @_;
- my $SuiteMessages = ReadLog($SuiteLog, $BaseName, $TestSet);
- my $TaskMessages = ReadLog($TaskLog, $BaseName, $TestSet);
-
- my $Diff = Algorithm::Diff->new($SuiteMessages, $TaskMessages,
- { keyGen => \&GetLineKey });
- while ($Diff->Next())
+ my $PrintSeparator;
+ foreach my $FileName ($Path, "$Path.err")
{
- if (! $Diff->Same())
+ if (open(my $LogFile, "<", $FileName))
{
- foreach my $Line ($Diff->Items(2))
+ my $First = 1;
+ foreach my $Line (<$LogFile>)
{
- if ($Line =~ m/: Test failed: / ||
- $Line =~ m/: unhandled exception [0-9a-fA-F]{8} at / ||
- $Line =~ m/The test timed out/)
+ $Line =~ s/\s*$//;
+ if ($First and $PrintSeparator)
{
- $Messages .= "$Line\n";
+ print $File "\n";
+ $First = 0;
}
+ print $File "$Line\n";
}
+ close($LogFile);
+ $PrintSeparator = 1;
}
}
-
- return $Messages;
}
sub SendLog($)
@@ -279,110 +197,28 @@ EOF
print $Sendmail "\nYou can also see the results at:\n$JobURL\n\n";
# Print the job summary
- my @FailureKeys;
+ my ($JobErrors, $ReportCounts);
foreach my $Key (@SortedKeys)
{
my $StepTask = $StepsTasks->GetItem($Key);
my $TaskDir = $StepTask->GetTaskDir();
- print $Sendmail "\n=== ", $StepTask->GetTitle(), " ===\n";
-
- my $LogFiles = GetLogFileNames($TaskDir);
- my $LogName = $LogFiles->[0] || "log";
- if (open LOGFILE, "<$TaskDir/$LogName")
+ my $LogNames = GetLogFileNames($TaskDir);
+ $JobErrors->{$Key}->{LogNames} = $LogNames;
+ foreach my $LogName (@$LogNames)
{
- my $HasLogEntries = !1;
- my $PrintedSomething = !1;
- my $CurrentDll = "";
- my $PrintedDll = "";
- my $Line;
- while (defined($Line = <LOGFILE>))
- {
- $HasLogEntries = 1;
- $Line =~ s/\s*$//;
- if ($Line =~ m/^([^:]+):[^ ]+ start [^ ]+ -$/)
- {
- $CurrentDll = $1;
- }
- my $Category = $LogName =~ /\.report$/ ?
- GetReportLineCategory($Line) :
- GetLogLineCategory($Line);
- if ($Category eq "error")
- {
- if ($PrintedDll ne $CurrentDll)
- {
- print $Sendmail "\n$CurrentDll:\n";
- $PrintedDll = $CurrentDll;
- }
- if ($Line =~ m/^[^:]+:([^ ]+)(?::[0-9a-f]+)? done \(258\)/)
- {
- print $Sendmail "$1: The test timed out\n";
- }
- else
- {
- print $Sendmail "$Line\n";
- }
- $PrintedSomething = 1;
- }
- }
- close LOGFILE;
+ my ($Groups, $Errors) = GetLogErrors("$TaskDir/$LogName");
+ next if (!$Groups or !@$Groups);
+ $JobErrors->{$Key}->{HasErrors} = 1;
+ $JobErrors->{$Key}->{$LogName}->{Groups} = $Groups;
+ $JobErrors->{$Key}->{$LogName}->{Errors} = $Errors;
- if (open ERRFILE, "<$TaskDir/log.err")
- {
- my $First = 1;
- while (defined($Line = <ERRFILE>))
- {
- if ($First)
- {
- print $Sendmail "\n";
- $First = !1;
- }
- $HasLogEntries = 1;
- $Line =~ s/\s*$//;
- print $Sendmail "$Line\n";
- $PrintedSomething = 1;
- }
- close ERRFILE;
- }
+ print $Sendmail "\n=== ", GetTitle($StepTask, $LogName), " ===\n";
- if (! $PrintedSomething)
- {
- if (! $HasLogEntries)
- {
- print $Sendmail "Empty test log and no error message\n";
- }
- elsif ($StepTask->Type eq "build")
- {
- print $Sendmail "No build failures found\n";
- }
- else
- {
- print $Sendmail "No test failures found\n";
- }
- }
- else
+ foreach my $GroupName (@$Groups)
{
- push @FailureKeys, $Key;
- }
- }
- elsif (open ERRFILE, "<$TaskDir/log.err")
- {
- my $HasErrEntries = !1;
- my $Line;
- while (defined($Line = <ERRFILE>))
- {
- $HasErrEntries = 1;
- $Line =~ s/\s*$//;
- print $Sendmail "$Line\n";
- }
- close ERRFILE;
- if (! $HasErrEntries)
- {
- print $Sendmail "No test log and no error message";
- }
- else
- {
- push @FailureKeys, $Key;
+ print $Sendmail ($GroupName ? "\n$GroupName:\n" : "\n");
+ print $Sendmail "$_\n" for (@{$Errors->{$GroupName}});
}
}
}
@@ -393,46 +229,25 @@ EOF
my $StepTask = $StepsTasks->GetItem($Key);
my $TaskDir = $StepTask->GetTaskDir();
- print $Sendmail <<"EOF";
+ foreach my $LogName (@{$JobErrors->{$Key}->{LogNames}})
+ {
+ print $Sendmail <<"EOF";
--$PART_BOUNDARY
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
EOF
- print $Sendmail "Content-Disposition: attachment; filename=",
- $StepTask->VM->Name, ".log\n\n";
- print $Sendmail "Not dumping logs in debug mode\n" if ($Debug);
-
- my $LogFiles = GetLogFileNames($TaskDir);
- my $LogName = $LogFiles->[0] || "log";
- my $PrintSeparator = !1;
- if (open LOGFILE, "<$TaskDir/$LogName")
- {
- my $Line;
- while (defined($Line = <LOGFILE>))
+ print $Sendmail "Content-Disposition: attachment; filename=",
+ $StepTask->VM->Name, "_$LogName\n\n";
+ if ($Debug)
{
- $Line =~ s/\s*$//;
- print $Sendmail "$Line\n" if (!$Debug);
- $PrintSeparator = 1;
+ print $Sendmail "Not dumping logs in debug mode\n";
}
- close LOGFILE;
- }
-
- if (open ERRFILE, "<$TaskDir/log.err")
- {
- my $Line;
- while (defined($Line = <ERRFILE>))
+ else
{
- if ($PrintSeparator)
- {
- print $Sendmail "\n" if (!$Debug);
- $PrintSeparator = !1;
- }
- $Line =~ s/\s*$//;
- print $Sendmail "$Line\n" if (!$Debug);
+ DumpLogAndErr($Sendmail, "$TaskDir/$LogName");
}
- close ERRFILE;
}
}
@@ -446,75 +261,62 @@ EOF
return;
}
+
#
# Build a job summary with only the new errors
#
- my $Messages = "";
- foreach my $Key (@FailureKeys)
+ my @Messages;
+ foreach my $Key (@SortedKeys)
{
+ next if (!$JobErrors->{$Key}->{HasErrors});
my $StepTask = $StepsTasks->GetItem($Key);
my $TaskDir = $StepTask->GetTaskDir();
- my ($BotFailure, $MessagesFromErr) = CheckErrLog("$TaskDir/log.err");
- if ($BotFailure)
+ if ($StepTask->Status eq "boterror")
{
- # TestBot errors are not the developer's fault and prevent us from doing
- # any meaningful analysis. So skip.
- Error "A TestBot error was found in $TaskDir/log.err\n";
+ # TestBot errors are not the developer's fault and prevent us from
+ # identifying new errors. So skip.
+ Error "A Task error was found in '$TaskDir/log.err'\n";
next;
}
- my $MessagesFromLog = "";
- my $LogFiles = GetLogFileNames($TaskDir);
- my $LogName = $LogFiles->[0] || "log";
- if ($LogName =~ /\.report$/)
+ foreach my $LogName (@{$JobErrors->{$Key}->{LogNames}})
{
- my $LatestName = "$DataDir/latest/". $StepTask->VM->Name ."_$LogName";
- my ($LatestBotFailure, $Dummy) = CheckErrLog("$LatestName.err");
- if (! $LatestBotFailure)
+ my $LogErrors = $JobErrors->{$Key}->{$LogName};
+ my $RefFileName = "$DataDir/latest". $StepTask->VM->Name ."_$LogName";
+ my ($NewGroups, $NewErrors) = GetNewLogErrors($RefFileName, $LogErrors->{Groups}, $LogErrors->{Errors});
+ if (!$NewGroups)
{
- if (defined($StepTask->CmdLineArg))
- {
- # Filter out failures that happened in the full test suite:
- # the test suite is run against code which is already in Wine
- # so any failure it reported is not caused by this patch.
- $StepTask->FileName =~ m/^(.*)_test/;
- my $BaseName = $1;
- $MessagesFromLog = CompareLogs($LatestName, "$TaskDir/$LogName",
- $BaseName, $StepTask->CmdLineArg);
- }
+ # There was no reference log (typical of build logs)
+ # so every error is new
+ $NewGroups = $LogErrors->{Groups};
+ $NewErrors = $LogErrors->{Errors};
}
- else
+ elsif (!@$NewGroups)
{
- Error "BotFailure found in $LatestName.err\n";
+ # There is no new error
+ next;
}
- }
- elsif (open(my $LogFile, "<", "$TaskDir/$LogName"))
- {
- foreach my $Line (<$LogFile>)
+
+ push @Messages, "\n=== ". GetTitle($StepTask, $LogName) ." ===\n";
+
+ foreach my $GroupName (@$NewGroups)
{
- my $Category = $LogName =~ /\.report$/ ?
- GetReportLineCategory($Line) :
- GetLogLineCategory($Line);
- $MessagesFromLog .= $Line if ($Category eq "error");
+ push @Messages, ($GroupName ? "\n$GroupName:\n" : "\n");
+ push @Messages, "$_\n" for (@{$NewErrors->{$GroupName}});
}
- close($LogFile);
- }
- if ($MessagesFromErr || $MessagesFromLog)
- {
- $Messages .= "\n=== " . $StepTask->GetTitle() . " ===\n" .
- $MessagesFromLog . $MessagesFromErr;
}
}
+
#
# Send a summary of the new errors to the mailing list
#
Debug("\n-------------------- Mailing list email --------------------\n");
- if ($Messages)
+ if (@Messages)
{
if ($Debug)
{
@@ -548,7 +350,7 @@ Your paranoid android.
EOF
- print $Sendmail $Messages;
+ print $Sendmail $_ for (@Messages);
close($Sendmail);
}
else
@@ -556,6 +358,7 @@ EOF
Debug("Found no error to report to the mailing list\n");
}
+
#
# Create a .testbot file for the patches website
#
@@ -570,7 +373,7 @@ EOF
{
# Only take into account new errors to decide whether the job was
# successful or not.
- DebugTee($Result, "Status: ". ($Messages ? "Failed" : "OK") ."\n");
+ DebugTee($Result, "Status: ". (@Messages ? "Failed" : "OK") ."\n");
DebugTee($Result, "Job-ID: ". $Job->Id ."\n");
DebugTee($Result, "URL: $JobURL\n");
@@ -579,37 +382,10 @@ EOF
my $StepTask = $StepsTasks->GetItem($Key);
my $TaskDir = $StepTask->GetTaskDir();
- print $Result "\n=== ", $StepTask->GetTitle(), " ===\n";
-
- my $LogFiles = GetLogFileNames($TaskDir);
- my $LogName = $LogFiles->[0] || "log";
- my $PrintSeparator = !1;
- if (open(my $logfile, "<", "$TaskDir/$LogName"))
- {
- my $Line;
- while (defined($Line = <$logfile>))
- {
- $Line =~ s/\s*$//;
- print $Result "$Line\n";
- $PrintSeparator = 1;
- }
- close($logfile);
- }
-
- if (open(my $errfile, "<", "$TaskDir/log.err"))
+ foreach my $LogName (@{$JobErrors->{$Key}->{LogNames}})
{
- my $Line;
- while (defined($Line = <$errfile>))
- {
- if ($PrintSeparator)
- {
- print $Result "\n";
- $PrintSeparator = !1;
- }
- $Line =~ s/\s*$//;
- print $Result "$Line\n";
- }
- close($errfile);
+ print $Result "=== ", GetTitle($StepTask, $LogName), " ===\n";
+ DumpLogAndErr($Result, "$TaskDir/$LogName");
}
}
print $Result "--- END FULL_LOGS ---\n";
diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index a5a1a68b5..19a39aaf5 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -27,11 +27,12 @@ WineTestBot::LogUtils - Provides functions to parse task logs
use Exporter 'import';
-our @EXPORT = qw(GetLogFileNames GetLogLabel GetLogErrors
+our @EXPORT = qw(GetLogFileNames GetLogLabel GetLogErrors GetNewLogErrors
GetLogLineCategory GetReportLineCategory
RenameReferenceLogs RenameTaskLogs
ParseTaskLog ParseWineTestReport);
+use Algorithm::Diff;
use File::Basename;
use WineTestBot::Config; # For $MaxUnitSize
@@ -697,6 +698,8 @@ sub GetLogErrors($)
$Line =~ s/\s*$//;
if (!$CurrentGroup)
{
+ # Note: $GroupName must not depend on the previous content as this
+ # would break diffs.
my $GroupName = $IsReport ? "Report errors" : "Task errors";
$CurrentGroup = _AddErrorGroup($Groups, $Errors, $GroupName);
}
@@ -714,4 +717,105 @@ sub GetLogErrors($)
return $NoLog ? (undef, undef) : ($Groups, $Errors);
}
+sub _DumpDiff($$)
+{
+ my ($Label, $Diff) = @_;
+
+ print STDERR "$Label:\n";
+ $Diff = $Diff->Copy();
+ while ($Diff->Next())
+ {
+ if ($Diff->Same())
+ {
+ print STDERR " $_\n" for ($Diff->Same());
+ }
+ else
+ {
+ print STDERR "-$_\n" for ($Diff->Items(1));
+ print STDERR "+$_\n" for ($Diff->Items(2));
+ }
+ }
+}
+
+=pod
+=over 12
+
+=item C<_GetLineKey()>
+
+This is a helper for GetNewLogErrors(). It reformats the log lines so they can
+meaningfully be compared to the reference log even if line numbers change, etc.
+
+=back
+=cut
+
+sub _GetLineKey($)
+{
+ my ($Line) = @_;
+ return undef if (!defined $Line);
+
+ # Remove the line number
+ $Line =~ s/^([_a-z0-9]+\.c:)\d+:( Test (?:failed|succeeded inside todo block): )/$1$2/;
+
+ # Note: Only the 'done (258)' lines are reported as errors and they are
+ # modified by GetLogErrors() so that they no longer contain the pid.
+ # So there is no need to remove the pid from the done lines.
+
+ return $Line;
+}
+
+=pod
+=over 12
+
+=item C<GetNewLogErrors()>
+
+Compares the specified errors to the reference log and returns only the ones
+that are new.
+
+Returns a list of error groups containing new errors and a hashtable containing
+the list of new errors for each group.
+
+=back
+=cut
+
+sub GetNewLogErrors($$$)
+{
+ my ($RefFileName, $Groups, $Errors) = @_;
+
+ my ($_Dummy, $RefErrors, $NoLog) = GetLogErrors($RefFileName);
+ return (undef, undef) if ($NoLog);
+
+ my (@NewGroups, %NewErrors);
+ foreach my $GroupName (@$Groups)
+ {
+ if ($RefErrors->{$GroupName})
+ {
+ my $Diff = Algorithm::Diff->new($RefErrors->{$GroupName},
+ $Errors->{$GroupName},
+ { keyGen => \&_GetLineKey });
+ my $CurrentGroup;
+ while ($Diff->Next())
+ {
+ # Skip if there are no new lines
+ next if ($Diff->Same() or !$Diff->Items(2));
+
+ if (!$CurrentGroup)
+ {
+ push @NewGroups, $GroupName;
+ $CurrentGroup = $NewErrors{$GroupName} = [];
+ }
+ push @$CurrentGroup, $Diff->Items(2);
+ }
+ }
+ else
+ {
+ # This module did not have errors before, so every error is new
+ push @NewGroups, $GroupName;
+ $NewErrors{$GroupName} = $Errors->{$GroupName};
+ my $Last = @{$Errors->{$GroupName}} - 1;
+ }
+ }
+
+ return (\@NewGroups, \%NewErrors);
+}
+
1;
--
2.18.0
More information about the wine-devel
mailing list