diff mbox series

[RFC] tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot

Message ID 20240801172332.65701-1-philmd@linaro.org
State Superseded
Headers show
Series [RFC] tests/avocado: Re-enable gdbsim-r5f562n8 testing U-Boot | expand

Commit Message

Philippe Mathieu-Daudé Aug. 1, 2024, 5:23 p.m. UTC
We disabled all RX tests on commit 9b45cc9931 ("docs/devel:
rationalise unstable gitlab tests under FLAKY_TESTS") for
being flaky. However I don't recall the U-Boot test to fail
(the problematic line checking the 'version' string is already
commented out), and I'm running this test reliably, so re-enable
it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 tests/avocado/machine_rx_gdbsim.py | 2 --
 1 file changed, 2 deletions(-)

Comments

Yoshinori Sato Aug. 5, 2024, 6:54 a.m. UTC | #1
On Fri, 02 Aug 2024 02:23:32 +0900,
Philippe Mathieu-Daudé wrote:
> 
> We disabled all RX tests on commit 9b45cc9931 ("docs/devel:
> rationalise unstable gitlab tests under FLAKY_TESTS") for
> being flaky. However I don't recall the U-Boot test to fail
> (the problematic line checking the 'version' string is already
> commented out), and I'm running this test reliably, so re-enable
> it.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  tests/avocado/machine_rx_gdbsim.py | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tests/avocado/machine_rx_gdbsim.py b/tests/avocado/machine_rx_gdbsim.py
> index 412a7a5089..a83873b738 100644
> --- a/tests/avocado/machine_rx_gdbsim.py
> +++ b/tests/avocado/machine_rx_gdbsim.py
> @@ -22,8 +22,6 @@ class RxGdbSimMachine(QemuSystemTest):
>      timeout = 30
>      KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
>  
> -    @skipUnless(os.getenv('QEMU_TEST_FLAKY_TESTS'), 'Test is unstable on GitLab')
> -
>      def test_uboot(self):
>          """
>          U-Boot and checks that the console is operational.
> -- 
> 2.45.2
> 

Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>
Alex Bennée Aug. 5, 2024, 12:39 p.m. UTC | #2
Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> We disabled all RX tests on commit 9b45cc9931 ("docs/devel:
> rationalise unstable gitlab tests under FLAKY_TESTS") for
> being flaky. However I don't recall the U-Boot test to fail
> (the problematic line checking the 'version' string is already
> commented out), and I'm running this test reliably, so re-enable
> it.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Queued to for-9.1/misc-fixes, thanks.
Philippe Mathieu-Daudé Aug. 5, 2024, 12:41 p.m. UTC | #3
On 5/8/24 08:54, Yoshinori Sato wrote:
> On Fri, 02 Aug 2024 02:23:32 +0900,
> Philippe Mathieu-Daudé wrote:
>>
>> We disabled all RX tests on commit 9b45cc9931 ("docs/devel:
>> rationalise unstable gitlab tests under FLAKY_TESTS") for
>> being flaky. However I don't recall the U-Boot test to fail
>> (the problematic line checking the 'version' string is already
>> commented out), and I'm running this test reliably, so re-enable
>> it.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> ---
>>   tests/avocado/machine_rx_gdbsim.py | 2 --
>>   1 file changed, 2 deletions(-)

> Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>

Thanks, I'll queue this patch for 9.2 to avoid messing
with the next release.

Regards,

Phil.
Alex Bennée Aug. 5, 2024, 12:56 p.m. UTC | #4
Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> On 5/8/24 08:54, Yoshinori Sato wrote:
>> On Fri, 02 Aug 2024 02:23:32 +0900,
>> Philippe Mathieu-Daudé wrote:
>>>
>>> We disabled all RX tests on commit 9b45cc9931 ("docs/devel:
>>> rationalise unstable gitlab tests under FLAKY_TESTS") for
>>> being flaky. However I don't recall the U-Boot test to fail
>>> (the problematic line checking the 'version' string is already
>>> commented out), and I'm running this test reliably, so re-enable
>>> it.
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>>> ---
>>>   tests/avocado/machine_rx_gdbsim.py | 2 --
>>>   1 file changed, 2 deletions(-)
>
>> Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>
>
> Thanks, I'll queue this patch for 9.2 to avoid messing
> with the next release.

Testing updates are usually fine, but I can leave it to you to merge if
you prefer.

>
> Regards,
>
> Phil.
diff mbox series

Patch

diff --git a/tests/avocado/machine_rx_gdbsim.py b/tests/avocado/machine_rx_gdbsim.py
index 412a7a5089..a83873b738 100644
--- a/tests/avocado/machine_rx_gdbsim.py
+++ b/tests/avocado/machine_rx_gdbsim.py
@@ -22,8 +22,6 @@  class RxGdbSimMachine(QemuSystemTest):
     timeout = 30
     KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 '
 
-    @skipUnless(os.getenv('QEMU_TEST_FLAKY_TESTS'), 'Test is unstable on GitLab')
-
     def test_uboot(self):
         """
         U-Boot and checks that the console is operational.