Dan Kenigsberg has submitted this change and it was merged. Change subject: log: Change vdsm log file to enforce user/group ......................................................................
log: Change vdsm log file to enforce user/group This commit adds a new log handler that extends WatchedFileHandler, This handler checks current user and group and checks with the expected values, to make sure logs are created with correct permissions. In case of bad access RuntimeError is thrown. This patch also updates logger.conf to use this handler. Change-Id: I0a4d7212cb311b22e4fb60ffdc45163a496a74d6 Signed-off-by: Dima Kuznetsov <[email protected]> Reviewed-on: http://gerrit.ovirt.org/26728 Reviewed-by: Saggi Mizrahi <[email protected]> --- M vdsm/logUtils.py M vdsm/logger.conf.in M vdsm/svdsm.logger.conf.in M vdsm/vdsm 4 files changed, 36 insertions(+), 5 deletions(-) Approvals: Saggi Mizrahi: Looks good to me, approved Dima Kuznetsov: Verified -- To view, visit http://gerrit.ovirt.org/26728 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a4d7212cb311b22e4fb60ffdc45163a496a74d6 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Nir Soffer <[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
