(kyuubi) branch master updated: [KYUUBI #6268] Specify logDir for RollingFile filePattern

[wangzhen]

This is an automated email from the ASF dual-hosted git repository.

wangzhen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new a26156b8f [KYUUBI #6268] Specify logDir for RollingFile filePattern
a26156b8f is described below

commit a26156b8f4835028faefe469fa7f3be8f1d6c42a
Author: wforget <643348...@qq.com>
AuthorDate: Tue Apr 9 17:27:56 2024 +0800

    [KYUUBI #6268] Specify logDir for RollingFile filePattern
    
    # :mag: Description
    ## Issue References 🔗
    
    This pull request fixes #
    
    ## Describe Your Solution 🔧
    
    We should specify `sys:logDir` for RollingFile filePattern, otherwise the 
log will be rolled to kyuubi work directory.
    
    ## Types of changes :bookmark:
    
    - [X] Bugfix (non-breaking change which fixes an issue)
    - [ ] New feature (non-breaking change which adds functionality)
    - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
    
    ## Test Plan 🧪
    
    #### Behavior Without This Pull Request :coffin:
    
    #### Behavior With This Pull Request :tada:
    
    #### Related Unit Tests
    
    ---
    
    # Checklist 📝
    
    - [X] This patch was not authored or co-authored using [Generative 
Tooling](https://www.apache.org/legal/generative-tooling.html)
    
    **Be nice. Be informative.**
    
    Closes #6268 from wForget/hotfix2.
    
    Closes #6268
    
    b599583be [wforget] Specify logDir for RollingFile filePattern
    
    Authored-by: wforget <643348...@qq.com>
    Signed-off-by: wforget <643348...@qq.com>
---
 conf/log4j2.xml.template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/conf/log4j2.xml.template b/conf/log4j2.xml.template
index f1ed28b2e..1f186bc67 100644
--- a/conf/log4j2.xml.template
+++ b/conf/log4j2.xml.template
@@ -34,7 +34,7 @@
             <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %tn %c: 
%m%n%ex"/>
         </Console>
         <RollingFile name="restAudit" 
fileName="${sys:logDir}/${sys:restAuditLogPath}"
-                     filePattern="${sys:restAuditLogFilePattern}">
+                     
filePattern="${sys:logDir}/${sys:restAuditLogFilePattern}">
             <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %c{1}: 
%m%n%ex"/>
             <Policies>
                 <SizeBasedTriggeringPolicy size="51200KB" />
@@ -42,7 +42,7 @@
             <DefaultRolloverStrategy max="10"/>
         </RollingFile>
         <RollingFile name="k8sAudit" 
fileName="${sys:logDir}/${sys:k8sAuditLogPath}"
-                     filePattern="${sys:k8sAuditLogFilePattern}">
+                     filePattern="${sys:logDir}/${sys:k8sAuditLogFilePattern}">
             <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %c{1}: 
%m%n%ex"/>
             <Policies>
                 <SizeBasedTriggeringPolicy size="51200KB" />
@@ -50,7 +50,7 @@
             <DefaultRolloverStrategy max="10"/>
         </RollingFile>
         <RollingFile name="opAudit" 
fileName="${sys:logDir}/${sys:opAuditLogPath}"
-                     filePattern="${sys:opAuditLogFilePattern}">
+                     filePattern="${sys:logDir}/${sys:opAuditLogFilePattern}">
             <PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %c{1}: 
%m%n%ex"/>
             <Policies>
                 <SizeBasedTriggeringPolicy size="51200KB" />