Skip to content

Commit 88c5e52

Browse files
authored
Merge pull request ClickHouse#80613 from azat/perf/fix-report-newline
ci: fix "Cannot parse input: expected '\n' before" in performance tests
2 parents bc9da3b + 54bafc1 commit 88c5e52

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

ci/jobs/performance_tests.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ def run_test(
165165
--profile-seconds 10 \
166166
{test_file}",
167167
verbose=True,
168+
strip=False,
168169
)
169170
duration = sw.duration
170171
if res != 0:

ci/praktika/utils.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@ def get_output(cls, command, strict=False, verbose=False):
182182
return res.stdout.strip()
183183

184184
@classmethod
185-
def get_res_stdout_stderr(cls, command, verbose=True):
185+
def get_res_stdout_stderr(cls, command, verbose=True, strip=True):
186186
if verbose:
187187
print(f"Run command [{command}]")
188188
res = subprocess.run(
@@ -192,7 +192,10 @@ def get_res_stdout_stderr(cls, command, verbose=True):
192192
stderr=subprocess.PIPE,
193193
text=True,
194194
)
195-
return res.returncode, res.stdout.strip(), res.stderr.strip()
195+
if strip:
196+
return res.returncode, res.stdout.strip(), res.stderr.strip()
197+
else:
198+
return res.returncode, res.stdout, res.stderr
196199

197200
@classmethod
198201
def check(

0 commit comments

Comments
 (0)