redis-benchmarks-specification icon indicating copy to clipboard operation
redis-benchmarks-specification copied to clipboard

[BUG]: tmp* folders are not deleting after execution of some tests

Open markovamaria opened this issue 1 year ago • 1 comments

Describe the bug tmp* folders are not deleting after execution of some tests. For benchmark version 72, home directory has 12 empty tmp* directories per run after execution.

To Reproduce Steps to reproduce the behavior:

$ numactl --physcpubind=1 ./r_servers/redis-server --protected-mode no --port 6379 --dir run_server_logs --logfile run_server.log --save "" &
$ server_pid=$!
$ redis-benchmarks-spec-client-runner --db_server_host localhost --db_server_port 6379 --client_aggregated_results_folder ./run --flushall_on_every_test_start --flushall_on_every_test_end
$ kill -9 $server_pid 

Expected behavior There is no empty tmp* directories in home directory after execution.

Screenshots for 3 runs home dir constains 36 empty tmp dirs after execution image

markovamaria avatar Jul 03 '23 16:07 markovamaria

From what I found in the code, temporary directory isn't deleted when benchmark is skipped for any reasons (f.e. WARNING Skipping test memtier_benchmark-1key-geo-60M-elements-geohash giving it implies dataset preload) because there is continue right after check. Could it be the case here?

Also running redis-benchmarks-spec-client-runner with --dry-run flag results in creating X empty temporary directories (where X is the number of run benchmarks) that are never deleted.

mpozniak95 avatar Aug 29 '23 15:08 mpozniak95