From 58785be56ad221bce552c571b624039a3c5c3da2 Mon Sep 17 00:00:00 2001 From: Changqing Li Date: Tue, 6 May 2025 18:53:37 +0800 Subject: [PATCH] send-error-report: make output align with original design when debug disabled 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/"} (From OE-Core rev: c45aca4592544d867f49055426e68dd338d4adcc) Signed-off-by: Changqing Li Signed-off-by: Richard Purdie --- scripts/send-error-report | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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