Skip to content

Commit 3003d72

Browse files
committed
[KYUUBI #6268] Specify logDir for RollingFile filePattern
# 🔍 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 🔖 - [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 ⚰️ #### Behavior With This Pull Request 🎉 #### 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 b599583 [wforget] Specify logDir for RollingFile filePattern Authored-by: wforget <[email protected]> Signed-off-by: wforget <[email protected]> (cherry picked from commit a26156b) Signed-off-by: wforget <[email protected]>
1 parent 9334c93 commit 3003d72

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

conf/log4j2.xml.template

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,15 @@
3535
</Filters>
3636
</Console>
3737
<RollingFile name="restAudit" fileName="${sys:logDir}/${sys:restAuditLogPath}"
38-
filePattern="${sys:restAuditLogFilePattern}">
38+
filePattern="${sys:logDir}/${sys:restAuditLogFilePattern}">
3939
<PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %c{1}: %m%n%ex"/>
4040
<Policies>
4141
<SizeBasedTriggeringPolicy size="51200KB" />
4242
</Policies>
4343
<DefaultRolloverStrategy max="10"/>
4444
</RollingFile>
4545
<RollingFile name="k8sAudit" fileName="${sys:logDir}/${sys:k8sAuditLogPath}"
46-
filePattern="${sys:k8sAuditLogFilePattern}">
46+
filePattern="${sys:logDir}/${sys:k8sAuditLogFilePattern}">
4747
<PatternLayout pattern="%d{yyyy-MM-dd HH:mm:ss.SSS} %p %c{1}: %m%n%ex"/>
4848
<Policies>
4949
<SizeBasedTriggeringPolicy size="51200KB" />

0 commit comments

Comments
 (0)