automation/eclair: Reduce verbosity of ECLAIR logs.

[XEN PATCH] automation/eclair: Reduce verbosity of ECLAIR logs.

Posted by Nicola Vetrini 11 months, 1 week ago

While activating verbose logging simplifies debugging, this causes
GitLab logs to be truncated, preventing the links to the ECLAIR
analysis database to be shown.

No functional change.

Fixes: c4392ec83244 ("automation: Add ECLAIR utilities and settings")
Signed-off-by: Nicola Vetrini <nicola.vetrini@bugseng.com>
---
 automation/eclair_analysis/ECLAIR/analysis.ecl | 2 --
 automation/eclair_analysis/ECLAIR/analyze.sh   | 2 ++
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/automation/eclair_analysis/ECLAIR/analysis.ecl b/automation/eclair_analysis/ECLAIR/analysis.ecl
index 824283a989c1..29409a9af0eb 100644
--- a/automation/eclair_analysis/ECLAIR/analysis.ecl
+++ b/automation/eclair_analysis/ECLAIR/analysis.ecl
@@ -10,8 +10,6 @@ setq(analysis_kind,getenv("ANALYSIS_KIND"))
 # strings_map("scheduled-analysis",500,"","^.*$",0)
 # map_strings("scheduled-analysis",analysis_kind)
 
--verbose
-
 -enable=B.REPORT.ECB
 -config=B.REPORT.ECB,output=join_paths(data_dir,"FRAME.@FRAME@.ecb")
 -config=B.REPORT.ECB,preprocessed=show
diff --git a/automation/eclair_analysis/ECLAIR/analyze.sh b/automation/eclair_analysis/ECLAIR/analyze.sh
index 1dc63c1bc2d0..2356fc4007dd 100755
--- a/automation/eclair_analysis/ECLAIR/analyze.sh
+++ b/automation/eclair_analysis/ECLAIR/analyze.sh
@@ -91,11 +91,13 @@ find "${ECLAIR_DATA_DIR}" -maxdepth 1 -name "FRAME.*.ecb" |
   sort | xargs cat |
   "${ECLAIR_BIN_DIR}eclair_report" \
     "-create_db='${PROJECT_ECD}'" \
+    -quiet \
     -load=/dev/stdin > "${ECLAIR_REPORT_LOG}" 2>&1
 
 # Create the Jenkins reports file.
 "${ECLAIR_BIN_DIR}eclair_report" \
   "-db='${PROJECT_ECD}'" \
+  -quiet \
   "-eval_file='${SCRIPT_DIR}/report.ecl'" \
   >> "${ECLAIR_REPORT_LOG}" 2>&1
 
-- 
2.43.0

Re: [XEN PATCH] automation/eclair: Reduce verbosity of ECLAIR logs.

Posted by Stefano Stabellini 11 months, 1 week ago

On Tue, 4 Mar 2025, Nicola Vetrini wrote:
> While activating verbose logging simplifies debugging, this causes
> GitLab logs to be truncated, preventing the links to the ECLAIR
> analysis database to be shown.
> 
> No functional change.
> 
> Fixes: c4392ec83244 ("automation: Add ECLAIR utilities and settings")
> Signed-off-by: Nicola Vetrini <nicola.vetrini@bugseng.com>

Acked-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  automation/eclair_analysis/ECLAIR/analysis.ecl | 2 --
>  automation/eclair_analysis/ECLAIR/analyze.sh   | 2 ++
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/automation/eclair_analysis/ECLAIR/analysis.ecl b/automation/eclair_analysis/ECLAIR/analysis.ecl
> index 824283a989c1..29409a9af0eb 100644
> --- a/automation/eclair_analysis/ECLAIR/analysis.ecl
> +++ b/automation/eclair_analysis/ECLAIR/analysis.ecl
> @@ -10,8 +10,6 @@ setq(analysis_kind,getenv("ANALYSIS_KIND"))
>  # strings_map("scheduled-analysis",500,"","^.*$",0)
>  # map_strings("scheduled-analysis",analysis_kind)
>  
> --verbose
> -
>  -enable=B.REPORT.ECB
>  -config=B.REPORT.ECB,output=join_paths(data_dir,"FRAME.@FRAME@.ecb")
>  -config=B.REPORT.ECB,preprocessed=show
> diff --git a/automation/eclair_analysis/ECLAIR/analyze.sh b/automation/eclair_analysis/ECLAIR/analyze.sh
> index 1dc63c1bc2d0..2356fc4007dd 100755
> --- a/automation/eclair_analysis/ECLAIR/analyze.sh
> +++ b/automation/eclair_analysis/ECLAIR/analyze.sh
> @@ -91,11 +91,13 @@ find "${ECLAIR_DATA_DIR}" -maxdepth 1 -name "FRAME.*.ecb" |
>    sort | xargs cat |
>    "${ECLAIR_BIN_DIR}eclair_report" \
>      "-create_db='${PROJECT_ECD}'" \
> +    -quiet \
>      -load=/dev/stdin > "${ECLAIR_REPORT_LOG}" 2>&1
>  
>  # Create the Jenkins reports file.
>  "${ECLAIR_BIN_DIR}eclair_report" \
>    "-db='${PROJECT_ECD}'" \
> +  -quiet \
>    "-eval_file='${SCRIPT_DIR}/report.ecl'" \
>    >> "${ECLAIR_REPORT_LOG}" 2>&1
>  
> -- 
> 2.43.0
>