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

[FIX] error, info 로그 저장 디렉토리 복구 #226

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

dragontaek-lee
Copy link
Member

Related Issue 🚀

Work Description ✏️

  • logback 세팅 시 error, info 디렉토리를 명시적으로 적어주기 위해서 경로를 수정했었습니다.
    • 인스턴스에서만 사용할 것을 고려하여 우분투 기준 디렉토리로 작성하였습니다.
  • 하지만 오류 대응을 위해 local에서 prod 서버를 가동할 시, 해당 디렉토리가 없어 문제가 발생하여 초기 경로로 복구해두었습니다.

PR Point 📸

  • local prod를 돌릴 때 log 파일을 볼 필요가 있을까 싶습니다!
    • local prod yml을 하나 더 파서 (e.x local-prod) prod로 가동할때만(인스턴스) log를 쌓는 것은 어떨까 싶습니다.
    • 이렇게 한다면 수정 전 코드대로 우분투 인스턴스 디렉토리 기준으로 해도 문제는 없을 것 같습니다.

@dragontaek-lee dragontaek-lee added bug Something isn't working yongtaek 용택 작업 labels Jan 24, 2024
@dragontaek-lee dragontaek-lee requested a review from thguss January 24, 2024 12:25
@dragontaek-lee dragontaek-lee self-assigned this Jan 24, 2024
Copy link
Collaborator

@thguss thguss left a comment

Choose a reason for hiding this comment

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

이슈 대응 감사합니다!
저도 로컬에서 실행한 prod 서버의 로그를 확인할 일은 거의 없다고 생각해요. 하지만 Swagger 이슈처럼 dev, prod 분기가 필요한 상황에는 로컬에서 prod 서버의 실행이 필요하다고 생각되어, 실행 부분만 정상적으로 돌아간다면 좋을 것 같아요!
처음에는 로컬에서 prod가 안 돌아가서 일정 기간부터 prod 서버에 새로운 코드가 반영되지 않았나 생각했는데 그건 아닌 것 같아서 다행입니다! 로컬 내 prod 서버 실행만 된다면 우분투 위치로 다시 돌려도 좋을 것 같아요!
빠른 대응 감사합니다~~

@dragontaek-lee dragontaek-lee added this pull request to the merge queue Feb 28, 2024
Merged via the queue into develop with commit df55d68 Feb 28, 2024
1 check passed
@thguss thguss deleted the yongtaek/#225 branch March 15, 2024 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/XS yongtaek 용택 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] logback 설정 이슈 해결
2 participants