From 555b4a00251188e1346883ad2f015af90a120094 Mon Sep 17 00:00:00 2001 From: Gabriela Cervantes Date: Mon, 13 Mar 2023 16:16:04 +0000 Subject: [PATCH] metrics: Fix memory inside container test This PR replaces ctr with CTR_EXE which is defined in the metrics lib common.bash as well as it uses sudo to have uniformity across the metrics scripts. Fixes #5524 Signed-off-by: Gabriela Cervantes --- metrics/density/memory_usage_inside_container.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/metrics/density/memory_usage_inside_container.sh b/metrics/density/memory_usage_inside_container.sh index 6eb90d215..4aba29c33 100755 --- a/metrics/density/memory_usage_inside_container.sh +++ b/metrics/density/memory_usage_inside_container.sh @@ -122,7 +122,7 @@ function main() { metrics_json_init for (( i=0; i<$iterations; i++ )) do - local output=$(ctr run --memory-limit $((MEMSIZE*1024)) --rm --runtime=$CTR_RUNTIME $IMAGE busybox sh -c "$CMD" 2>&1) + local output=$(sudo -E "${CTR_EXE}" run --memory-limit $((MEMSIZE*1024)) --rm --runtime=$CTR_RUNTIME $IMAGE busybox sh -c "$CMD" 2>&1) if [ ${output_format} = "json" ]; then parse_results "${output}" "${memtotalAvg}" "${memfreeAvg}" "${memavailableAvg}"