almalinux
/
pungi
15
1
Fork 0
Code Issues Pull Requests Releases Activity

[buildinstall] Add more debugging output 

When it fails, a traceback will be printed.

Signed-off-by: Lubomír Sedlář <lsedlar@redhat.com>
This commit is contained in:
Lubomír Sedlář 2016-03-16 18:16:24 +01:00
parent f5897eccbb
commit 0759c7fb48
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pungi/phases/buildinstall.py
View File

@ -22,6 +22,7 @@ import pipes
import tempfile import tempfile
import shutil import shutil
import re import re
import traceback
from kobo.threads import ThreadPool, WorkerThread from kobo.threads import ThreadPool, WorkerThread
from kobo.shortcuts import run from kobo.shortcuts import run
@ -386,9 +387,11 @@ class BuildinstallThread(WorkerThread):
if not compose.can_fail(variant, arch, 'buildinstall'): if not compose.can_fail(variant, arch, 'buildinstall'):
raise raise
else: else:
tb = traceback.format_exc()
self.pool.log_info( self.pool.log_info(
'[FAIL] Buildinstall for variant %s arch %s failed, but going on anyway.\n%s' '[FAIL] Buildinstall for variant %s arch %s failed, but going on anyway.\n%s'
% (variant.uid if variant else 'None', arch, exc)) % (variant.uid if variant else 'None', arch, exc))
self.pool.log_debug(tb)
def worker(self, compose, arch, variant, cmd, num): def worker(self, compose, arch, variant, cmd, num):
runroot = compose.conf.get("runroot", False) runroot = compose.conf.get("runroot", False)

2
pungi/wrappers/kojiwrapper.py
View File

@ -86,7 +86,7 @@ class KojiWrapper(object):
contains the id, it will be captured and returned. contains the id, it will be captured and returned.
""" """
task_id = None task_id = None
retcode, output = run(command, can_fail=True, logfile=log_file) retcode, output = run(command, can_fail=True, logfile=log_file, show_cmd=True)
if "--task-id" in command: if "--task-id" in command:
first_line = output.splitlines()[0] first_line = output.splitlines()[0]
if re.match(r'^\d+$', first_line): if re.match(r'^\d+$', first_line):