diff mbox series

[yocto-autobuilder2] observer: Fix failed ptest links

Message ID 20251205-mathieu-ptest-link-fixes-v1-1-1fc026d5043c@bootlin.com
State New
Headers show
Series [yocto-autobuilder2] observer: Fix failed ptest links | expand

Commit Message

Mathieu Dubois-Briand Dec. 5, 2025, 9:50 a.m. UTC
The new log observer added a few weeks ago fails to correctly show links
to failed ptest logs. It looks like some unexpected newline character is
messing with the parsing.

Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
---
 steps/observer.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


---
base-commit: e8c4db23afb334de278f3c5c7800c33fe32f7438
change-id: 20251205-mathieu-ptest-link-fixes-5fb6fe1c4a1e

Best regards,
diff mbox series

Patch

diff --git a/steps/observer.py b/steps/observer.py
index a89f0c5b1962..7231d29c33ea 100644
--- a/steps/observer.py
+++ b/steps/observer.py
@@ -60,7 +60,7 @@  class SimpleLogObserver(ShellCommand):
                 self.errorLines.append(logname + ": " + line)
             if line.startswith("Builder:"):
                 self.builder = line.split(':')[1].strip()
-            url, matched = self.yocto_io_re.subn('https://\\1', line)
+            url, matched = self.yocto_io_re.subn('https://\\1', line.strip())
             if matched:
                 self.links.append(url)
                 if not self.testResultsLink and url.endswith("/testresults"):