20250207/crontab_save_log 김규영 기술블로그 제출 #117
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
K-KY
requested review from
day1companyhaewooklee and
day1chan99k
as code owners
baki-719
approved these changes
Feb 5, 2025
There was a problem hiding this comment.
안녕하세요 규영님!
우선 너무 늦게 남겨드려서 죄송합니다!
기술적으로 틀리거나 잘못된 내용은 없는 것 같아요.
다만, 전체적인 글의 구성이 아쉬운점이 있네요
소위 기술블로그라하는 개발관련 아티클의 대부분은
어떠한 문제상황이 발생하였고 이를 해결하기 위해 어떠한 기술을 사용했거나 한다.
그리고 직접 적용해보니 이 방법이나 기술은 이러한 장단점이 있었다.
와 같은 방식으로 흐르게 되는데요
요 아티클은
- 어떠한 문제상황인건지 설명이 부족하고
- 어떠한 기술을 사용해서 극복을 시도했는데 왜 이 기술을 선택했는가? 애대한 설명이 부족합니다.
- crontab 자체에 대한 설명이 부족합니다.
아예 docker volume을 언급하면서 글을 끝마쳤는데
docker volume을 적용하고 docker volume에 대한 설명을 더 깊게 써보는 것도 좋을꺼 같아요!
그래도 바쁜 일정중에 이렇게 글을 쓰시느라 고생 많으 셨어요
Comment on lines
+14
to
+22
| > 앱의 로그를 파일로 관리하라는 피드백을 받았습니다. | ||
| > | ||
| > 현재 애플리케이션을 도커에서 실행중이기 때문에 도커 외부에 저장해야합니다. | ||
| > | ||
| > 왜냐면 컨테이너의 데이터는 휘발성이니까요 | ||
| > | ||
| > 앱 내부에서 파일로 저장하는게 더 좋은 방법인 것 같지만 이걸 할 당시에는 내부에서는 저장하지 않고 있었어요 | ||
| > | ||
| > 서버에 들어가지 않고 로그를 확인할 수 있는 수단이 필요했기 때문에 일단 crontab과 shell 스크립트를 사용해서 로그를 저장하기로 했습니다. |
There was a problem hiding this comment.
개요 부분이 아쉽네요
- 로그를 왜 파일로 남겨야하는가?
- 파일로 남겼을때 어떤 이유로 이런 crontab을 사용하게 되었는가?
이 두가지가 개요 부분에 담겨 있어야하는데
1번 파일로 왜 남겨야하는가? 이게 피드백때문은 아니자나요? 왜 로그를 파일로 남겨야하는지를 좀 더 이해하고 작성해주세요
2번 앱 내부에서 그 당시에는 저장하고 있지 않았다고 하셨는데
내부에 저장하는걸 우선적으로 하지 않고 왜 shell script를 사용했는가에 대한 정당성이 부족하네요
Comment on lines
+32
to
+38
| - 일단 crontab에 대해 대충 설명을 해보면 | ||
| - 일정 시간마다 지정한 동작을 수행하도록 합니다. | ||
| - \* * * * * echo "123123" 처럼 작성합니다 | ||
| - 이건 1분에 한번씩 동작을 수행하는걸 의미합니다. | ||
| - 왼쪽부터 *은 분, 시, 일, 월, 요일을 의미합니다. | ||
| - 각 자리의 * 은 매분, 매시, 매일, 매월, 모든 요일을 의미하며 | ||
| - \* 1 * * * 은 매일 01시에 수행 |
There was a problem hiding this comment.
이 내용은 crontab의 표현식이지 crontab이 무엇인가에 대한 설명이 아닙니다.
아래에 crontab의 소개 글을 첨부했지만 소개는 넘어가고 있으니 차라리 영어로 된 소개글을
다 해석하고 요약해서 적어주는게 좋을꺼 같아요
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.