Francesco Romani has posted comments on this change.

Change subject: sampling: make _getInterfaceStats a function
......................................................................


Patch Set 19:

Jenjins failure unrelated

00:10:22.063 
======================================================================
00:10:22.063 ERROR: setting another mirror action should not obstract the first 
one
00:10:22.063 
----------------------------------------------------------------------
00:10:22.063 Traceback (most recent call last):
00:10:22.063   File "/tmp/run/vdsm/tests/testValidation.py", line 105, in 
wrapper
00:10:22.063     return f(*args, **kwargs)
00:10:22.063   File "/tmp/run/vdsm/tests/tcTests.py", line 446, in setUp
00:10:22.063     _checkDependencies()
00:10:22.063   File "/tmp/run/vdsm/tests/tcTests.py", line 185, in 
_checkDependencies
00:10:22.063     dev.delDevice()
00:10:22.063   File "/tmp/run/vdsm/tests/tcTests.py", line 105, in delDevice
00:10:22.063     check_call([EXT_BRCTL, 'delbr', self.devName])
00:10:22.063   File "/tmp/run/vdsm/tests/tcTests.py", line 62, in check_call
00:10:22.063     out, err)
00:10:22.063 ExecError: Command ['/usr/sbin/brctl', 'delbr', 'vdsm-9exlnKNr3X'] 
returned non-zero exit status 1.
00:10:22.063 -------------------- >> begin captured logging << 
--------------------
00:10:22.063 root: DEBUG: /usr/sbin/brctl addbr vdsm-9exlnKNr3X (cwd None)
00:10:22.063 root: DEBUG: SUCCESS: <err> = ''; <rc> = 0
00:10:22.063 root: DEBUG: /sbin/ip link set vdsm-9exlnKNr3X up (cwd None)
00:10:22.063 root: DEBUG: SUCCESS: <err> = ''; <rc> = 0
00:10:22.063 root: DEBUG: /usr/sbin/tc qdisc add dev vdsm-9exlnKNr3X ingress 
(cwd None)
00:10:22.063 root: DEBUG: SUCCESS: <err> = ''; <rc> = 0
00:10:22.063 root: DEBUG: /sbin/ip link set vdsm-9exlnKNr3X down (cwd None)
00:10:22.063 root: DEBUG: SUCCESS: <err> = ''; <rc> = 0
00:10:22.063 root: DEBUG: /usr/sbin/brctl delbr vdsm-9exlnKNr3X (cwd None)
00:10:22.063 root: DEBUG: FAILED: <err> = "bridge vdsm-9exlnKNr3X is still up; 
can't delete it\n"; <rc> = 1
00:10:22.063 --------------------- >> end captured logging << 
---------------------

(again?! I thought we solved this some time ago)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I0b5796f3463585794d2696ce73f8357cb0fd1f78
Gerrit-PatchSet: 19
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Ido Barkan <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to