Giuseppe Vallarelli has posted comments on this change.

Change subject: tests: addNetwork bond with one or many vlans.
......................................................................


Patch Set 7:

(1 comment)

Thanks good catch!

....................................................
File tests/functional/utils.py
Line 186:         return bond_name in self.netinfo.bondings and \
Line 187:             (not nics or set(nics) ==
Line 188:              set(self.netinfo.bondings[bond_name]['slaves']))
Line 189: 
Line 190:     def vlanExists(self, vlan_name, dev=None):
Done
Line 191:         return vlan_name in self.netinfo.vlans and \
Line 192:             (not dev or dev ==
Line 193:              self.netinfo.vlans[vlan_name]['iface'])
Line 194: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I84b1582bbc9de09009a7e076386658035317f7db
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to