Francois Gouget : testbot/WineRunReconfig: Don't run WineTest on extra Wine VMs.

[Alexandre Julliard]

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

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Tue Mar 10 12:36:49 2020 +0100

testbot/WineRunReconfig: Don't run WineTest on extra Wine VMs.

The role 'extra' means one can manually schedule a job on the VM but
that nothing gets automatically run on it, including WineTest.
This also means there will be no 'new failure' detection on these VMs.

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

---

 testbot/bin/WineRunReconfig.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/bin/WineRunReconfig.pl b/testbot/bin/WineRunReconfig.pl
index 644d546..2d45ed5 100755
--- a/testbot/bin/WineRunReconfig.pl
+++ b/testbot/bin/WineRunReconfig.pl
@@ -550,7 +550,7 @@ if ($NewStatus eq 'completed')
     FatalError("Could not recreate the $IdleSnapshot snapshot: $ErrMessage\n");
   }
 
-  if ($VM->Type eq "wine")
+  if ($VM->Type eq "wine" and $VM->Role =~ /^(?:base|winetest)$/)
   {
     require WineTestBot::SpecialJobs;
     $ErrMessage = WineTestBot::SpecialJobs::AddWineTestJob([$VM], $VM->Name);