CI: Remove KFAIL from Raspberry

Message ID 20250218065703.29629-1-ubely@ilbers.de
State Accepted, archived
Headers show
Series CI: Remove KFAIL from Raspberry | expand

Commit Message

Uladzimir Bely Feb. 18, 2025, 6:56 a.m. UTC
RPi related build issues like one fixed in 55e8b2aa have not been
reproduced for a long time, so the targets look stable enough to
remove CI protection from the tests.

Signed-off-by: Uladzimir Bely <ubely@ilbers.de>
---
 testsuite/citest.py | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

Comments

Uladzimir Bely Feb. 27, 2025, 7:50 a.m. UTC | #1
On Tue, 2025-02-18 at 09:56 +0300, Uladzimir Bely wrote:
> RPi related build issues like one fixed in 55e8b2aa have not been
> reproduced for a long time, so the targets look stable enough to
> remove CI protection from the tests.
> 
> Signed-off-by: Uladzimir Bely <ubely@ilbers.de>
> ---
>  testsuite/citest.py | 15 +++------------
>  1 file changed, 3 insertions(+), 12 deletions(-)
> 
> diff --git a/testsuite/citest.py b/testsuite/citest.py
> index ee965278..47f49263 100755
> --- a/testsuite/citest.py
> +++ b/testsuite/citest.py
> @@ -175,10 +175,7 @@ class CrossTest(CIBaseTest):
>          ]
>  
>          self.init()
> -        try:
> -            self.perform_build_test(targets)
> -        except exceptions.TestFail:
> -            self.cancel('KFAIL')
> +        self.perform_build_test(targets)
>  
>  
>  class WicTest(CIBaseTest):
> @@ -271,10 +268,7 @@ class NoCrossTest(CIBaseTest):
>          ]
>  
>          self.init()
> -        try:
> -            self.perform_build_test(targets, cross=False)
> -        except exceptions.TestFail:
> -            self.cancel('KFAIL')
> +        self.perform_build_test(targets, cross=False)
>  
>      def test_nocross_rpi_debsrc(self):
>          targets = [
> @@ -284,10 +278,7 @@ class NoCrossTest(CIBaseTest):
>          ]
>  
>          self.init()
> -        try:
> -            self.perform_build_test(targets, cross=False,
> debsrc_cache=True)
> -        except exceptions.TestFail:
> -            self.cancel('KFAIL')
> +        self.perform_build_test(targets, cross=False,
> debsrc_cache=True)
>  
>      def test_nocross_trixie(self):
>          targets = [
> -- 
> 2.45.3
> 

Applied to next.

Patch

diff --git a/testsuite/citest.py b/testsuite/citest.py
index ee965278..47f49263 100755
--- a/testsuite/citest.py
+++ b/testsuite/citest.py
@@ -175,10 +175,7 @@  class CrossTest(CIBaseTest):
         ]
 
         self.init()
-        try:
-            self.perform_build_test(targets)
-        except exceptions.TestFail:
-            self.cancel('KFAIL')
+        self.perform_build_test(targets)
 
 
 class WicTest(CIBaseTest):
@@ -271,10 +268,7 @@  class NoCrossTest(CIBaseTest):
         ]
 
         self.init()
-        try:
-            self.perform_build_test(targets, cross=False)
-        except exceptions.TestFail:
-            self.cancel('KFAIL')
+        self.perform_build_test(targets, cross=False)
 
     def test_nocross_rpi_debsrc(self):
         targets = [
@@ -284,10 +278,7 @@  class NoCrossTest(CIBaseTest):
         ]
 
         self.init()
-        try:
-            self.perform_build_test(targets, cross=False, debsrc_cache=True)
-        except exceptions.TestFail:
-            self.cancel('KFAIL')
+        self.perform_build_test(targets, cross=False, debsrc_cache=True)
 
     def test_nocross_trixie(self):
         targets = [