From 9a817d71060203876b0a3338ff7826b4e40d6002 Mon Sep 17 00:00:00 2001 From: Alexander Schwartz Date: Mon, 6 May 2024 20:50:51 +0200 Subject: [PATCH] Avoid division by zero when calculating the average Related to #799 Signed-off-by: Alexander Schwartz --- .github/actions/prometheus-metrics-calc/action.yml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/actions/prometheus-metrics-calc/action.yml b/.github/actions/prometheus-metrics-calc/action.yml index 43d65d035..477db47a2 100644 --- a/.github/actions/prometheus-metrics-calc/action.yml +++ b/.github/actions/prometheus-metrics-calc/action.yml @@ -111,7 +111,11 @@ runs: #calculating the number of xsite requests during the test execution xsite_reqs_total_duration=$(awk "BEGIN {print ($num2-$num1); exit}") #calculating the average duration per request in ms - averageXsiteReplicationTimePerReq=$(awk "BEGIN {print int($xsite_reqs_total_duration*1000/$xsite_reqs); exit}") + if [[ $xsite_reqs -gt 0 ]]; then + averageXsiteReplicationTimePerReq=$(awk "BEGIN {print int($xsite_reqs_total_duration*1000/$xsite_reqs); exit}") + else + averageXsiteReplicationTimePerReq="" + fi #Putting metric into JSON jq --arg cacheName "${{ inputs.ispnCacheName}}" --arg val "${averageXsiteReplicationTimePerReq}" --arg testName "${{ inputs.performedTestName }}" \ '.[$testName].ispnStatistics |= .+ {"cacheName": ($cacheName), "averageXsiteReplicationTime":($val|tonumber)}' ${HORREUM_OUTPUT_FILE_NAME} > tmp.json && \