From: Changqing Li Date: Tue, 6 May 2025 10:53:37 +0000 (+0800) Subject: send-error-report: make output align with original design when debug disabled X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c45aca4592544d867f49055426e68dd338d4adcc;p=thirdparty%2Fopenembedded%2Fopenembedded-core-contrib.git 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/"} Signed-off-by: Changqing Li Signed-off-by: Richard Purdie --- 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