Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: pr-ism/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthrough이 PR은 Docker 기반 배포 준비를 추가합니다. Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.dockerignore:
- Around line 16-21: 현재 .dockerignore의 규칙(docs/, *.md, !README.md 등)만으로는
src/main/resources/static/docs 하위의 생성 문서가 도커 빌드 컨텍스트에 포함되어 이미지에 들어갈 수 있습니다;
.dockerignore에 명시적으로 src/main/resources/static/docs/ 혹은 **/static/docs/ 같은 패턴을
추가해 생성 산출물 경로를 제외하고, 기존 예외(!README.md)와 충돌하지 않도록 확인하세요(참조: 현재 파일의 docs/, *.md,
!README.md 규칙 및 빌드 스크립트의 -x copyApiDocument 동작을 함께 고려).
In `@Dockerfile`:
- Around line 27-39: The Dockerfile forces injection of the Pinpoint Java agent
in the ENTRYPOINT using PINPOINT_AGENT_PATH and PINPOINT_AGENT_VERSION, which
will fail if those envs or the jar are absent; change this to be conditional:
add ARG/ENV defaults for PINPOINT_AGENT_PATH and PINPOINT_AGENT_VERSION (empty)
and modify the ENTRYPOINT to check for non-empty PINPOINT_AGENT_PATH and
PINPOINT_AGENT_VERSION and the existence of the agent jar before appending the
-javaagent option (or move the logic into a small startup shell script invoked
by ENTRYPOINT that builds JAVA_AGENT_OPTS only when
${PINPOINT_AGENT_PATH}/pinpoint-bootstrap-${PINPOINT_AGENT_VERSION}.jar exists),
so the container starts normally when Pinpoint vars are unset.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository: pr-ism/coderabbit/.coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 93f779ad-5ab0-4d1e-a806-fb26b1dd9dfa
📒 Files selected for processing (3)
.dockerignoreDockerfilebuild.gradle
| docs/ | ||
| *.md | ||
| !README.md | ||
| .github | ||
| .env | ||
| .env.local |
There was a problem hiding this comment.
생성 문서 경로를 Docker 컨텍스트에서 제외해 빌드 재현성을 높여주세요.
현재 규칙만으로는 src/main/resources/static/docs가 포함될 수 있습니다. Line 11의 -x copyApiDocument와 결합되면 로컬 잔여 산출물이 이미지에 들어갈 여지가 있습니다.
제안 diff
.env
.env.local
+src/main/resources/static/docs/관련 개념: Docker build context 최소화/재현 가능한 빌드(Reproducible Build) 가이드도 함께 참고해 주세요.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| docs/ | |
| *.md | |
| !README.md | |
| .github | |
| .env | |
| .env.local | |
| docs/ | |
| *.md | |
| !README.md | |
| .github | |
| .env | |
| .env.local | |
| src/main/resources/static/docs/ |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.dockerignore around lines 16 - 21, 현재 .dockerignore의 규칙(docs/, *.md,
!README.md 등)만으로는 src/main/resources/static/docs 하위의 생성 문서가 도커 빌드 컨텍스트에 포함되어
이미지에 들어갈 수 있습니다; .dockerignore에 명시적으로 src/main/resources/static/docs/ 혹은
**/static/docs/ 같은 패턴을 추가해 생성 산출물 경로를 제외하고, 기존 예외(!README.md)와 충돌하지 않도록
확인하세요(참조: 현재 파일의 docs/, *.md, !README.md 규칙 및 빌드 스크립트의 -x copyApiDocument 동작을 함께
고려).
apptie
left a comment
apptie
left a comment
There was a problem hiding this comment.
고생하셨습니다
간단한 수정사항이 있어 RC 드렸습니다
| ENTRYPOINT ["sh", "-c", "\ | ||
| if [ -n \"${PINPOINT_AGENT_PATH}\" ] && [ -f \"${PINPOINT_AGENT_PATH}/pinpoint-bootstrap-${PINPOINT_AGENT_VERSION}.jar\" ]; then \ | ||
| exec java \ | ||
| -javaagent:${PINPOINT_AGENT_PATH}/pinpoint-bootstrap-${PINPOINT_AGENT_VERSION}.jar \ |
There was a problem hiding this comment.
필수
현재 pinpoint apm을 활용해 느리게 실행되는 api나 쿼리를 측정하는 것이 중요한 상황입니다
특히 우리 서비스가 통계 서비스라 자칫하면 쿼리가 과하게 느리게 실행될 수 있습니다
그래서 지금처럼 if문 분기를 통해 pinpoint가 없으면 그냥 일반적인 자바 실행보다는
pinpoint가 없으면 실패하는 것이 자연스러워 보입니다
| COPY src ./src | ||
| RUN ./gradlew clean bootJar -x test -x copyApiDocument --no-daemon | ||
|
|
||
| FROM amazoncorretto:21-headless |
There was a problem hiding this comment.
필수
amazoncorretto:21-al2023-headless
이걸로 바꿔야되네요
슬랙 봇 쪽도 해당 이미지로 변경하도록 하겠습니다
3 participants
관련 이슈 번호
이 PR을 통해 해결하려는 문제가 무엇인가요?
서버 환경 변경 시에도 안정적으로 애플리케이션을 실행하기 위해 Docker를 적용합니다
이 PR에서 핵심적으로 변경된 사항은 무엇일까요?
핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?
Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요