-
Notifications
You must be signed in to change notification settings - Fork 8
chore: 워크플로우 경로, 환경변수 변경 #291
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
Conversation
|
""" Walkthrough
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (1)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration 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.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/workflows/dev-cd.yml (2)
38-46: 1. 개발 서버 시크릿 및 디렉토리 경로 확인
- 워크플로우가 참조하는DEV_HOST,DEV_USERNAME,DEV_PRIVATE_KEY시크릿이 리포에 정확히 설정되어 있는지 확인하세요.
-target: "/home/${{ secrets.DEV_USERNAME }}/solid-connection-dev/"디렉토리가 원격 서버에 존재하고 권한이 적절한지 점검해야 합니다.
-appleboy/scp-action@master대신 명시적인 릴리스 태그로 버전을 고정해 보안과 안정성을 강화하는 것을 고려하세요.
65-76: 4. 원격에서 Docker Compose 명령 실행 검토
- 원격 서버에docker composeCLI(Compose V2)가 설치되어 있는지 확인하세요.
-script_stop: true설정으로 오류 발생 시 워크플로우가 즉시 중단되는 것이 의도된 동작인지 검토해야 합니다.
- 필요 시docker compose pull또는 자원 최적화를 위한docker system prune단계 추가를 고려하세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/dev-cd.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build
🔇 Additional comments (2)
.github/workflows/dev-cd.yml (2)
47-55: 2. Dockerfile 전송 설정 검토
- 앞서 언급한 시크릿과 디렉토리 설정이 이 단계에도 동일하게 적용되는지 재확인하세요.
-source: "./Dockerfile"경로가 올바른지, 프로젝트 루트에Dockerfile이 위치하는지 확인해야 합니다.
56-64: 3. Compose 파일 전송 및 파일명 일관성
-source: "./docker-compose.dev.yml"파일명이 로컬과 리포지토리의 실제 파일명과 일치하는지 검증하세요.
-target디렉토리 경로가 앞단과 동일한 구조로 지정되었는지 점검해야 합니다.
stage -> dev 변경
nayonsoso
left a comment
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.
수고하셨습니다 🥳
관련 이슈
작업 내용
폴더명과 환경변수명의 환경 이름을 추가적으로 변경했습니다
특이 사항
리뷰 요구사항 (선택)