mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-25 17:12:03 +00:00
Address review comment
This commit is contained in:
parent
7f90b7983c
commit
8a4654c39e
@ -146,7 +146,7 @@ def main():
|
||||
|
||||
run_log_path = logs_path / "run.log"
|
||||
timeout_expired = False
|
||||
timeout = 90 * 60
|
||||
timeout = 1 * 60
|
||||
with TeePopen(run_cmd, run_log_path, timeout=timeout) as process:
|
||||
retcode = process.wait()
|
||||
if process.timeout_exceeded:
|
||||
@ -186,10 +186,9 @@ def main():
|
||||
state, description, test_results, additional_logs = process_results(output_path)
|
||||
|
||||
if timeout_expired:
|
||||
test_result_name = "Check timeout expired"
|
||||
test_results.append(TestResult(test_result_name, "FAIL", timeout))
|
||||
test_results.append(TestResult.create_check_timeout_expired(timeout))
|
||||
state = "failure"
|
||||
description = format_description(test_result_name)
|
||||
description = format_description(test_results[-1].name)
|
||||
|
||||
ch_helper = ClickHouseHelper()
|
||||
s3_path_prefix = os.path.join(
|
||||
|
@ -249,6 +249,10 @@ class TestResult:
|
||||
assert file.exists(), file
|
||||
self.log_files.append(file)
|
||||
|
||||
@staticmethod
|
||||
def create_check_timeout_expired(timeout : float) -> "TestResult":
|
||||
return TestResult("Check timeout expired", "FAIL", timeout)
|
||||
|
||||
|
||||
TestResults = List[TestResult]
|
||||
|
||||
|
@ -175,7 +175,7 @@ def run_stress_test(docker_image_name):
|
||||
logging.info("Going to run stress test: %s", run_command)
|
||||
|
||||
timeout_expired = False
|
||||
timeout = 60 * 1
|
||||
timeout = 60 * 150
|
||||
with TeePopen(run_command, run_log_path, timeout=timeout) as process:
|
||||
retcode = process.wait()
|
||||
if process.timeout_exceeded:
|
||||
|
Loading…
Reference in New Issue
Block a user