fix

Signed-off-by: Sayali Gaikawad <[email protected]>
gaiksaya · Aug 23, 2024 · a1e28a3 · a1e28a3
1 parent a6f4724
commit a1e28a3
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/validation_workflow/tar/validation_tar.py b/src/validation_workflow/tar/validation_tar.py
@@ -49,14 +49,15 @@ def validation(self) -> bool:
         if self.check_cluster_readiness():
             test_result, counter = ApiTestCases().test_apis(self.args.version, self.args.projects,
                                                             self.check_for_security_plugin(os.path.join(self.tmp_dir.path, "opensearch")) if self.args.allow_http else True)
-            shutil.copy(os.path.join(self.tmp_dir.path, 'opensearch', 'logs', 'opensearch.log'), os.path.join('/tmp', 'opensearch.log'))
             if (test_result):
                 logging.info(f'All tests Pass : {counter}')
                 return True
             else:
+                shutil.copy(os.path.join(self.tmp_dir.path, 'opensearch', 'logs', 'opensearch.log'), os.path.join('/tmp', 'opensearch.log'))
                 self.cleanup()
                 raise Exception(f'Not all tests Pass : {counter}')
         else:
+            shutil.copy(os.path.join(self.tmp_dir.path, 'opensearch', 'logs', 'opensearch.log'), os.path.join('/tmp', 'opensearch.log'))
             self.cleanup()
             raise Exception("Cluster is not ready for API test")