Change in vdsm[master]: testSetupNetworksAddDelDhcp: Delete a lease file left over b...

danken at redhat.com danken at redhat.com
Fri Jan 16 11:55:20 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: testSetupNetworksAddDelDhcp: Delete a lease file left over by dhclient
......................................................................


testSetupNetworksAddDelDhcp: Delete a lease file left over by dhclient

The non-obvious reason we must not leave lease files laying around
is that the extended version of the test would, in bridged cases, detect
DHCPv4 or DHCPv6 even in permutations in which either was not requested,
because we reuse a bridge name.

Change-Id: I1af7c7acc3fcb6526da1e12f8b5fe82e8415536b
Signed-off-by: Ondřej Svoboda <osvoboda at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/36974
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/functional/dhcp.py
M tests/functional/networkTests.py
2 files changed, 54 insertions(+), 36 deletions(-)

Approvals:
  Ondřej Svoboda: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1af7c7acc3fcb6526da1e12f8b5fe82e8415536b
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ido Barkan <ibarkan at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list