Skip to content

Update deployment script for app directory creation#22

Merged
hiho1010 merged 1 commit into
mainfrom
dev
Dec 22, 2025
Merged

Update deployment script for app directory creation#22
hiho1010 merged 1 commit into
mainfrom
dev

Conversation

@hiho1010

@hiho1010 hiho1010 commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 배포 프로세스의 파일 시스템 초기화 및 권한 설정 개선

✏️ Tip: You can customize this high-level summary in your review settings.

@hiho1010 hiho1010 merged commit 148d851 into main Dec 22, 2025
3 of 4 checks passed
@coderabbitai

coderabbitai Bot commented Dec 22, 2025

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

프로덕션 배포 스크립트에서 애플리케이션 디렉토리로 이동하기 전에 /srv/todaybook/${APP_NAME} 디렉토리를 생성하고 ssm-user에게 소유권을 할당하는 런타임 파일시스템 설정을 추가합니다.

Changes

응집체 / 파일 변경 요약
배포 워크플로우 구성
\.github/workflows/cd-release-deploy.yml``
프로덕션 배포 단계에서 deploy.sh 실행 전에 애플리케이션 디렉토리를 생성하고 소유권을 설정하는 파일시스템 초기화 로직 추가

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5분

  • 단일 워크플로우 파일 수정
  • 반복 가능한 셸 명령어 추가로 영향 범위 제한
  • 배포 전 초기화 로직으로 기존 파이프라인 동작에 직접적인 영향 없음

Possibly related PRs

Poem

🐰 배포 전 디렉토리를 준비하는 작은 손길,
권한을 다시 맞추고, 경로를 열어주지.
셸 스크립트의 마법으로 배포가 순탄하리,
파일 시스템이 안녕한 그곳에서! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 18f4ca6 and 3c7577f.

📒 Files selected for processing (1)
  • .github/workflows/cd-release-deploy.yml

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

hiho1010 added a commit that referenced this pull request Dec 22, 2025
Merge pull request #22 from today-book/dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant