sascha/workload-automation
1
0
Fork 0
mirror of https://github.com/ARM-software/workload-automation.git synced 2025-03-21 01:59:13 +00:00
Code Issues Releases Wiki Activity

speedometer: Introduce trivial cleanups

Browse Source 
- Remove unused imports
- Handle the case that @candidate_files may be undefined
- Customize the log message regarding Speedometer timeout

Signed-off-by: Metin Kaya <metin.kaya@arm.com>
This commit is contained in:
Metin Kaya 2025-02-26 11:11:22 +00:00 committed by Marc Bonnici
parent 0732fa9cf0
commit 523fb3f659
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
wa/workloads/speedometer/__init__.py
View File

@ -1,4 +1,4 @@
# Copyright 2014-2018 ARM Limited # Copyright 2014-2025 ARM Limited
# #
# Licensed under the Apache License, Version 2.0 (the "License"); # Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License. # you may not use this file except in compliance with the License.
@ -14,10 +14,8 @@
# #
from collections import defaultdict from collections import defaultdict
from http.server import SimpleHTTPRequestHandler, HTTPServer from http.server import SimpleHTTPRequestHandler, HTTPServer
import logging
import os import os
import re import re
import subprocess
import tarfile import tarfile
import tempfile import tempfile
import threading import threading
@ -275,6 +273,7 @@ class Speedometer(Workload):
benchmark_complete = False benchmark_complete = False
while not benchmark_complete: while not benchmark_complete:
if self.target_file_was_created(local_storage): if self.target_file_was_created(local_storage):
candidate_files = []
if ( if (
iterations % (find_period_s // sleep_period_s) == 0 iterations % (find_period_s // sleep_period_s) == 0
or not local_storage_seen or not local_storage_seen
@ -308,12 +307,12 @@ class Speedometer(Workload):
iterations += 1 iterations += 1
if iterations > ((timeout_period_m * 60) // sleep_period_s): if iterations > ((timeout_period_m * 60) // sleep_period_s):
# We've been waiting 15 minutes for Speedometer to finish running - give up. # We've been waiting <timeout_period_m> minutes for Speedometer to finish running - give up.
if not local_storage_seen: if not local_storage_seen:
raise WorkloadError( raise WorkloadError(
"Speedometer did not complete within 15m - Local Storage wasn't found" f"Speedometer did not complete within {timeout_period_m} minutes - Local Storage wasn't found"
) )
raise WorkloadError("Speedometer did not complete within 15 minutes.") raise WorkloadError(f"Speedometer did not complete within {timeout_period_m} minutes.")
time.sleep(sleep_period_s) time.sleep(sleep_period_s)