Alon Bar-Lev has posted comments on this change. Change subject: packaging: moved log location change to patch ......................................................................
Patch Set 3: (1 comment) one thing I think I forgot! add logrotate for these logs... can you please? http://gerrit.ovirt.org/#/c/24408/3/packaging/jasper-war-patches/900_004_props_change_logs_location.patch.in File packaging/jasper-war-patches/900_004_props_change_logs_location.patch.in: Line 13: # jasperserver.root is used only by JasperServer Community Edition. Line 14: # JasperServer Professional users should look for jasperserver-pro.root lower in this file. Line 15: log4j.appender.jasperanalysis=org.apache.log4j.RollingFileAppender Line 16: -log4j.appender.jasperanalysis.File=${ovirt-engine-reports.root}/WEB-INF/logs/jasperanalysis.log Line 17: +log4j.appender.jasperanalysis.File=@PKG_LOG_DIR@/jasperanalysis.log maybe it is easier to create symlink instead of patching? Line 18: log4j.appender.jasperanalysis.MaxFileSize=1024KB Line 19: log4j.appender.jasperanalysis.MaxBackupIndex=1 -- To view, visit http://gerrit.ovirt.org/24408 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id52f44264db2e69a83d696af1748b010b950f17d Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <yd...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com> Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
