Dan Kenigsberg has submitted this change and it was merged. Change subject: utils: Remove duplicate set non-blocking functions ......................................................................
utils: Remove duplicate set non-blocking functions Current code had 2 implementations of functions that turned on the O_NONBLOCK flag on file descriptors, none of which were in lib, to avoid creating yet another implementation for lib code to use, one of existing implementations was moved to utils. Change-Id: I1fd242e8b437e485e45127d083c35957ccecf577 Signed-off-by: Dima Kuznetsov <[email protected]> Reviewed-on: http://gerrit.ovirt.org/29413 Reviewed-by: Yaniv Bronhaim <[email protected]> Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Saggi Mizrahi <[email protected]> --- M lib/vdsm/utils.py M vdsm/protocoldetector.py M vdsm/storage/misc.py M vdsm/storage/remoteFileHandler.py 4 files changed, 10 insertions(+), 19 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, but someone else must approve Saggi Mizrahi: Looks good to me, approved Dima Kuznetsov: Verified -- To view, visit http://gerrit.ovirt.org/29413 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1fd242e8b437e485e45127d083c35957ccecf577 Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
