[fix][[program-service] createSchedule 시 totalCapacity 입력 제거 및 remainingCount 값과 일치화, deploy 설정 수정#29
Conversation
- CreateScheduleRequest: totalCapacity 필드 제거 - CreateScheduleCommand: totalCapacity 필드 제거 - ProgramCommandService.createSchedule(): venueValidation.totalCapacity()를 totalCapacity로 직접 사용 - UpdateScheduleRequest / UpdateScheduleCommand: totalCapacity 필드 제거 - ProgramCommandService.updateSchedule(): totalCapacity 관련 검증 제거
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: first-ticket/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
haeun228
left a comment
haeun228
left a comment
There was a problem hiding this comment.
도커파일이랑 워크플로우는 어떤 이유로 수정하신 걸까요?
CI 실패하는거 수정하시고 머지하시면 될거같아요
| - name: Fill in the new image ID in the Amazon ECS task definition | ||
| id: render-task-def | ||
| uses: aws-actions/amazon-ecs-render-task-definition@v1 | ||
| with: | ||
| task-definition: task-definition.json | ||
| container-name: venue-service | ||
| image: ${{ steps.ecr-login.outputs.registry }}/${{ env.ECR_REPOSITORY }}:${{ github.sha }} | ||
|
|
||
|
|
There was a problem hiding this comment.
task definition에 제일 최신 이미지를 사용하도록 설정해놔서 이 부분은 필요 없습니다
There was a problem hiding this comment.
아 latest를 푸시하는 건 배포가 제대로 적용되지 않을 수 있다고 들어서 적용했습니다..! Dockerfile에 이상한 부분 때문에 빌드가 실패한 것 같습니다.. 이 부분은 문제 없이 되긴하는데 latest로 통일하겠습니다
| task-definition: task-definition.json | ||
| service: program-service | ||
| task-definition: ${{ steps.render-task-def.outputs.task-definition }} | ||
| service: venue-service |
There was a problem hiding this comment.
어쩐지 venue는 배포가 잘 되더라구요 수정하겠습니다!
| - name: Download task definition | ||
| run: | | ||
| aws ecs describe-task-definition --task-definition program-service \ | ||
| --query taskDefinition > task-definition.json | ||
|
|
||
| - name: Download task definition | ||
| run: | | ||
| aws ecs describe-task-definition --task-definition program-service \ | ||
| --query taskDefinition > task-definition.json | ||
|
|
|
API 변경한 대로 테스트 코드도 수정하셔야 CI 통과될 것 같습니다 |
2 participants
🌱 설명
📌 관련 이슈
💻 커밋 유형
📝 체크리스트
📚 추가 설명