Dan Kenigsberg has submitted this change and it was merged. Change subject: use with statement to take care of unclosed files ......................................................................
use with statement to take care of unclosed files Change-Id: I0c33b22aa9828245c4598622ae6870f13b797036 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128715 Signed-off-by: Yeela Kaplan <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31717 Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/netinfo.py M lib/vdsm/tool/configurators/configfile.py M tests/capsTests.py M tests/miscTests.py M tests/netconfTests.py M tests/remoteFileHandlerTests.py M vds_bootstrap/setup M vds_bootstrap/vds_bootstrap.py M vdsm/caps.py M vdsm/network/configurators/ifcfg.py M vdsm/network/models.py M vdsm/rpc/process-schema.py M vdsm/storage/sd.py M vdsm/supervdsmServer M vdsm/virt/sampling.py M vdsm_reg/deployUtil.py.in M vdsm_reg/vdsm-reg-setup.in 17 files changed, 121 insertions(+), 82 deletions(-) Approvals: Yeela Kaplan: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/31717 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c33b22aa9828245c4598622ae6870f13b797036 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[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
