thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-30 03:22:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #46520 from ClickHouse/rollout-42891

Browse Source 
Rollback unnecessary sync because of checking exit code
This commit is contained in:
Alexey Milovidov 2023-02-17 15:13:06 +03:00 committed by GitHub
commit 937bd6dd4f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tests/integration/ci-runner.py
View File

@ -380,11 +380,6 @@ class ClickhouseIntegrationTestsRunner:
) )
def _compress_logs(self, dir, relpaths, result_path): def _compress_logs(self, dir, relpaths, result_path):
# We execute sync in advance to have all files written after containers
# are finished or killed
subprocess.check_call( # STYLE_CHECK_ALLOW_SUBPROCESS_CHECK_CALL
"sync", shell=True
)
retcode = subprocess.call( # STYLE_CHECK_ALLOW_SUBPROCESS_CHECK_CALL retcode = subprocess.call( # STYLE_CHECK_ALLOW_SUBPROCESS_CHECK_CALL
"tar czf {} -C {} {}".format(result_path, dir, " ".join(relpaths)), "tar czf {} -C {} {}".format(result_path, dir, " ".join(relpaths)),
shell=True, shell=True,