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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> Gerrit-Reviewer: Livnat Peer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
