-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT] 로그백 설정 #173
base: develop
Are you sure you want to change the base?
[FEAT] 로그백 설정 #173
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 한번 논의해봐야겠네요.! 고생하셨습니당!
<onMismatch>DENY</onMismatch> | ||
</filter> | ||
|
||
<file>log/warnLog.log</file> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기는 ${LOG_PATH}를 안쓰나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이건 고쳐서 다시 푸시하겠습니당 감사해요!
|
||
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy"> | ||
<fileNamePattern>${LOG_PATH}/%d{yyyy-MM, aux}/${LOG_FILE_NAME}.%d{yyyy-MM-dd}.log</fileNamePattern> | ||
<totalSizeCap>20GB</totalSizeCap> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저희 인스턴스 생각하면 20기가면 큰 것 같습니다. 더 작게 제한하면 좋을 것 같아요. 2GB라던지!?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
로그파일은 ec2말고 s3에 저장하는 경우도 있던데, 어떻게 생각하세욤?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2GB로 줄이는게 나을 듯 하네용.
그리고 s3에 저장하는 경우는 한 번 찾아보고 적용하겠습니당.
땡큐!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
|
||
<rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy"> | ||
<fileNamePattern>${LOG_PATH}/%d{yyyy-MM, aux}/${LOG_FILE_NAME}.%d{yyyy-MM-dd}.log</fileNamePattern> | ||
<totalSizeCap>20GB</totalSizeCap> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
로그파일은 ec2말고 s3에 저장하는 경우도 있던데, 어떻게 생각하세욤?
Related Issue 🚀
Work Description ✏️
console에 표시하고 errorLog.log, warnLog.log에 로그 남겨두도록 설정.
warn중에서는 어떤 내용을 필터링 해야할 지 아직 정해지지 않아 우선 모든 warn을 남겨두도록 설정했습니당. 이 부분은 추후에 다시 얘기해보면 좋을 것 같아욥
PR Point 📸