Ido Barkan has posted comments on this change.

Change subject: netinfo: Displays active slave for the bond in getVdsCaps 
vdsmapi-schema: Adding Active slave support networkTests: Test to check if 
active slave exists
......................................................................


Patch Set 5: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/32915/5/tests/functional/networkTests.py
File tests/functional/networkTests.py:

Line 162: class Alarm(Exception):
Line 163:     pass
Line 164: 
Line 165: 
Line 166: class SetupNetworkError(Exception):
you should have removed this Exception and _create_bon__with_mode bur probably 
forgotten.
Line 167:     """
Line 168:     Exception for _create_bond__with_mode
Line 169:     """
Line 170:     pass


-- 
To view, visit http://gerrit.ovirt.org/32915
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib6c40755c361523a28cbd68af110e3b20cb9ae25
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Genadi Chereshnya <gcher...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Genadi Chereshnya <gcher...@redhat.com>
Gerrit-Reviewer: Ido Barkan <ibar...@redhat.com>
Gerrit-Reviewer: Meni Yakove <myak...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to