Change in vdsm[master]: configNetTests: test atomicBackup
danken at redhat.com
danken at redhat.com
Thu Jul 5 09:34:12 UTC 2012
Dan Kenigsberg has submitted this change and it was merged.
Change subject: configNetTests: test atomicBackup
......................................................................
configNetTests: test atomicBackup
The sematics of this function were far from clear to me before I've
written this unit test. While writing this test, I've found the issues
that are fixed in the preceding patches.
I hope this test makes the function of _atomicBackup() clearer not only
to me.
Change-Id: I2925bc9b4db86d9a38fad761e58d164b3ad7bfd4
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/configNetworkTests.py
1 file changed, 50 insertions(+), 0 deletions(-)
Approvals:
Dan Kenigsberg: Verified; Looks good to me, approved
Igor Lvovsky: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/5954
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2925bc9b4db86d9a38fad761e58d164b3ad7bfd4
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer at redhat.com>
More information about the vdsm-patches
mailing list