diff mbox series

send-error-report: make output align with original design when debug disabled

Message ID 20250506105337.2875114-1-changqing.li@windriver.com
State New
Headers show
Series send-error-report: make output align with original design when debug disabled | expand

Commit Message

Changqing Li May 6, 2025, 10:53 a.m. UTC 
From: Changqing Li <changqing.li@windriver.com>

First, it is better to output the error report web link by default when
the report is uploaded successfully like before. This is useful, user
can find the detail the log info from the return link. Yocto Autobuilder
also benifit from this return link.

Second, why don't set level to logging.INFO to make the error report web
link outputed? Because "-j" option want to "Return the result in json
format, silences all other output", So "INFO:" added by logging system
is not wanted, so use print directly.

Example output:
Without "-j":
Preparing to send errors to: http://x.x.x.x:8000
Your entry can be found here: http://x.x.x.x:8000/Errors/Build/25/

With "-j":
{"build_id": 27, "build_url": "http://x.x.x.x:8000/Errors/Build/27/", "failures": [{"id": 26, "url": "http://x.x.x.x:8000/Errors/Details/26/"}], "num_similar_errors": 20, "similar_errors_url": "http://x.x.x.x:8000/Errors/SimilarTo/26/"}

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 scripts/send-error-report | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/scripts/send-error-report b/scripts/send-error-report
index 6418dab165..cc1bc7c2b1 100755
--- a/scripts/send-error-report
+++ b/scripts/send-error-report
@@ -155,7 +155,7 @@  def send_data(data, args):
 
     log.debug(f"Response Status: {response.status}")
     log.debug(f"Response Headers: {response.getheaders()}")
-    log.info(response.read().decode('utf-8'))
+    print(response.read().decode('utf-8'))
 
 def validate_server_url(args):
     # Get the error report server from an argument