Skip to content
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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

[FEAT] 로그백 설정 #173

wants to merge 4 commits into from

Conversation

kjy-asl
Copy link
Contributor

@kjy-asl kjy-asl commented May 13, 2024

Related Issue 🚀

Work Description ✏️

  • logback-spring.xml 을 통해 로그기록 남겨두는 경로와 레벨 설정.
    console에 표시하고 errorLog.log, warnLog.log에 로그 남겨두도록 설정.
    warn중에서는 어떤 내용을 필터링 해야할 지 아직 정해지지 않아 우선 모든 warn을 남겨두도록 설정했습니당. 이 부분은 추후에 다시 얘기해보면 좋을 것 같아욥

PR Point 📸

Copy link
Member

@dragontaek-lee dragontaek-lee left a 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>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기는 ${LOG_PATH}를 안쓰나요?

Copy link
Contributor Author

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>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저희 인스턴스 생각하면 20기가면 큰 것 같습니다. 더 작게 제한하면 좋을 것 같아요. 2GB라던지!?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

로그파일은 ec2말고 s3에 저장하는 경우도 있던데, 어떻게 생각하세욤?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2GB로 줄이는게 나을 듯 하네용.
그리고 s3에 저장하는 경우는 한 번 찾아보고 적용하겠습니당.
땡큐!

Copy link
Contributor

@GaHee99 GaHee99 left a 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>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

로그파일은 ec2말고 s3에 저장하는 경우도 있던데, 어떻게 생각하세욤?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants