Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
…o chore/#186/image-transform-infra
gichanGim
left a comment
There was a problem hiding this comment.
수고하셨습니다~~ 몇가지 코멘트 남겨놓았으니 확인 한번 해주시면 감사할 것 같습니다!
There was a problem hiding this comment.
ci-image-pipeline-infra.yml이 worker 테스트랑 ZIP 빌드까지 다 하고 있는데, ci-image-transform-worker.yml이랑 트리거 경로(worker/, media-specs/)가 겹쳐서 worker 코드만 고쳐도 두 워크플로가 같은 npm test랑 package:lambda를 각각 돌리는 것 같습니다.
인프라 CI는 infra/media/** 에서만 돌게 하고 worker 쪽은 기존 워크플로에 맡기거나, 반대로 하나로 합치는 게 어떨까요?
There was a problem hiding this comment.
같은 테스트를 두 번 돌리던 부분은 제거했습니다!
다만 테스트가 통과해도 배포 파일을 만드는 과정에서 문제가 생길 수 있어서, ZIP 생성과 SAM 검증은 남겨뒀습니다.
| Role: !GetAtt ImageTransformFunctionRole.Arn | ||
| Architectures: | ||
| - x86_64 | ||
| AutoPublishAlias: live |
There was a problem hiding this comment.
alias로 버전 관리하시는 거 롤백에 좋을 것 같은데, 오래된 버전 정리 계획이 있는지 궁금합니다!
AutoPublishAlias를 쓰면 배포할 때마다 새 Lambda 버전이 published되고, 명시적으로 지우지 않으면 계속 쌓이는 걸로 알고 있어서, dev에서 자주 배포하면 나중에 계정 코드 스토리지를 넘을수도 있을 것 같습니다.
README "장애와 rollback" 절에 몇 개까지 보관하고 어떻게 정리할지 한 줄 정도 있으면나중에 헷갈리지 않을 것 같아요!
There was a problem hiding this comment.
문제 생겼을 때 이전 버전으로 돌아갈 수 있도록 최신 5개와 최근 30일 버전은 남기려고 합니다.
사용 중이거나 복구에 필요한 버전도 지우지 않도록 README에 적었습니다.
매번 수동으로 정리하는 건 번거로워서, 나중에 자동 정리는 별도 작업으로 추가하려고 합니다.
437dde1 to
dd94616
Compare
5aa9956 to
a988a8b
Compare
🔥 작업 내용 (What)
이미지 변환에 필요한 S3·SQS·Lambda와 배포 절차를 AWS SAM으로 관리하도록 추가했습니다. 기존 이미지 전달용 S3와 CloudFront는 그대로 사용합니다. 실제 AWS 리소스를 생성한 것은 아닙니다.
🧩 변경 사항 (Details)
리뷰 반영
b459d7c를 반영했습니다.npm test는 제거했습니다.문서 후속 보완
7449b7c에서 수동 정리가 임시 절차임을 명시했습니다. 실행 코드와 AWS 권한은 바꾸지 않았습니다.79bdb4b기준이며, 이번 문서 보완은 diff 검사와 독립 검토를 진행했습니다.테스트
79bdb4b의 SAM CI에서 Linux package 생성·Sharp 로딩·SAM lint/build 통과.gradlew.bat build --no-daemon --max-workers=1통과: 서버 테스트 234개, 실패·오류·건너뛴 테스트 0개.git diff --check통과. 이번 로컬 환경에서는 actionlint를 별도로 실행하지 않았습니다.📸 스크린샷
화면 변경은 없습니다.
develop기준 Java CI도 확인해야 합니다.🔗 관련 이슈
Closes #186
선행 PR: #185
✅ 체크리스트