Francois Gouget : testbot: Requeue the task if the VM is not ready for it.

Alexandre Julliard julliard at winehq.org
Mon Sep 23 14:35:46 CDT 2019


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Sep 23 16:59:31 2019 +0200

testbot: Requeue the task if the VM is not ready for it.

In order to run a task a VM should be in the 'running' state and powered
on. If either check fails that's not the fault of the task and a second
attempt may succeed. So don't end the task with a boterror. Instead
requeue it and put the VM offline so its true state is evaluated (and
notify the administrator of course).
This implies only resetting the VM error count after successfully
running a task otherwise we could get into an infinite loop putting it
offline at the start of every task.

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

---

 testbot/bin/LibvirtTool.pl               |  1 -
 testbot/bin/WineRunBuild.pl              | 32 ++++++++++++++++++++++---
 testbot/bin/WineRunReconfig.pl           | 37 ++++++++++++++++++++++++----
 testbot/bin/WineRunTask.pl               | 37 ++++++++++++++++++++++++----
 testbot/bin/WineRunWineTest.pl           | 41 +++++++++++++++++++++++++++-----
 testbot/lib/WineTestBot/LibvirtDomain.pm | 13 ++++++++++
 6 files changed, 143 insertions(+), 18 deletions(-)

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



More information about the wine-cvs mailing list