[GitHub] [airflow] uranusjr commented on a change in pull request #15080: Printing stats is faster now

[GitBox <gi...@apache.org>]

uranusjr commented on a change in pull request #15080:
URL: https://github.com/apache/airflow/pull/15080#discussion_r603776496



##########
File path: scripts/ci/libraries/_docker_engine_resources.sh
##########
@@ -22,27 +22,25 @@ function docker_engine_resources::print_overall_stats() {
     echo "Overall resource statistics"
     echo
     docker stats --all --no-stream --no-trunc
-    docker run --rm --entrypoint /bin/bash "${AIRFLOW_CI_IMAGE}" -c "free -h"
-    df --human || true
+    docker run --rm --entrypoint /bin/bash "debian:buster-slim" -c "cat /proc/meminfo"
+    df -H || true

Review comment:
       Is this intentional? `df --human` is 1024-based (same as `df -h`), but `df -H` is 100-based (same as `df --si`).
   
   Otherwise lgtm.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org