This is an automated email from the ASF dual-hosted git repository.

lukhut pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git


The following commit(s) were added to refs/heads/main by this push:
     new f14c61f0d1 [AOT] Remove workaround to help resolve test flakiness 
(#15181)
f14c61f0d1 is described below

commit f14c61f0d177a8cf3e6e61a269c036d2b11472ba
Author: Luke Hutton <[email protected]>
AuthorDate: Mon Jul 3 11:25:48 2023 +0100

    [AOT] Remove workaround to help resolve test flakiness (#15181)
    
    Now that #10300 and #10314 have been closed, we should be able to
    remove an old previous attempt to help resolve test flakiness.
---
 python/tvm/testing/aot.py | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/python/tvm/testing/aot.py b/python/tvm/testing/aot.py
index b2814aff2d..9ee3a84c8a 100644
--- a/python/tvm/testing/aot.py
+++ b/python/tvm/testing/aot.py
@@ -17,7 +17,6 @@
 # pylint: disable=use-list-literal, consider-using-with, 
f-string-without-interpolation
 """Common functions for AOT test cases"""
 import contextlib
-import sys
 import datetime
 import os
 import pathlib
@@ -830,14 +829,7 @@ def run_and_check(
         if verbose:
             print("Run command:\n", run_command)
 
-        # TODO(lhutton1) This is a quick and dirty work around to help 
temporarily reduce
-        # the flakyness of the tests. Will remove once #10300 and #10314 are 
resolved.
-        try:
-            _subprocess_check_log_output(run_command, build_path, run_log_path)
-        except RuntimeError as err:
-            print("Failed to run the module, having a second attempt...", 
file=sys.stderr)
-            print(err, file=sys.stderr)
-            _subprocess_check_log_output(run_command, build_path, run_log_path)
+        _subprocess_check_log_output(run_command, build_path, run_log_path)
 
         with open(run_log_path) as run_log:
             assert AOT_SUCCESS_TOKEN in run_log.read()

Reply via email to