Dan Kenigsberg has submitted this change and it was merged.

Change subject: tcTests: add verbosity to addBridge failure
......................................................................


tcTests: add verbosity to addBridge failure

All too often recently, testUtf8BridgeEthtoolDrvinfo (and others) fail
with

    CalledProcessError: Command '['/usr/sbin/brctl', 'addbr', 
'vdsmtest-ME5LU']' returned non-zero exit status 1

when run by Jenkins.

This patch includes the stdout and stderr of the failing command so we
can understand the issue better, and fix it or skip it.

Change-Id: I83cdf07ccbf82ef27527d358a96892058fc597fc
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40910
Continuous-Integration: Jenkins CI
Reviewed-by: Ido Barkan <ibar...@redhat.com>
Reviewed-by: Francesco Romani <from...@redhat.com>
---
M tests/tcTests.py
1 file changed, 11 insertions(+), 3 deletions(-)

Approvals:
  Ido Barkan: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Verified; Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/40910
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I83cdf07ccbf82ef27527d358a96892058fc597fc
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Petr Horáček <phora...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to