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