Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-03 Thread Pavel Dovgalyuk
On 03.12.2020 19:56, Cleber Rosa wrote: On Thu, Dec 03, 2020 at 09:29:10AM +0300, Pavel Dovgalyuk wrote: On 02.12.2020 18:22, John Snow wrote: On 12/2/20 1:31 AM, Pavel Dovgalyuk wrote: This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tes

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-03 Thread Cleber Rosa
On Thu, Dec 03, 2020 at 09:29:10AM +0300, Pavel Dovgalyuk wrote: > On 02.12.2020 18:22, John Snow wrote: > > On 12/2/20 1:31 AM, Pavel Dovgalyuk wrote: > > > > > > > > > > > > This patch adds timeout parameter to vm.wait() calls, > > > > > > because the default > > > > > > value is just 30 seconds

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-02 Thread Pavel Dovgalyuk
On 02.12.2020 18:22, John Snow wrote: On 12/2/20 1:31 AM, Pavel Dovgalyuk wrote: This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tests may last for more time. This doesn't sound right -- the timeout isn't meant to be for the entire d

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-02 Thread John Snow
On 12/2/20 1:31 AM, Pavel Dovgalyuk wrote: This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tests may last for more time. This doesn't sound right -- the timeout isn't meant to be for the entire duration of the test, the timeout is fro

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-01 Thread Pavel Dovgalyuk
On 01.12.2020 22:15, John Snow wrote: On 11/16/20 6:13 AM, Philippe Mathieu-Daudé wrote: Cc'ing John. On Mon, Nov 16, 2020 at 11:08 AM Pavel Dovgalyuk wrote: This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tests may last for more time

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-12-01 Thread John Snow
On 11/16/20 6:13 AM, Philippe Mathieu-Daudé wrote: Cc'ing John. On Mon, Nov 16, 2020 at 11:08 AM Pavel Dovgalyuk wrote: This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tests may last for more time. This doesn't sound right -- the time

Re: [PATCH] tests/acceptance: fix timeout for vm.wait

2020-11-16 Thread Philippe Mathieu-Daudé
Cc'ing John. On Mon, Nov 16, 2020 at 11:08 AM Pavel Dovgalyuk wrote: > > This patch adds timeout parameter to vm.wait() calls, because the default > value is just 30 seconds, and tests may last for more time. > > Signed-off-by: Pavel Dovgalyuk > --- > tests/acceptance/boot_linux_console.py |

[PATCH] tests/acceptance: fix timeout for vm.wait

2020-11-16 Thread Pavel Dovgalyuk
This patch adds timeout parameter to vm.wait() calls, because the default value is just 30 seconds, and tests may last for more time. Signed-off-by: Pavel Dovgalyuk --- tests/acceptance/boot_linux_console.py |8 tests/acceptance/replay_kernel.py |2 +- 2 files changed, 5 in