Francois Gouget : testbot: Move reference logs to the task and prepare for having more than one.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: a6a86fd439e014b0ab80713bfdb2532d0fe8587d
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=a6a86fd439e014b0ab80713bfdb2532d0fe8587d

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Feb  4 03:50:07 2020 +0100

testbot: Move reference logs to the task and prepare for having more than one.

The old reference report filename format was '<vmname>_<reportname>'
which means there could only be one per report. Each job should only
have one report of a given type per VM so add it to the reference
filename.
Also the use of an underscore as the separator made it hard to split
the components apart as both <vmname> and <reportname> can contain
underscores such as 'w1064v1809_fr' and 'win32_fr_FR.report'. Dashes
are a better choice since they are not allowed in either the VM name
or the report name.
So the new refeence filename format is '<vmname>-job<jobid>-<reportname>'.

Note:
* After applying this patch run UpdateTaskLogs to move and rename the
  existing reference reports.
* When downgrading to run code before this patch it's best to run
  UpdateTaskLogs --delete before downgrading and then UpdateTaskLogs
  after the downgrade.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/bin/Janitor.pl                |   3 +-
 testbot/bin/UpdateTaskLogs            | 113 ++++++++++++++++++++++++++++++----
 testbot/bin/WineSendLog.pl            |   2 +-
 testbot/lib/WineTestBot/LogUtils.pm   |   5 +-
 testbot/lib/WineTestBot/StepsTasks.pm |   2 +-
 testbot/lib/WineTestBot/Tasks.pm      |   2 +-
 testbot/web/JobDetails.pl             |   4 +-
 7 files changed, 110 insertions(+), 21 deletions(-)

Diff:   https://source.winehq.org/git/tools.git/?a=commitdiff;h=a6a86fd439e014b0ab80713bfdb2532d0fe8587d