diff mbox series

[yocto-autobuilder-helper,2/2] scripts/yocto-supported-distros: ignore missing on AB for stable branches

Message ID 20250305-yocto-supported-distros-fixes-v1-2-308df26dd6d9@bootlin.com
State New
Headers show
Series Fixes for yocto-supported-distros | expand

Commit Message

Antonin Godard March 5, 2025, 10:03 a.m. UTC 
Add a stable_release boolean argument to _compare(). If True just
ignore missing workers on the AB, because we want to keep the full list
of tested releases in poky.conf for stable branches.

Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
---
 scripts/yocto-supported-distros | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/scripts/yocto-supported-distros b/scripts/yocto-supported-distros
index 533e3d48bddee9c685d4e94add14c66d2406621c..112d967149e3542462db3fe8ccd3d7a66dacca57 100755
--- a/scripts/yocto-supported-distros
+++ b/scripts/yocto-supported-distros
@@ -199,7 +199,7 @@  def _mangle_worker(worker: str) -> str:
     return ""
 
 
-def _compare(ab_workers: set, poky_workers: set):
+def _compare(ab_workers: set, poky_workers: set, stable_release: bool):
     ok = True
 
     print("Configured on the autobuilder:")
@@ -218,9 +218,13 @@  def _compare(ab_workers: set, poky_workers: set):
 
     ab_missing = poky_workers.difference(sorted(list(ab_workers)))
     if ab_missing:
-        _print_worker_list_warning(sorted(list(ab_missing)), "Missing on the autobuilder but listed in poky.conf")
+        if stable_release:
+            print("Missing entries on the autobuilder while listed in poky.conf, "
+                  "but comparing for a stable release so ignoring")
+        else:
+            _print_worker_list_warning(sorted(list(ab_missing)), "Missing on the autobuilder but listed in poky.conf")
+            ok = False
         print()
-        ok = False
 
     return ok
 
@@ -256,6 +260,8 @@  def main():
 
     possible_workers = {}
 
+    stable_release = True
+
     for release in releases:
 
         if release != "master" and release not in config.workers_prev_releases:
@@ -267,6 +273,7 @@  def main():
                 exit(1)
 
         if release == "master":
+            stable_release = False
             possible_workers.update({release: config.all_workers})
             continue
 
@@ -291,7 +298,7 @@  def main():
             if mangled_w:
                 ab_workers.add(mangled_w)
 
-        if not _compare(ab_workers, poky_workers):
+        if not _compare(ab_workers, poky_workers, stable_release):
             print("Mismatches were found")
         else:
             print("All good!")