Skip to content

Commit 2d51155

Browse files
Oleg Chaplashkinylobankov
Oleg Chaplashkin
authored andcommitted
Do not raise error in save_artifacts function
The error has been replaced with a log entry (if the copying process failed, the current test will be marked as failed). Also, the PID of the server process was removed from the log string because the process may be missing. Close #321
1 parent 251b35f commit 2d51155

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

luatest/server.lua

+2-2
Original file line numberDiff line numberDiff line change
@@ -353,8 +353,8 @@ end
353353
function Server:save_artifacts()
354354
local ok, err = fio.copytree(self.workdir, self.artifacts)
355355
if not ok then
356-
error(('Failed to copy artifacts for server (alias: %s, workdir: %s, pid: %d): %s')
357-
:format(self.alias, fio.basename(self.workdir), self.process.pid, err))
356+
log.error(('Failed to copy artifacts for server (alias: %s, workdir: %s): %s')
357+
:format(self.alias, fio.basename(self.workdir), err))
358358
end
359359
end
360360

0 commit comments

Comments
 (0)