Sandro Bonazzola has submitted this change and it was merged. Change subject: config: fix exception handling ...................................................................... config: fix exception handling Fix: Traceback (most recent call last): File "/usr/bin/engine-log-collector", line 1437, in <module> setup_pg_defaults() File "/usr/bin/engine-log-collector", line 165, in setup_pg_defaults pg_user = get_pg_var('admin') or pg_user File "/usr/bin/engine-log-collector", line 138, in get_pg_var if e.errno != errno.ENOENT: NameError: global name 'e' is not defined Change-Id: I87c5beb46a5bd8df6e01e0472db0f5dc45bfd9c1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41483 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87c5beb46a5bd8df6e01e0472db0f5dc45bfd9c1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com&gt; Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com&gt; Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com&gt; Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com&gt; Gerrit-Reviewer: automation(a)ovirt.org