[GitHub] [tvm] manupa-arm commented on a change in pull request #10408: [AOT] Rerun FVP test incase of first attempt failure

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

manupa-arm commented on a change in pull request #10408:
URL: https://github.com/apache/tvm/pull/10408#discussion_r816486407



##########
File path: tests/python/contrib/test_ethosu/infra.py
##########
@@ -251,13 +251,23 @@ def verify_source(
     """
     interface_api = "c"
     test_runner = create_test_runner(accel, enable_usmp)
-    run_and_check(
-        models,
-        test_runner,
-        interface_api,
-        workspace_byte_alignment=16,
-        data_linkage=AOTDataLinkage(section="ethosu_scratch", alignment=16),
-    )
+
+    def run_mod():
+        run_and_check(
+            models,
+            test_runner,
+            interface_api,
+            workspace_byte_alignment=16,
+            data_linkage=AOTDataLinkage(section="ethosu_scratch", alignment=16),
+        )
+
+    # TODO(lhutton1) This is a quick and dirty work around to help temporarily reduce
+    # the flakyness of the tests. Will remove once #10300 is resolved.
+    try:
+        run_mod()
+    except AssertionError:

Review comment:
       @lhutton1 thanks for quickly taking this up!
   
   We need to print out the logs if it fails (even once) if we are ever to fix this. i.e. we let it rerun but we do print the logs.
   Generally, printing out run_log and compile_log would give the sufficient info to debug this.




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

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