Skip to content

fix: dev/prod 배포 수정 (#42) - #44

Merged
young0206 merged 2 commits into
developfrom
fix/42-prod-dev
Jul 22, 2026
Merged

young0206 merged 2 commits into
developfrom
fix/42-prod-dev

Conversation

@young0206

@young0206 young0206 commented Jul 22, 2026 •

Copy link
Copy Markdown
Contributor

🔗 관련 이슈 (Related Issue)

Closes #42

📝 작업 내용

  • Dev/Prod CD workflow 수정

    • JAR 파일명 변경에 맞춰 배포 대상 파일 경로 수정
    • 배포 시 테스트 실행하도록 Gradle 빌드 단계 수정
    • Prod 배포 workflow의 cancel-in-progress 설정 변경으로 진행 중인 배포 취소 방지
    • 배포 안정성을 위해 build 결과물 확인 단계 추가
  • 환경별 Spring Profile 설정 추가

    • application-dev.yml 추가
      • dev 환경 Swagger 활성화 설정 추가
    • application-prod.yml 추가
      • prod 환경 Swagger 비활성화 설정 추가
  • 배포 스크립트 JAR 경로 변경

    • 버전이 포함된 JAR 파일명 대신 고정된 slatto.jar 사용하도록 수정
    • Dev/Prod 서버 deploy script의 JAR 경로 변경

✅ PR 체크리스트

  • PR 제목은 커밋 컨벤션을 따랐습니다.
  • 관련 이슈를 연결했습니다.
  • 변경 사항에 대한 테스트를 진행했습니다.

Summary by CodeRabbit

  • 새로운 기능

    • Dev 환경에서 Swagger UI와 API 문서가 활성화됩니다.
  • 버그 수정

    • Dev 배포 시 테스트를 포함해 애플리케이션 패키지를 생성하도록 개선했습니다.
    • 배포 아티팩트 경로를 표준화했습니다.
  • 개선

    • Prod 환경에서는 Swagger UI와 API 문서가 비활성화됩니다.
    • Prod 배포 시 빌드 결과를 확인할 수 있습니다.

@young0206 young0206 self-assigned this Jul 22, 2026
@young0206 young0206 added the bug Something isn't working label Jul 22, 2026
@coderabbitai

coderabbitai Bot commented Jul 22, 2026 •

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 35e78b5e-50f6-40be-b120-460946819928

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Dev·Prod 배포 워크플로의 JAR 빌드 및 전달 방식이 변경되고, 환경별 Springdoc 노출 설정이 추가되었습니다.

Changes

배포 및 환경 설정

Layer / File(s) Summary
JAR 빌드 및 전달 변경
.github/workflows/cd-dev.yml, .github/workflows/cd-prod.yml
Dev 빌드에서 테스트를 실행하고 slatto.jar를 EC2로 복사하며, Prod 빌드 후 build/libs 내용을 출력합니다.
환경별 Springdoc 설정
src/main/resources/application-dev.yml, src/main/resources/application-prod.yml
Dev 프로파일에서는 Swagger UI와 API 문서를 활성화하고, Prod 프로파일에서는 비활성화합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • SLAT-TO/SLAT-TO-BE#38: cd-dev.yml의 JAR 빌드 및 EC2 복사 경로 변경과 직접 관련됩니다.
  • SLAT-TO/SLAT-TO-BE#41: cd-prod.yml의 빌드 결과 확인 단계와 동일한 프로덕션 워크플로를 다룹니다.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 주요 변경인 Dev/Prod 배포 수정과 JAR 경로 조정이 제목에 잘 반영되어 있습니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 체크리스트가 템플릿 형식을 거의 모두 충족합니다.
Linked Issues check ✅ Passed 이슈 #42의 핵심인 JAR 경로 문제와 prod 배포 안정성 개선 요구를 반영했습니다.
Out of Scope Changes check ✅ Passed 추가된 dev/prod Springdoc 프로파일 설정은 PR 목표에 포함되어 있어 범위를 벗어나지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/cd-prod.yml:
- Around line 39-40: Update the “Check build output” step in the production
workflow to explicitly verify that build/libs/slatto.jar exists, and fail the
step immediately when it is missing; retain directory listing only as optional
diagnostic output.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 88d75cfc-321e-441c-ad62-3c934ce334be

📥 Commits

Reviewing files that changed from the base of the PR and between cb016d8 and 801c560.

📒 Files selected for processing (4)
  • .github/workflows/cd-dev.yml
  • .github/workflows/cd-prod.yml
  • src/main/resources/application-dev.yml
  • src/main/resources/application-prod.yml

Comment thread .github/workflows/cd-prod.yml Outdated

@chazy-d chazy-d left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

jar 파일명 수정 확인했습니다!

@young0206
young0206 merged commit fe1f6fc into develop Jul 22, 2026
2 checks passed
@young0206
young0206 deleted the fix/42-prod-dev branch July 22, 2026 13:11
@coderabbitai coderabbitai Bot mentioned this pull request Jul 22, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FIX: JAR 파일명 변경에 따른 Dev/Prod 배포 실패 수정

2 participants