Dan Kenigsberg has submitted this change and it was merged. Change subject: Change file() to open() continued ......................................................................
Change file() to open() continued Update file calls to open calls due to file's deprecation, and also use with statements to take care of closing files. Change-Id: Iab7c37fdc18cafa9dfcbd2e9a8fe4f82022dcca0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128715 Signed-off-by: Yeela Kaplan <[email protected]> Reviewed-on: http://gerrit.ovirt.org/33859 Reviewed-by: Dima Kuznetsov <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M lib/vdsm/netinfo.py M lib/vdsm/utils.py M tests/hooksTests.py M tests/netconfTests.py M tests/tcTests.py M vds_bootstrap/vds_bootstrap.py M vdsm/API.py M vdsm/caps.py M vdsm/hooks.py M vdsm/ppc64HardwareInfo.py M vdsm/substitute_constants.py M vdsm/virt/sampling.py M vdsm_hooks/hostusb/after_vm_destroy.py M vdsm_hooks/hostusb/before_vm_start.py M vdsm_hooks/hugepages/after_vm_destroy.py M vdsm_hooks/hugepages/before_vm_migrate_destination.py M vdsm_hooks/hugepages/before_vm_start.py M vdsm_reg/deployUtil.py.in 18 files changed, 157 insertions(+), 152 deletions(-) Approvals: Yeela Kaplan: Verified Dima Kuznetsov: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33859 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab7c37fdc18cafa9dfcbd2e9a8fe4f82022dcca0 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yeela Kaplan <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
