Dan Kenigsberg has submitted this change and it was merged. Change subject: Check if log file exist on startup and log directory is accessible ......................................................................
Check if log file exist on startup and log directory is accessible If file isn't exist we need to continue. Otherwise, os.access returns false and we report permission error. When continue we need to verify that vdsm has the right permissions to create the log file in the log directory. Change-Id: I300adc5ac3d9b12fee49023a54e5ffb4dee98da1 Signed-off-by: Yaniv Bronhaim <[email protected]> --- M vdsm/vdsm 1 file changed, 8 insertions(+), 0 deletions(-) Approvals: Adam Litke: Verified; Looks good to me, but someone else must approve Yaniv Bronhaim: Verified Ohad Basan: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved Objections: Saggi Mizrahi: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/10194 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I300adc5ac3d9b12fee49023a54e5ffb4dee98da1 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ohad Basan <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
